Pin async commit/rollback "no transaction" swallow behaviour

AsyncConnection.commit / rollback silently absorb the server's
"cannot commit / rollback - no transaction is active" reply to match
the stdlib sqlite3 contract (and the sync Connection path). Only the
sync side had explicit tests; add the async counterparts so a
regression that widened or narrowed _is_no_transaction_error surfaces
immediately, and cover the re-raise path for unrelated
OperationalErrors.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: antoineleclair

tests/test_aio_commit_rollback_no_tx.py

@@ -0,0 +1,70 @@
+"""Async commit/rollback "no transaction is active" swallow behaviour.
+
+Mirrors the sync-side tests for ``_is_no_transaction_error``. The
+async path has the same silent-no-op contract (matches stdlib
+sqlite3); without dedicated tests, a regression that widened or
+narrowed the match would surface only via integration.
+"""
+
+from unittest.mock import AsyncMock, MagicMock
+
+import pytest
+
+import dqliteclient.exceptions as _client_exc
+from dqlitedbapi.aio.connection import AsyncConnection
+
+
+def _prime(address: str = "localhost:19001") -> AsyncConnection:
+    """Build an AsyncConnection with a mocked inner client connection."""
+    conn = AsyncConnection(address, database="x")
+    inner = MagicMock()
+    inner.close = AsyncMock()
+    inner.execute = AsyncMock()
+    conn._async_conn = inner
+    return conn
+
+
+class TestAsyncCommitNoTxSwallow:
+    async def test_commit_swallows_no_transaction_error(self) -> None:
+        conn = _prime()
+        assert conn._async_conn is not None
+        conn._async_conn.execute.side_effect = _client_exc.OperationalError(
+            1, "cannot commit - no transaction is active"
+        )
+        await conn.commit()  # silent no-op
+
+    async def test_rollback_swallows_no_transaction_error(self) -> None:
+        conn = _prime()
+        assert conn._async_conn is not None
+        conn._async_conn.execute.side_effect = _client_exc.OperationalError(
+            1, "cannot rollback - no transaction is active"
+        )
+        await conn.rollback()  # silent no-op
+
+    async def test_commit_re_raises_other_operational_errors(self) -> None:
+        conn = _prime()
+        assert conn._async_conn is not None
+        conn._async_conn.execute.side_effect = _client_exc.OperationalError(
+            10, "some unrelated error"
+        )
+        with pytest.raises(_client_exc.OperationalError, match="some unrelated error"):
+            await conn.commit()
+
+    async def test_rollback_re_raises_other_operational_errors(self) -> None:
+        conn = _prime()
+        assert conn._async_conn is not None
+        conn._async_conn.execute.side_effect = _client_exc.OperationalError(
+            10, "some unrelated error"
+        )
+        with pytest.raises(_client_exc.OperationalError, match="some unrelated error"):
+            await conn.rollback()
+
+    def test_sync_no_tx_helper_matches(self) -> None:
+        """Sanity check for sync/async parity on the helper that drives
+        both the dbapi sync Connection.commit/rollback and the
+        AsyncConnection.commit/rollback swallow paths.
+        """
+        from dqlitedbapi.connection import _is_no_transaction_error
+
+        exc = _client_exc.OperationalError(1, "cannot commit - no transaction is active")
+        assert _is_no_transaction_error(exc) is True