sadpandajoe commented on code in PR #43729:
URL: https://github.com/apache/superset/pull/43729#discussion_r3909433535
##########
superset-frontend/plugins/plugin-chart-echarts/test/Timeseries/transformProps.test.ts:
##########
@@ -2877,6 +2877,77 @@ test('boundary label alignment is dropped when the
orientation moves the time ax
expect(horizontal.axisLabel.alignMaxLabel).toBeUndefined();
// The boundary labels themselves stay forced in both orientations.
+
+describe('xAxisLabelInterval string "0" is converted to number 0', () => {
Review Comment:
Agreed—the nested `describe` runs while the enclosing boundary-label test is
executing, so Jest rejects the file and none of the new regression assertions
run. Could this block be moved to module scope with its fixture available there?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
