hexcore 1.6.2__tar.gz → 1.6.3__tar.gz

{hexcore-1.6.2 → hexcore-1.6.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hexcore
-Version: 1.6.2
+Version: 1.6.3
 Summary: Núcleo reutilizable para proyectos Python con arquitectura hexagonal y event handling. Provee abstracciones, utilidades y contratos para DDD, eventos y desacoplamiento de infraestructura.
 Author-email: "David Latosefki (Indroic)" <indroic@outlook.com>
 License-Expression: MIT

{hexcore-1.6.2 → hexcore-1.6.3}/hexcore/infrastructure/repositories/orms/sqlalchemy/session.py

@@ -23,11 +23,6 @@ AsyncSessionLocal: async_sessionmaker[AsyncSession] = async_sessionmaker(
 
 
 async def get_async_db_session():
-    """Generador de sesiones asíncronas con gestión de transacciones integrada."""
+    """Generador de sesiones asíncronas delegando el ciclo de vida al context manager."""
     async with AsyncSessionLocal() as db:
-        try:
-            yield db
-        except Exception:
-            if db.in_transaction():
-                await db.rollback()
-            raise
+        yield db

{hexcore-1.6.2 → hexcore-1.6.3}/hexcore/infrastructure/uow/__init__.py

@@ -39,16 +39,20 @@ class SqlAlchemyUnitOfWork(IUnitOfWork):
         exc_tb: t.Optional[t.Any],
     ) -> None:
         # El ciclo de vida de la sesion se maneja en la dependencia/factory
-        # que la crea. Aqui solo manejamos rollback en errores.
-        await super().__aexit__(exc_type, exc_val, exc_tb)
+        # que la crea. Evitamos rollback duplicado para no interferir con
+        # el unwind del contexto externo de AsyncSession.
+        if exc_type:
+            self.clear_tracked_entities()
 
     async def commit(self):
         """
         Confirma la transacción y despacha los eventos.
         """
         await self.session.commit()
-        await self.dispatch_events()
-        self.clear_tracked_entities()
+        try:
+            await self.dispatch_events()
+        finally:
+            self.clear_tracked_entities()
 
     async def rollback(self):
         if self.session.in_transaction():

{hexcore-1.6.2 → hexcore-1.6.3}/hexcore.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hexcore
-Version: 1.6.2
+Version: 1.6.3
 Summary: Núcleo reutilizable para proyectos Python con arquitectura hexagonal y event handling. Provee abstracciones, utilidades y contratos para DDD, eventos y desacoplamiento de infraestructura.
 Author-email: "David Latosefki (Indroic)" <indroic@outlook.com>
 License-Expression: MIT

{hexcore-1.6.2 → hexcore-1.6.3}/hexcore.egg-info/SOURCES.txt

@@ -98,4 +98,5 @@ scripts/__init__.py
 scripts/fix_pyi_files.py
 scripts/fix_types_pyi_aliases.py
 scripts/main.py
-tests/test_basic.py
+tests/test_basic.py
+tests/test_uow_session_regression.py

{hexcore-1.6.2 → hexcore-1.6.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "hexcore"
-version = "1.6.2"
+version = "1.6.3"
 authors = [
   { name="David Latosefki (Indroic)", email="indroic@outlook.com" },
 ]

hexcore-1.6.3/tests/test_uow_session_regression.py

@@ -0,0 +1,110 @@
+import asyncio
+from collections.abc import AsyncGenerator
+from types import TracebackType
+from typing import Any, cast
+from unittest.mock import AsyncMock, MagicMock, patch
+
+from hexcore.infrastructure.repositories.orms.sqlalchemy.session import (
+    get_async_db_session,
+)
+from hexcore.infrastructure.uow import SqlAlchemyUnitOfWork
+
+
+class _DummySessionContext:
+    def __init__(self, session: Any) -> None:
+        self._session = session
+
+    async def __aenter__(self) -> Any:
+        return self._session
+
+    async def __aexit__(
+        self,
+        exc_type: type[BaseException] | None,
+        exc_val: BaseException | None,
+        exc_tb: TracebackType | None,
+    ) -> bool:
+        return False
+
+
+def _build_sql_uow(session: Any) -> SqlAlchemyUnitOfWork:
+    with patch("hexcore.infrastructure.uow.discover_sql_repositories", return_value={}):
+        return SqlAlchemyUnitOfWork(session=session)
+
+
+def test_get_async_db_session_no_manual_rollback_on_error():
+    async def _run():
+        session = MagicMock()
+        session.rollback = AsyncMock()
+
+        with patch(
+            "hexcore.infrastructure.repositories.orms.sqlalchemy.session.AsyncSessionLocal",
+            return_value=_DummySessionContext(session),
+        ):
+            session_generator = cast(AsyncGenerator[Any, None], get_async_db_session())
+            yielded = await anext(session_generator)
+            assert yielded is session
+
+            try:
+                await session_generator.athrow(ValueError("boom"))
+            except ValueError:
+                pass
+
+        session.rollback.assert_not_awaited()
+
+    asyncio.run(_run())
+
+
+def test_sql_uow_aexit_does_not_rollback_session():
+    async def _run():
+        session = MagicMock()
+        session.rollback = AsyncMock()
+
+        uow = _build_sql_uow(session)
+        uow.clear_tracked_entities = MagicMock()
+
+        await uow.__aexit__(ValueError, ValueError("boom"), None)
+
+        session.rollback.assert_not_awaited()
+        uow.clear_tracked_entities.assert_called_once()
+
+    asyncio.run(_run())
+
+
+def test_sql_uow_rollback_keeps_explicit_rollback_behavior():
+    async def _run():
+        session = MagicMock()
+        session.in_transaction.return_value = True
+        session.rollback = AsyncMock()
+
+        uow = _build_sql_uow(session)
+        uow.clear_tracked_entities = MagicMock()
+
+        await uow.rollback()
+
+        session.rollback.assert_awaited_once()
+        uow.clear_tracked_entities.assert_called_once()
+
+    asyncio.run(_run())
+
+
+def test_sql_uow_commit_clears_entities_even_if_dispatch_fails():
+    async def _run():
+        session = MagicMock()
+        session.commit = AsyncMock()
+
+        uow = _build_sql_uow(session)
+        uow.dispatch_events = AsyncMock(side_effect=RuntimeError("dispatch failed"))
+        uow.clear_tracked_entities = MagicMock()
+
+        raised = None
+        try:
+            await uow.commit()
+        except RuntimeError as exc:
+            raised = exc
+
+        assert raised is not None
+        session.commit.assert_awaited_once()
+        uow.dispatch_events.assert_awaited_once()
+        uow.clear_tracked_entities.assert_called_once()
+
+    asyncio.run(_run())