fix(plugin-chart-echarts): remove columns from formData

apache-superset · Jan 15, 2021 · dcf0504 · dcf0504
1 parent 2d87d6f
commit dcf0504
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/plugins/plugin-chart-echarts/src/BoxPlot/buildQuery.ts b/plugins/plugin-chart-echarts/src/BoxPlot/buildQuery.ts
@@ -45,6 +45,7 @@ export default function buildQuery(formData: BoxPlotQueryFormData) {
         ...baseQueryObject,
         is_timeseries: distributionColumns.length === 0,
         groupby: (groupby || []).concat(distributionColumns),
+        columns: undefined,
         post_processing: [
           {
             operation: 'boxplot',

diff --git a/plugins/plugin-chart-echarts/test/BoxPlot/buildQuery.test.ts b/plugins/plugin-chart-echarts/test/BoxPlot/buildQuery.test.ts
@@ -38,6 +38,7 @@ describe('BoxPlot buildQuery', () => {
     expect(query.is_timeseries).toEqual(true);
     expect(query.metrics).toEqual(['foo']);
     expect(query.groupby).toEqual(['bar']);
+    expect(query.columns).toEqual(undefined);
   });
 
   it('should build non-timeseries query object when columns is defined', () => {
@@ -46,5 +47,6 @@ describe('BoxPlot buildQuery', () => {
     expect(query.is_timeseries).toEqual(false);
     expect(query.metrics).toEqual(['foo']);
     expect(query.groupby).toEqual(['bar', 'qwerty']);
+    expect(query.columns).toEqual(undefined);
   });
 });