Merge pull request #129 from mschoettle/python313

Add support for Python 3.13

Author: DanCardin

.github/workflows/build.yml

@@ -24,10 +24,10 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        python-version: ["3.9", "3.10", "3.11", "3.12"]
+        python-version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
         pytest-version: ["7.0.0", "8.3.5"]
         pytest-asyncio-version: ["0.16.0", "0.23.0"]
-        sqlalchemy-version: ["1.4.0", "2.0.40"]
+        sqlalchemy-version: ["2.0.40"]
 
         exclude:
           # old versions of pytest-asyncio use old pytest hook syntax

src/pytest_alembic/executor.py

@@ -183,6 +183,10 @@ async def run(engine):
 
         if isinstance(self.connection, Engine):
             with self.connection.begin() as connection:
-                return fn(connection=connection, **kwargs)
+                result = fn(connection=connection, **kwargs)
+
+            # SQLite does not close via the context manager, close expliclitly
+            connection.close()
+            return result
 
         return fn(connection=self.connection, **kwargs)

src/pytest_alembic/plugin/fixtures.py

@@ -122,4 +122,8 @@ def alembic_config() -> Union[Dict[str, Any], alembic.config.Config, Config]:
 @pytest.fixture()
 def alembic_engine():
     """Override this fixture to provide pytest-alembic powered tests with a database handle."""
-    return sqlalchemy.create_engine("sqlite:///")
+    engine = sqlalchemy.create_engine("sqlite:///")
+    try:
+        yield engine
+    finally:
+        engine.dispose()