diff --git a/src/chart_types/xy_chart/legend/legend.test.ts b/src/chart_types/xy_chart/legend/legend.test.ts index 043e1c9086..13a9daea75 100644 --- a/src/chart_types/xy_chart/legend/legend.test.ts +++ b/src/chart_types/xy_chart/legend/legend.test.ts @@ -216,5 +216,41 @@ describe('Legends', () => { expect(label).toBe('a - b'); label = getSeriesColorLabel(['a', 'b'], false, spec2); expect(label).toBe('a - b'); + + label = getSeriesColorLabel([null], true); + expect(label).toBeUndefined(); + label = getSeriesColorLabel([null], true, spec1); + expect(label).toBe('Spec 1 title'); + label = getSeriesColorLabel([null], true, spec2); + expect(label).toBe('spec2'); + label = getSeriesColorLabel([undefined], true); + expect(label).toBeUndefined(); + label = getSeriesColorLabel([undefined], true, spec1); + expect(label).toBe('Spec 1 title'); + label = getSeriesColorLabel([undefined], true, spec2); + expect(label).toBe('spec2'); + + label = getSeriesColorLabel([0], true); + expect(label).toBeUndefined(); + label = getSeriesColorLabel([0], true, spec1); + expect(label).toBe('Spec 1 title'); + + label = getSeriesColorLabel([null], false); + expect(label).toBeUndefined(); + label = getSeriesColorLabel([null], false, spec1); + expect(label).toBe('Spec 1 title'); + label = getSeriesColorLabel([null], false, spec2); + expect(label).toBe('spec2'); + label = getSeriesColorLabel([undefined], false); + expect(label).toBeUndefined(); + label = getSeriesColorLabel([undefined], false, spec1); + expect(label).toBe('Spec 1 title'); + label = getSeriesColorLabel([undefined], false, spec2); + expect(label).toBe('spec2'); + + label = getSeriesColorLabel([0], false); + expect(label).toBe('0'); + label = getSeriesColorLabel([0], false, spec1); + expect(label).toBe('0'); }); }); diff --git a/src/chart_types/xy_chart/legend/legend.ts b/src/chart_types/xy_chart/legend/legend.ts index 511ca1b593..72e41e04bd 100644 --- a/src/chart_types/xy_chart/legend/legend.ts +++ b/src/chart_types/xy_chart/legend/legend.ts @@ -71,8 +71,7 @@ export function getSeriesColorLabel( spec?: BasicSeriesSpec, ): string | undefined { let label = ''; - - if (hasSingleSeries || colorValues.length === 0 || !colorValues[0]) { + if (hasSingleSeries || colorValues.length === 0 || colorValues[0] == null) { if (!spec) { return; }