Skip to content

Commit

Permalink
Fix unit tests
Browse files Browse the repository at this point in the history
- chartTheme was null
  • Loading branch information
mofojed committed Nov 10, 2023
1 parent e87ae11 commit 438ed4f
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions packages/chart/src/FigureChartModel.test.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import dh from '@deephaven/jsapi-shim';
import { TestUtils } from '@deephaven/utils';
import { PlotData } from 'plotly.js';
import { Data } from 'plotly.js';
import ChartTestUtils from './ChartTestUtils';
import type { ChartTheme } from './ChartTheme';
import FigureChartModel from './FigureChartModel';
Expand Down Expand Up @@ -468,7 +468,7 @@ it('emits finished loading if no series are added', () => {
const figure = chartTestUtils.makeFigure({
charts: [],
});
const model = new FigureChartModel(dh, figure);
const model = new FigureChartModel(dh, figure, chartTheme);
const callback = jest.fn();
model.subscribe(callback);

Expand All @@ -482,7 +482,7 @@ it('emits finished loading if no series are added', () => {
});

describe('legend visibility', () => {
function testLegend(showLegend: boolean | null): Partial<PlotData>[] {
function testLegend(showLegend: boolean | null): Partial<Data>[] {
const series1 = chartTestUtils.makeSeries({ name: 'S1' });
const chart = chartTestUtils.makeChart({ series: [series1], showLegend });
const figure = chartTestUtils.makeFigure({
Expand Down

0 comments on commit 438ed4f

Please sign in to comment.