This is an automated email from the ASF dual-hosted git repository. diegopucci pushed a commit to branch geido/refactor/native-filters-form in repository https://gitbox.apache.org/repos/asf/superset.git
commit 4134228d515a14cc7601fd1d32de2b1379917133 Merge: f9d9c99ee9 6ede3271ff Author: Diego Pucci <[email protected]> AuthorDate: Tue Oct 15 16:36:19 2024 +0300 Merge branch 'master' of https://github.com/apache/superset into geido/refactor/native-filters-form docs/static/resources/openapi.json | 3 +- superset-frontend/jest.config.js | 2 +- superset-frontend/package-lock.json | 455 ++++++++++++--------- superset-frontend/package.json | 8 +- .../superset-ui-chart-controls/package.json | 6 +- .../src/utils/D3Formatting.ts | 2 + .../packages/superset-ui-core/package.json | 38 +- .../factories/createMemoryFormatter.ts | 55 +++ .../superset-ui-core/src/number-format/index.ts | 1 + .../superset-ui-core/src/query/types/Query.ts | 1 + .../test/chart/clients/ChartClient.test.ts | 2 +- .../test/connection/SupersetClient.test.ts | 8 +- .../test/connection/SupersetClientClass.test.ts | 37 +- .../test/connection/callApi/callApi.test.ts | 78 ++-- .../callApi/callApiAndParseWithTimeout.test.ts | 13 +- .../test/connection/callApi/parseResponse.test.ts | 14 +- .../factories/createMemoryFormatter.test.ts | 94 +++++ .../test/number-format/index.test.ts | 2 + .../query/api/legacy/getDatasourceMetadata.test.ts | 4 +- .../test/query/api/legacy/getFormData.test.ts | 4 +- .../test/query/api/setupClientForTest.ts | 1 + .../test/query/api/v1/getChartData.test.ts | 4 +- .../test/query/api/v1/makeApi.test.ts | 4 +- .../test/time-comparison/fetchTimeRange.test.ts | 20 +- .../legacy-plugin-chart-partition/package.json | 2 +- .../plugins/plugin-chart-table/package.json | 6 +- superset-frontend/spec/helpers/testing-library.tsx | 2 +- .../ShareSqlLabQuery/ShareSqlLabQuery.test.tsx | 2 +- .../SqlLab/components/SouthPane/SouthPane.test.tsx | 2 +- .../AlteredSliceTag/AlteredSliceTag.test.jsx | 2 +- .../ErrorMessage/InvalidSQLErrorMessage.test.tsx | 2 +- .../ErrorMessage/MarshmallowErrorMessage.test.tsx | 2 +- .../FiltersConfigModal/FiltersConfigModal.test.tsx | 4 +- .../tests/CurrentCalendarFrame.test.tsx | 2 +- .../ColumnSelectPopover.test.tsx | 2 +- .../OAuth2ClientField.test.tsx | 2 +- .../databases/DatabaseModal/index.test.tsx | 2 + .../UploadDataModel/UploadDataModal.test.tsx | 6 +- superset-frontend/src/setup/setupErrorMessages.ts | 4 + superset-frontend/src/setup/setupFormatters.ts | 5 +- superset/config.py | 3 + superset/errors.py | 3 + superset/sql/parse.py | 108 ++++- superset/sql_lab.py | 36 ++ tests/integration_tests/sql_lab/api_tests.py | 2 +- tests/unit_tests/db_engine_specs/test_base.py | 16 +- tests/unit_tests/sql/parse_tests.py | 34 ++ tests/unit_tests/sql_lab_test.py | 115 +++++- 48 files changed, 847 insertions(+), 373 deletions(-) diff --cc tests/integration_tests/sql_lab/api_tests.py index e6c1f686b1,cf1e190bbb..e1ea4cd395 --- a/tests/integration_tests/sql_lab/api_tests.py +++ b/tests/integration_tests/sql_lab/api_tests.py @@@ -281,10 -281,10 +281,10 @@@ class TestSqlLabApi(SupersetTestCase) "/api/v1/sqllab/format_sql/", json=data, ) - success_resp = {"result": "SELECT\n 1\nFROM my_table"} + success_resp = {"result": "SELECT 1\nFROM my_table"} resp_data = json.loads(rv.data.decode("utf-8")) - self.assertDictEqual(resp_data, success_resp) # noqa: PT009 - assert rv.status_code == 200 + self.assertDictEqual(resp_data, success_resp) + self.assertEqual(rv.status_code, 200) @mock.patch("superset.commands.sql_lab.results.results_backend_use_msgpack", False) def test_execute_required_params(self):
