fix(charts): Time grain is None when dataset uses Jinja (#25842)

This commit is contained in:
Antonio Rivero
2023-11-06 18:51:28 +01:00
committed by GitHub
parent 30cd422331
commit 7536dd12cd
6 changed files with 23 additions and 14 deletions

View File

@@ -121,3 +121,16 @@ test('should fallback to formData.time_grain_sqla if extra_form_data.time_grain_
expressionType: 'SQL',
});
});
test('should not omit extras.time_grain_sqla from queryContext so dashboards apply them', () => {
Object.defineProperty(supersetCoreModule, 'hasGenericChartAxes', {
value: true,
});
const modifiedFormData = {
...formData,
extra_form_data: { time_grain_sqla: TimeGranularity.QUARTER },
};
const queryContext = buildQuery(modifiedFormData);
const [query] = queryContext.queries;
expect(query.extras?.time_grain_sqla).toEqual(TimeGranularity.QUARTER);
});