Skip to content

Commit ae7a947

Browse files
Superset Devclaude
andcommitted
fix: prettier on remaining country-map files
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent cc7ba36 commit ae7a947

2 files changed

Lines changed: 13 additions & 11 deletions

File tree

superset-frontend/plugins/plugin-chart-country-map/src/plugin/index.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -48,13 +48,7 @@ export default class CountryMapChartPlugin extends ChartPlugin {
4848
'and aggregated regional layers.',
4949
),
5050
name: t('Country Map'),
51-
tags: [
52-
t('2D'),
53-
t('Comparison'),
54-
t('Geo'),
55-
t('Range'),
56-
t('Report'),
57-
],
51+
tags: [t('2D'), t('Comparison'), t('Geo'), t('Range'), t('Report')],
5852
// TODO: thumbnail + example images come in a follow-up commit
5953
// (need to render real outputs first).
6054
thumbnail: '',

superset-frontend/plugins/plugin-chart-country-map/test/plugin/controlPanel.test.ts

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -108,9 +108,13 @@ test('country selector visibility hides on Admin 0', () => {
108108
expect(c).not.toBeNull();
109109
// Admin 0 (number 0 OR string "0") AND no composite → hidden
110110
expect(c.visibility({ controls: { admin_level: { value: 0 } } })).toBe(false);
111-
expect(c.visibility({ controls: { admin_level: { value: '0' } } })).toBe(false);
111+
expect(c.visibility({ controls: { admin_level: { value: '0' } } })).toBe(
112+
false,
113+
);
112114
// Admin 1 → visible
113-
expect(c.visibility({ controls: { admin_level: { value: '1' } } })).toBe(true);
115+
expect(c.visibility({ controls: { admin_level: { value: '1' } } })).toBe(
116+
true,
117+
);
114118
// Composite set → hidden regardless of admin_level
115119
expect(
116120
c.visibility({
@@ -128,8 +132,12 @@ test('region_set selector only visible when admin_level === aggregated', () => {
128132
expect(
129133
c.visibility({ controls: { admin_level: { value: 'aggregated' } } }),
130134
).toBe(true);
131-
expect(c.visibility({ controls: { admin_level: { value: '1' } } })).toBe(false);
132-
expect(c.visibility({ controls: { admin_level: { value: '0' } } })).toBe(false);
135+
expect(c.visibility({ controls: { admin_level: { value: '1' } } })).toBe(
136+
false,
137+
);
138+
expect(c.visibility({ controls: { admin_level: { value: '0' } } })).toBe(
139+
false,
140+
);
133141
});
134142

135143
test('region_set choices key off the selected country', () => {

0 commit comments

Comments
 (0)