fastapi-rtk 2.19.0__tar.gz → 2.19.1__tar.gz

{fastapi_rtk-2.19.0 → fastapi_rtk-2.19.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: fastapi-rtk
-Version: 2.19.0
+Version: 2.19.1
 Summary: A package that provides a set of tools to build a FastAPI application with a Class-Based CRUD API.
 Project-URL: Homepage, https://codeberg.org/datatactics/fastapi-rtk
 Project-URL: Issues, https://codeberg.org/datatactics/fastapi-rtk/issues

fastapi_rtk-2.19.1/fastapi_rtk/_version.py

@@ -0,0 +1 @@
+__version__ = "2.19.1"

{fastapi_rtk-2.19.0 → fastapi_rtk-2.19.1}/fastapi_rtk/backends/sqla/extensions/audit/_lifecycle_service.py

@@ -256,6 +256,8 @@ class AuditLifecycleService:
             session (sqlalchemy.orm.Session): The SQLAlchemy session that is about to be committed.
         """
         session_key = _session_key(session)
+        # A flag left by an earlier commit would send this transaction's entries to a new session.
+        self.audit._session_is_commited.pop(session_key, None)
         entries = self.audit._session_pending_entries.pop(session_key, [])
         if not entries:
             return

{fastapi_rtk-2.19.0 → fastapi_rtk-2.19.1}/tests/unit/backends/sqla/test_audit.py

@@ -667,6 +667,22 @@ class TestAuditLifecycleIntegration:
         rows = self._audit_for(engine, AuditTable, 105)
         assert rows == []
 
+    def test_second_commit_on_same_session_writes_audit_row(
+        self, lifecycle_engine_and_models
+    ):
+
+        engine, Tracked, _, AuditTable = lifecycle_engine_and_models
+        with sqlalchemy.orm.Session(engine) as s:
+            s.add(Tracked(id=108, name="first"))
+            s.flush()
+            s.commit()
+            s.add(Tracked(id=109, name="second"))
+            s.flush()
+            s.commit()
+
+        rows = self._audit_for(engine, AuditTable, 109)
+        assert [r.operation for r in rows] == [AuditOperation.INSERT]
+
     def test_coalesced_repeat_updates_become_single_entry(
         self, lifecycle_engine_and_models
     ):
@@ -946,8 +962,9 @@ class TestBeforeCommitWithoutExistingSession:
 
 class TestSessionStateIsWeaklyKeyed:
     """The session-keyed class state must not keep a session alive after it is
-    dropped: `_after_commit` sets `_session_is_commited` without ever popping
-    it, so a strong key would retain every committed session forever."""
+    dropped: `_after_commit` sets `_session_is_commited` and only a later
+    commit on the same session pops it, so a strong key would retain every
+    session that committed once forever."""
 
     def test_committed_sessions_are_released(self, lifecycle_engine_and_models):
         engine, Tracked, _, _ = lifecycle_engine_and_models

fastapi_rtk-2.19.0/fastapi_rtk/_version.py

@@ -1 +0,0 @@
-__version__ = "2.19.0"