Stop caching in-memory databases

Author: dan-fernandes

tiled/_tests/test_sync.py

@@ -1,7 +1,6 @@
 import asyncio
 import contextlib
 import tempfile
-import uuid
 
 import awkward
 import h5py
@@ -25,7 +24,6 @@
 def client_factory(readable_storage=None):
     with tempfile.TemporaryDirectory() as tempdir:
         catalog = in_memory(
-            named_memory=str(uuid.uuid4())[:8],
             writable_storage=str(tempdir),
             readable_storage=readable_storage,
         )

tiled/_tests/test_validation.py

@@ -1,8 +1,6 @@
 """
 This tests tiled's validation registry
 """
-import uuid
-
 import numpy as np
 import pandas as pd
 import pytest
@@ -108,7 +106,6 @@ def test_unknown_spec_strict(tmpdir):
                 "path": "/",
                 "args": {
                     "writable_storage": str(tmpdir),
-                    "named_memory": str(uuid.uuid4())[:8],
                 },
             },
         ],

tiled/server/app.py

@@ -473,7 +473,7 @@ def override_get_settings():
             # not set, use a SQLite database in memory. Horizontally scaled deployments
             # must specify a persistent database.
             settings.database_settings.uri = (
-                settings.database_settings.uri or "sqlite://"
+                settings.database_settings.uri or "sqlite:///file:authdb?mode=memory&cache=shared&uri=true"
             )
         if (
             authenticators

tiled/server/connection_pool.py

@@ -73,9 +73,9 @@ def open_database_connection_pool(database_settings: DatabaseSettings) -> AsyncE
             pool_pre_ping=database_settings.pool_pre_ping,
         )
 
-    # Cache the engine so we don't create more than one pool per database_settings.
-    monitor_db_pool(engine.pool, sanitize_uri(database_settings.uri)[0])
-    _connection_pools[database_settings] = engine
+        # Cache the engine so we don't create more than one pool per database_settings.
+        monitor_db_pool(engine.pool, sanitize_uri(database_settings.uri)[0])
+        _connection_pools[database_settings] = engine
 
     # For SQLite, ensure that foreign key constraints are enforced.
     if engine.dialect.name == "sqlite":