mirror of
https://github.com/apache/superset.git
synced 2026-07-20 05:36:00 +00:00
test_execute_sql_statements and test_execute_sql_statements_ctas mocked `query.database` as a bare MagicMock, so the new pre-split mutation call in execute_sql_statements (which re-parses the mutator's return value as SQL) got back an unconfigured MagicMock instead of a string, leaving zero parsed statements and tripping the "mutator removed all executable statements" guard. Configure mutate_sql_based_on_config to pass the SQL through unchanged, matching the real Database method's default behavior and the pattern already used in the new unit tests. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>