Use SingletonThreadPool for in-memory SQLite database

  Fix bug where NullPool is used for in-memory SQLite created using
  filename uri instead of ":memory:".

  Checks for in-memory SQLite database created using filename uri like
  "file:foo.db?mode=memory&uri=true" in addition to in-memory SQLite
  database created using the standard ":memory:".

  Fixes: #6379

diff --git a/lib/sqlalchemy/dialects/sqlite/pysqlite.py b/lib/sqlalchemy/dialects/sqlite/pysqlite.py
index 0b091e73b648fd1d4f6caab7a546cd6e699d1bff..20a4bb7acb2831b030676bc2e1a5a0297e5bde0a 100644 (file)
--- a/lib/sqlalchemy/dialects/sqlite/pysqlite.py
+++ b/lib/sqlalchemy/dialects/sqlite/pysqlite.py
@@ -474,7 +474,9 @@ class SQLiteDialect_pysqlite(SQLiteDialect):
 
     @classmethod
     def _is_url_file_db(cls, url):
-        if url.database and url.database != ":memory:":
+        if (url.database and url.database != ":memory:") and (
+            url.query.get("mode", None) != "memory"
+        ):
             return True
         else:
             return False

diff --git a/test/dialect/test_sqlite.py b/test/dialect/test_sqlite.py
index 7fd7ac333cfa6be329ec1fbdd3ebd9476f317b83..bae97bb9a2ff6394dfbe83dccc831b16bf926e41 100644 (file)
--- a/test/dialect/test_sqlite.py
+++ b/test/dialect/test_sqlite.py
@@ -710,6 +710,11 @@ class DialectTest(
         e = create_engine("sqlite+pysqlite:///:memory:")
         assert e.pool.__class__ is pool.SingletonThreadPool
 
+        e = create_engine(
+            "sqlite+pysqlite:///file:foo.db?mode=memory&uri=true"
+        )
+        assert e.pool.__class__ is pool.SingletonThreadPool
+
         e = create_engine("sqlite+pysqlite:///foo.db")
         assert e.pool.__class__ is pool.NullPool