fixed test snapshots and rangeMax.

elastic · Mar 2, 2022 · e5c10d2 · e5c10d2
1 parent ddb1a09
commit e5c10d2
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 3 deletions.
diff --git a/...ns/expression_gauge/common/expression_functions/__snapshots__/gauge_function.test.ts.snap b/...ns/expression_gauge/common/expression_functions/__snapshots__/gauge_function.test.ts.snap
diff --git a/...xpressions/expression_gauge/public/components/__snapshots__/gauge_component.test.tsx.snap b/...xpressions/expression_gauge/public/components/__snapshots__/gauge_component.test.tsx.snap
diff --git a/src/plugins/chart_expressions/expression_gauge/public/components/gauge_component.test.tsx b/src/plugins/chart_expressions/expression_gauge/public/components/gauge_component.test.tsx
@@ -54,6 +54,7 @@ jest.mock('@elastic/charts', () => {
 });
 
 const chartsThemeService = chartPluginMock.createSetupContract().theme;
+const paletteThemeService = chartPluginMock.createSetupContract().palettes;
 const formatService = fieldFormatsServiceMock.createStartContract();
 const args: GaugeArguments = {
   labelMajor: 'Gauge',
@@ -81,12 +82,13 @@ const createData = (
 describe('GaugeComponent', function () {
   let wrapperProps: GaugeRenderProps;
 
-  beforeAll(() => {
+  beforeAll(async () => {
     wrapperProps = {
       data: createData(),
       chartsThemeService,
       args,
       formatFactory: formatService.deserialize,
+      paletteService: await paletteThemeService.getPalettes(),
     };
   });
 

diff --git a/src/plugins/vis_types/gauge/public/to_ast.ts b/src/plugins/vis_types/gauge/public/to_ast.ts
@@ -81,8 +81,8 @@ export const toExpressionAst: VisToExpressionAst<GaugeVisParams> = (vis, params)
       range: percentageMode ? 'percent' : 'number',
       continuity: 'none',
       gradient: true,
-      rangeMax: percentageMode ? 100 : Infinity,
-      rangeMin: 0,
+      rangeMax: percentageMode ? 100 : stopsWithColors.stop[stopsWithColors.stop.length - 1],
+      rangeMin: stopsWithColors.stop[0],
     });
 
     gauge.addArgument('palette', buildExpression([palette]));