diff --git a/scripts/python_tests.sh b/scripts/python_tests.sh index c3f27d17f78c..a63e00c77549 100755 --- a/scripts/python_tests.sh +++ b/scripts/python_tests.sh @@ -32,4 +32,5 @@ superset init echo "Running tests" -pytest --durations-min=2 --maxfail=1 --cov-report= --cov=superset ./tests/integration_tests "$@" +#pytest --durations-min=2 --maxfail=1 --cov-report= --cov=superset ./tests/integration_tests "$@" +pytest --durations-min=2 --maxfail=1 tests/integration_tests/celery_tests.py "$@" diff --git a/tests/integration_tests/celery_tests.py b/tests/integration_tests/celery_tests.py index 3bd82211e5da..4dfd3165c83d 100644 --- a/tests/integration_tests/celery_tests.py +++ b/tests/integration_tests/celery_tests.py @@ -256,8 +256,8 @@ def test_run_sync_query_cta_config(test_client, ctas_method): lambda d, u, s, sql: CTAS_SCHEMA_NAME, ) def test_run_async_query_cta_config(test_client, ctas_method): - if backend() in {"sqlite", "mysql"}: - # sqlite doesn't support schemas, mysql is flaky + if backend() == "sqlite": + # sqlite doesn't support schemas return tmp_table_name = f"{TEST_ASYNC_CTA_CONFIG}_{ctas_method.lower()}" result = run_sql( @@ -287,10 +287,6 @@ def test_run_async_query_cta_config(test_client, ctas_method): @pytest.mark.usefixtures("load_birth_names_dashboard_with_slices", "login_as_admin") @pytest.mark.parametrize("ctas_method", [CtasMethod.TABLE, CtasMethod.VIEW]) def test_run_async_cta_query(test_client, ctas_method): - if backend() == "mysql": - # failing - return - table_name = f"{TEST_ASYNC_CTA}_{ctas_method.lower()}" result = run_sql( test_client, @@ -318,10 +314,6 @@ def test_run_async_cta_query(test_client, ctas_method): @pytest.mark.usefixtures("load_birth_names_dashboard_with_slices", "login_as_admin") @pytest.mark.parametrize("ctas_method", [CtasMethod.TABLE, CtasMethod.VIEW]) def test_run_async_cta_query_with_lower_limit(test_client, ctas_method): - if backend() == "mysql": - # failing - return - tmp_table = f"{TEST_ASYNC_LOWER_LIMIT}_{ctas_method.lower()}" result = run_sql( test_client, @@ -331,7 +323,10 @@ def test_run_async_cta_query_with_lower_limit(test_client, ctas_method): async_=True, tmp_table=tmp_table, ) + print(result) query = wait_for_success(result) + print(query) + print(query.to_dict()) assert QueryStatus.SUCCESS == query.status sqlite_select_sql = f"SELECT\n *\nFROM {tmp_table}\nLIMIT {query.limit}\nOFFSET 0" diff --git a/tests/integration_tests/reports/commands_tests.py b/tests/integration_tests/reports/commands_tests.py index e4c64a51b91d..3936888b03ff 100644 --- a/tests/integration_tests/reports/commands_tests.py +++ b/tests/integration_tests/reports/commands_tests.py @@ -1548,8 +1548,6 @@ def test_report_schedule_working_timeout(create_report_slack_chart_working): datetime.utcnow(), ).run() - # Only needed for MySQL, understand why - db.session.commit() logs = db.session.query(ReportExecutionLog).all() # Two logs, first is created by fixture assert len(logs) == 2 @@ -2076,9 +2074,6 @@ def test_grace_period_error(email_mock, create_invalid_sql_alert_email_chart): datetime.utcnow(), ).run() - # Only needed for MySQL, understand why - db.session.commit() - # Assert the email smtp address, asserts a notification was sent with the error assert email_mock.call_args[0][0] == DEFAULT_OWNER_EMAIL assert (