chore(tests): Fix MySQL logic

apache · Jun 25, 2024 · 7425e77 · 7425e77
1 parent 31afb62
commit 7425e77
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 8 deletions.
diff --git a/tests/integration_tests/databases/commands_tests.py b/tests/integration_tests/databases/commands_tests.py
@@ -1108,9 +1108,6 @@ def test_database_tables_superset_exception(
         self, mock_g, mock_can_access_database, mock_find_by_id
     ):
         database = get_example_database()
-        if database.backend == "mysql":
-            return
-
         mock_find_by_id.return_value = database
         mock_can_access_database.side_effect = SupersetException("Test Error")
         mock_g.user = security_manager.find_user("admin")

diff --git 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 (