fix: Deque mutated during iteration (#24550)

superset/utils/core.py

@@ -850,23 +850,23 @@ def pessimistic_connection_handling(some_engine: Engine) -> None:
             # restore 'close with result'
             connection.should_close_with_result = save_should_close_with_result
 
-        if some_engine.dialect.name == "sqlite":
+    if some_engine.dialect.name == "sqlite":
 
-            @event.listens_for(some_engine, "connect")
-            def set_sqlite_pragma(  # pylint: disable=unused-argument
-                connection: sqlite3.Connection,
-                *args: Any,
-            ) -> None:
-                r"""
-                Enable foreign key support for SQLite.
+        @event.listens_for(some_engine, "connect")
+        def set_sqlite_pragma(  # pylint: disable=unused-argument
+            connection: sqlite3.Connection,
+            *args: Any,
+        ) -> None:
+            r"""
+            Enable foreign key support for SQLite.
 
-                :param connection: The SQLite connection
-                :param \*args: Additional positional arguments
-                :see: https://docs.sqlalchemy.org/en/latest/dialects/sqlite.html
-                """
+            :param connection: The SQLite connection
+            :param \*args: Additional positional arguments
+            :see: https://docs.sqlalchemy.org/en/latest/dialects/sqlite.html
+            """
 
-                with closing(connection.cursor()) as cursor:
-                    cursor.execute("PRAGMA foreign_keys=ON")
+            with closing(connection.cursor()) as cursor:
+                cursor.execute("PRAGMA foreign_keys=ON")
 
 
 def send_email_smtp(  # pylint: disable=invalid-name,too-many-arguments,too-many-locals

tests/unit_tests/databases/ssh_tunnel/dao_tests.py

@@ -35,7 +35,7 @@ def test_create_ssh_tunnel():
         "password": "bar",
     }
 
-    result = SSHTunnelDAO.create(properties)
+    result = SSHTunnelDAO.create(properties, commit=False)
 
     assert result is not None
     assert isinstance(result, SSHTunnel)