This is an automated email from the ASF dual-hosted git repository.
hainenber pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git
The following commit(s) were added to refs/heads/master by this push:
new 0b77ace1109 chore: fix lint issue with `no-unsafe-optional-chaining`
rule (#38103)
0b77ace1109 is described below
commit 0b77ace11090552c7f26ba2e06754544e917dbfd
Author: Đỗ Trọng Hải <[email protected]>
AuthorDate: Thu Feb 19 23:54:37 2026 +0700
chore: fix lint issue with `no-unsafe-optional-chaining` rule (#38103)
Signed-off-by: hainenber <[email protected]>
---
.../test/MixedTimeseries/transformProps.test.ts | 2 +-
.../plugin-chart-echarts/test/Timeseries/transformProps.test.ts | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git
a/superset-frontend/plugins/plugin-chart-echarts/test/MixedTimeseries/transformProps.test.ts
b/superset-frontend/plugins/plugin-chart-echarts/test/MixedTimeseries/transformProps.test.ts
index 76562999515..57b91f8330a 100644
---
a/superset-frontend/plugins/plugin-chart-echarts/test/MixedTimeseries/transformProps.test.ts
+++
b/superset-frontend/plugins/plugin-chart-echarts/test/MixedTimeseries/transformProps.test.ts
@@ -431,5 +431,5 @@ test('should add a formula annotation when X-axis column
has dataset-level label
expect(formulaSeries).toBeDefined();
expect(formulaSeries?.data).toBeDefined();
expect(Array.isArray(formulaSeries?.data)).toBe(true);
- expect((formulaSeries?.data as unknown[]).length).toBeGreaterThan(0);
+ expect((formulaSeries?.data as unknown[])?.length).toBeGreaterThan(0);
});
diff --git
a/superset-frontend/plugins/plugin-chart-echarts/test/Timeseries/transformProps.test.ts
b/superset-frontend/plugins/plugin-chart-echarts/test/Timeseries/transformProps.test.ts
index 89303f7f1bb..f8384bfbe78 100644
---
a/superset-frontend/plugins/plugin-chart-echarts/test/Timeseries/transformProps.test.ts
+++
b/superset-frontend/plugins/plugin-chart-echarts/test/Timeseries/transformProps.test.ts
@@ -304,8 +304,8 @@ describe('EchartsTimeseries transformProps', () => {
expect(formulaSeries).toBeDefined();
expect(formulaSeries?.data).toBeDefined();
expect(Array.isArray(formulaSeries?.data)).toBe(true);
- expect((formulaSeries?.data as unknown[]).length).toBeGreaterThan(0);
- const firstDataPoint = (formulaSeries?.data as [number, number][])[0];
+ expect((formulaSeries?.data as unknown[])?.length).toBeGreaterThan(0);
+ const firstDataPoint = (formulaSeries?.data as [number, number][])?.[0];
expect(firstDataPoint).toBeDefined();
expect(firstDataPoint[1]).toBe(firstDataPoint[0] * 2);
});
@@ -384,7 +384,7 @@ describe('EchartsTimeseries transformProps', () => {
result.echartOptions.series as SeriesOption[] | undefined
)?.find((s: SeriesOption) => s.name === 'My Formula');
expect(formulaSeries).toBeDefined();
- const firstDataPoint = (formulaSeries?.data as [number, number][])[0];
+ const firstDataPoint = (formulaSeries?.data as [number, number][])?.[0];
expect(firstDataPoint).toBeDefined();
expect(firstDataPoint[0]).toBe(firstDataPoint[1] * 2);
});