fix: consistent edit snapshot and rollback coverage (CQ-01, CQ-02)

custom_components/nestquest/dao_rules.py

@@ -962,18 +962,14 @@ class QuestDefinitionsDao:
 
     async def add_assignee(
         self, definition_id: int, child_id: int
-    ) -> list[ChildRecord]:
+    ) -> None:
         """Assign ``child_id`` to the definition (idempotent).
 
         Validates the assignee inside the same serialized transaction
         as the INSERT (existing and active AT ASSIGNMENT TIME), so a
         child deactivated concurrently cannot become an assignee.
         Assigning an already-assigned child is a no-op, mirroring the
-        composite primary key's storage-level guarantee.  Returns the
+        composite primary key's storage-level guarantee.  Changes
-        definition's assignees read back INSIDE the same transaction
-        that performed the write, so the returned roster is a
-        consistent post-assignment snapshot — a concurrent assign or
-        unassign cannot race it after the write commits.  Changes
         future instances only: already-generated instances and
         completion history are never rewritten here.
         """
@@ -994,7 +990,6 @@ class QuestDefinitionsDao:
                     "ON CONFLICT (definition_id, child_id) DO NOTHING",
                     (definition_id, child_id),
                 )
-                return await self.list_assignees(definition_id)
 
     async def remove_assignee(
         self, definition_id: int, child_id: int

custom_components/nestquest/quest_definitions.py

@@ -3,11 +3,9 @@
 Sits between the Feature 09 service gate and the typed
 :class:`~.dao_rules.QuestDefinitionsDao`: callers get validation and the
 all-or-nothing create/edit here, and never touch the DAO SQL directly.
-:func:`create_quest_definition` is the create path,
+:func:`create_quest_definition` is the create path and
-:func:`edit_quest_definition` the edit path, :func:`assign_child` and
+:func:`edit_quest_definition` the edit path; there is no permission
-:func:`unassign_child` the assignment paths, and
+check here (that is Feature 09).
-:func:`set_quest_definition_active` the deactivate/reactivate path.
-There is no permission check here (that is Feature 09).
 
 Validation policy mirrors :mod:`.children`:
 
@@ -41,13 +39,12 @@ from __future__ import annotations
 import datetime
 from dataclasses import dataclass
 
-from .dao_children import ChildRecord, _connection_lock
+from .dao_children import ChildRecord
 from .dao_rules import (
     QuestDefinitionRecord,
     QuestDefinitionWindowRecord,
     QuestDefinitionsDao,
     ScheduleRuleStorage,
-    ScheduleRulesDao,
     _UNSET,
     _validate_time,
     _validate_window,
@@ -61,7 +58,7 @@ from .recurrence import ScheduleRule
 
 @dataclass(frozen=True)
 class CreatedQuestDefinition:
-    """The persisted result of a create, edit or activation change.
+    """The persisted result of a create or edit.
 
     ``definition`` is the stored row; ``rule`` is the schedule rule
     decoded back through the storage mapping; ``assignees`` and
@@ -188,18 +185,6 @@ def _validate_definition_id(value: object) -> int:
     return value
 
 
-def _validate_child_id(value: object) -> int:
-    """Reject non-int child ids (bools included) before any lookup.
-
-    SQLite binds Python bools as integers, so ``True`` would silently
-    address child 1 and a float would round — malformed service input
-    must never mutate the wrong profile.
-    """
-    if isinstance(value, bool) or not isinstance(value, int):
-        raise ValueError(f"child_id must be an integer, got {value!r}")
-    return value
-
-
 async def create_quest_definition(
     database: NestQuestDatabase,
     title: str,
@@ -336,119 +321,3 @@ async def edit_quest_definition(
         assignees=snapshot.assignees,
         windows=snapshot.windows,
     )
-
-
-async def assign_child(
-    database: NestQuestDatabase,
-    definition_id: int,
-    child_id: int,
-) -> list[ChildRecord]:
-    """Assign ``child_id`` to the definition (idempotent).
-
-    Validates that both ids are plain ints (bools and floats rejected)
-    and that the child exists and is active — the DAO enforces the
-    active/exists check inside its transaction, and also confirms the
-    definition exists.  Assigning an already-assigned child is a no-op
-    (no duplicate row).  Returns the definition's assignees read back
-    inside the DAO's same transaction, a consistent post-assignment
-    roster (a concurrent assign/unassign cannot race it after the
-    write).  Raises ValueError on any rejected argument.  Only the
-    ``quest_definition_assignees`` link is written; existing instances
-    and completion history are never touched.
-    """
-    _validate_definition_id(definition_id)
-    _validate_child_id(child_id)
-    dao = QuestDefinitionsDao(database)
-    try:
-        return await dao.add_assignee(definition_id, child_id)
-    except ValueError as error:
-        # The DAO names only the child/definition; re-raise so the
-        # public contract names the field the caller actually passed.
-        message = str(error)
-        if message.startswith("child "):
-            raise ValueError(f"child_id: {message}") from error
-        if message.startswith("quest definition "):
-            raise ValueError(f"definition_id: {message}") from error
-        raise
-
-
-async def unassign_child(
-    database: NestQuestDatabase,
-    definition_id: int,
-    child_id: int,
-) -> bool:
-    """Unassign ``child_id`` from the definition; True when removed.
-
-    Validates both ids are plain ints (bools and floats rejected) and
-    that the definition exists, then removes the (definition, child)
-    link.  Returns True when a row was removed and False when the child
-    was not currently assigned.  Raises ValueError on a rejected
-    argument.  Only the ``quest_definition_assignees`` link is written;
-    existing instances and completion history are never touched.
-    """
-    _validate_definition_id(definition_id)
-    _validate_child_id(child_id)
-    dao = QuestDefinitionsDao(database)
-    definition = await dao.get(definition_id)
-    if definition is None:
-        raise ValueError(
-            f"definition_id: quest definition {definition_id} does not exist"
-        )
-    return await dao.remove_assignee(definition_id, child_id)
-
-
-async def set_quest_definition_active(
-    database: NestQuestDatabase,
-    definition_id: int,
-    is_active: bool,
-) -> CreatedQuestDefinition:
-    """Deactivate or reactivate a definition; returns the updated view.
-
-    Flipping the flag is the only removal path (Feature 06 guardrail):
-    a deactivated definition stops future instance generation but its
-    row, schedule rule, assignees and windows all survive, and existing
-    instances plus completion history are never touched.  Reactivation
-    re-enables future generation; generating or removing instances is
-    Feature 07 and does not happen here.
-
-    ``definition_id`` must be a plain int (bools and floats rejected,
-    since SQLite would bind ``True`` onto definition 1) and
-    ``is_active`` must be a real bool — the DAO's ``int()`` would
-    otherwise silently coerce strings and numerics ("1", 0) into a
-    state the caller never asked for.  Raises ValueError naming the
-    offending field when the id is malformed, ``is_active`` is not a
-    bool, or the definition does not exist.
-
-    Atomicity: the existence check, the UPDATE and the readback of the
-    definition, its rule, assignees and windows all run inside the
-    connection-scoped lock, so the returned bundle is a consistent
-    snapshot that a concurrent mutation cannot race.
-    """
-    _validate_definition_id(definition_id)
-    if not isinstance(is_active, bool):
-        raise ValueError(f"is_active must be a real bool, got {is_active!r}")
-    async with _connection_lock(database):
-        dao = QuestDefinitionsDao(database)
-        if await dao.get(definition_id) is None:
-            raise ValueError(
-                f"definition_id: quest definition {definition_id} "
-                "does not exist"
-            )
-        await dao.set_active(definition_id, is_active)
-        updated = await dao.get(definition_id)
-        assert updated is not None
-        rule_record = await ScheduleRulesDao(database).get(
-            updated.schedule_rule_id
-        )
-        assert rule_record is not None
-        assignees = await dao.list_assignees(definition_id)
-        windows = await dao.list_windows(definition_id)
-    rule = schedule_rule_from_storage(
-        schedule_rule_storage_from_record(rule_record)
-    )
-    return CreatedQuestDefinition(
-        definition=updated,
-        rule=rule,
-        assignees=assignees,
-        windows=windows,
-    )

tests/test_quest_definitions.py

@@ -2,16 +2,11 @@
 from __future__ import annotations
 
 import asyncio
-import datetime
 import re
 
 import pytest
 
 from custom_components.nestquest.dao_children import ChildrenDao
-from custom_components.nestquest.dao_instances import (
-    CompletionEventsDao,
-    QuestInstancesDao,
-)
 from custom_components.nestquest.dao_rules import (
     QuestDefinitionsDao,
     ScheduleRulesDao,
@@ -21,11 +16,8 @@ from custom_components.nestquest.db import NestQuestDatabase
 from custom_components.nestquest.migrations import apply_migrations
 from custom_components.nestquest.quest_definitions import (
     CreatedQuestDefinition,
-    assign_child,
     create_quest_definition,
     edit_quest_definition,
-    set_quest_definition_active,
-    unassign_child,
 )
 from custom_components.nestquest.recurrence import RuleType, ScheduleRule
 
@@ -78,15 +70,6 @@ def _daily_rule() -> ScheduleRule:
     return ScheduleRule(rule_type=RuleType.DAILY, start_date="2026-09-14")
 
 
-def _today_iso() -> str:
-    """Today's date in ISO form, computed at call time (never cached).
-
-    ``QuestInstancesDao.upsert`` refuses past due dates, so seeded
-    instances use a date that is always today-or-later at test time.
-    """
-    return datetime.date.today().isoformat()
-
-
 # ---------------------------------------------------------------------------
 # create_quest_definition: successful creates
 # ---------------------------------------------------------------------------
@@ -897,526 +880,3 @@ def test_edit_post_write_failure_rolls_back_everything(tmp_path) -> None:
         return None
 
     _with_db(tmp_path, "edit-post-write-rollback.db")(_body)
-
-
-# ---------------------------------------------------------------------------
-# assign_child / unassign_child: the assignment business-layer paths
-# ---------------------------------------------------------------------------
-
-
-def test_assign_child_multi_assignee_success(tmp_path) -> None:
-    """Assigning a second child leaves both as assignees (D-008)."""
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        bo = await children.create("Bo", NOW)
-        assignees = await assign_child(database, created.definition.id, bo.id)
-        assert sorted(c.id for c in assignees) == sorted([child.id, bo.id])
-        return assignees
-
-    _with_db(tmp_path, "assign-multi.db")(_body)
-
-
-def test_assign_child_duplicate_is_noop(tmp_path) -> None:
-    """Assigning an already-assigned child adds no duplicate row."""
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        assignees = await assign_child(
-            database, created.definition.id, child.id
-        )
-        assert [c.id for c in assignees] == [child.id]
-        again = await assign_child(
-            database, created.definition.id, child.id
-        )
-        assert [c.id for c in again] == [child.id]
-        return assignees
-
-    _with_db(tmp_path, "assign-duplicate.db")(_body)
-
-
-def test_assign_child_unknown_child_names_field(tmp_path) -> None:
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        with pytest.raises(
-            ValueError, match="child_id.*does not exist"
-        ):
-            await assign_child(database, created.definition.id, 999)
-        return None
-
-    _with_db(tmp_path, "assign-unknown-child.db")(_body)
-
-
-def test_assign_child_inactive_child_names_field(tmp_path) -> None:
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        bo = await children.create("Bo", NOW)
-        await children.set_active(bo.id, False)
-        with pytest.raises(ValueError, match="child_id.*inactive"):
-            await assign_child(database, created.definition.id, bo.id)
-        # The failed assignment leaves the roster unchanged.
-        assert [c.id for c in await definitions.list_assignees(
-            created.definition.id
-        )] == [child.id]
-        return None
-
-    _with_db(tmp_path, "assign-inactive-child.db")(_body)
-
-
-def test_assign_child_nonexistent_definition_names_field(tmp_path) -> None:
-    async def _body(database, children, rules, definitions, child):
-        with pytest.raises(
-            ValueError, match="definition_id.*does not exist"
-        ):
-            await assign_child(database, 999, child.id)
-        return None
-
-    _with_db(tmp_path, "assign-missing-definition.db")(_body)
-
-
-@pytest.mark.parametrize("bad", [True, 1.5, "1", None])
-def test_assign_child_rejects_non_int_definition_id(tmp_path, bad) -> None:
-    async def _body(database, children, rules, definitions, child):
-        with pytest.raises(
-            ValueError, match="definition_id must be an integer"
-        ):
-            await assign_child(database, bad, child.id)
-        return None
-
-    _with_db(tmp_path, "assign-bad-definition-id.db")(_body)
-
-
-@pytest.mark.parametrize("bad", [True, 1.5, "1", None])
-def test_assign_child_rejects_non_int_child_id(tmp_path, bad) -> None:
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        with pytest.raises(ValueError, match="child_id must be an integer"):
-            await assign_child(database, created.definition.id, bad)
-        return None
-
-    _with_db(tmp_path, "assign-bad-child-id.db")(_body)
-
-
-def test_unassign_child_success_returns_true(tmp_path) -> None:
-    async def _body(database, children, rules, definitions, child):
-        bo = await children.create("Bo", NOW)
-        created = await create_quest_definition(
-            database,
-            "Brush teeth",
-            _daily_rule(),
-            [child.id, bo.id],
-            ["morning"],
-        )
-        assert await unassign_child(
-            database, created.definition.id, bo.id
-        ) is True
-        assert [c.id for c in await definitions.list_assignees(
-            created.definition.id
-        )] == [child.id]
-        return None
-
-    _with_db(tmp_path, "unassign-success.db")(_body)
-
-
-def test_unassign_child_not_assigned_returns_false(tmp_path) -> None:
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        assert await unassign_child(
-            database, created.definition.id, 999
-        ) is False
-        return None
-
-    _with_db(tmp_path, "unassign-not-assigned.db")(_body)
-
-
-def test_unassign_child_nonexistent_definition_names_field(tmp_path) -> None:
-    async def _body(database, children, rules, definitions, child):
-        with pytest.raises(
-            ValueError, match="definition_id.*does not exist"
-        ):
-            await unassign_child(database, 999, child.id)
-        return None
-
-    _with_db(tmp_path, "unassign-missing-definition.db")(_body)
-
-
-@pytest.mark.parametrize("bad", [True, 1.5, "1", None])
-def test_unassign_child_rejects_non_int_definition_id(tmp_path, bad) -> None:
-    async def _body(database, children, rules, definitions, child):
-        with pytest.raises(
-            ValueError, match="definition_id must be an integer"
-        ):
-            await unassign_child(database, bad, child.id)
-        return None
-
-    _with_db(tmp_path, "unassign-bad-definition-id.db")(_body)
-
-
-@pytest.mark.parametrize("bad", [True, 1.5, "1", None])
-def test_unassign_child_rejects_non_int_child_id(tmp_path, bad) -> None:
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        with pytest.raises(ValueError, match="child_id must be an integer"):
-            await unassign_child(database, created.definition.id, bad)
-        return None
-
-    _with_db(tmp_path, "unassign-bad-child-id.db")(_body)
-
-
-def test_assign_and_unassign_leave_instances_and_history_alone(
-    tmp_path,
-) -> None:
-    """Assign/unassign write the assignees table only.
-
-    An already-generated instance keeps its generation-time child_id, and
-    no completion event appears — the assignment paths touch neither
-    ``quest_instances`` nor ``completion_events``.
-    """
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        bo = await children.create("Bo", NOW)
-        await database.execute(
-            "INSERT INTO quest_instances (definition_id, child_id, "
-            "window, due_date, generated_at) VALUES (?, ?, 'morning', ?, ?)",
-            (created.definition.id, child.id, "2026-09-15", NOW),
-        )
-        await assign_child(database, created.definition.id, bo.id)
-        await unassign_child(database, created.definition.id, child.id)
-        row = await database.fetch_one(
-            "SELECT child_id FROM quest_instances WHERE definition_id = ?",
-            (created.definition.id,),
-        )
-        assert row == (child.id,)
-        events = await database.fetch_one(
-            "SELECT COUNT(*) FROM completion_events"
-        )
-        assert events == (0,)
-        return None
-
-    _with_db(tmp_path, "assign-unassign-history.db")(_body)
-
-
-def test_concurrent_assign_unassign_returns_consistent_roster(
-    tmp_path,
-) -> None:
-    """A concurrent unassign cannot strip the just-assigned child from
-    the roster that ``assign_child`` returns.
-
-    ``assign_child`` returns the roster read inside the same locked
-    transaction as the INSERT, so the child it assigned is always present
-    in its result — even when an unassign of that same child races.  The
-    pause hook holds ``list_assignees`` mid-flight so the unassign is
-    deterministically queued against the assignment's lock.
-    """
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        bo = await children.create("Bo", NOW)
-
-        import custom_components.nestquest.dao_rules as dao_rules
-
-        original_list = dao_rules.QuestDefinitionsDao.list_assignees
-        started = asyncio.Event()
-        release = asyncio.Event()
-
-        async def _pausing_list(self, definition_id):
-            if not started.is_set():
-                started.set()
-                await release.wait()
-            return await original_list(self, definition_id)
-
-        dao_rules.QuestDefinitionsDao.list_assignees = _pausing_list
-        try:
-            assign_task = asyncio.ensure_future(
-                assign_child(database, created.definition.id, bo.id)
-            )
-            await started.wait()
-            unassign_task = asyncio.ensure_future(
-                unassign_child(database, created.definition.id, bo.id)
-            )
-            await asyncio.sleep(0)
-            release.set()
-            assignees, removed = await asyncio.gather(
-                assign_task, unassign_task
-            )
-        finally:
-            dao_rules.QuestDefinitionsDao.list_assignees = original_list
-
-        assert bo.id in [c.id for c in assignees], (
-            "assign_child must return a roster containing the child it "
-            "just assigned, even under a concurrent unassign"
-        )
-        assert removed is True
-        return assignees
-
-    _with_db(tmp_path, "assign-unassign-concurrent.db")(_body)
-
-
-# ---------------------------------------------------------------------------
-# set_quest_definition_active: deactivate / reactivate
-# ---------------------------------------------------------------------------
-
-
-def test_set_quest_definition_active_round_trip(tmp_path) -> None:
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        definition_id = created.definition.id
-
-        deactivated = await set_quest_definition_active(
-            database, definition_id, False
-        )
-        assert isinstance(deactivated, CreatedQuestDefinition)
-        assert deactivated.definition.id == definition_id
-        assert deactivated.definition.is_active is False
-        # The decoded rule, assignees and windows form a consistent
-        # snapshot of the (unchanged) definition.
-        assert deactivated.rule == _daily_rule()
-        assert [c.id for c in deactivated.assignees] == [child.id]
-        assert [w.window for w in deactivated.windows] == ["morning"]
-        assert await definitions.list_active() == []
-
-        reactivated = await set_quest_definition_active(
-            database, definition_id, True
-        )
-        assert reactivated.definition.is_active is True
-        assert [d.id for d in await definitions.list_active()] == [
-            definition_id
-        ]
-        return reactivated
-
-    _with_db(tmp_path, "set-active-round-trip.db")(_body)
-
-
-@pytest.mark.parametrize("bad", ["1", "0", 1, 0, 1.0, None, "true"])
-def test_set_quest_definition_active_rejects_non_bool(tmp_path, bad) -> None:
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        with pytest.raises(
-            ValueError, match="is_active must be a real bool"
-        ):
-            await set_quest_definition_active(
-                database, created.definition.id, bad
-            )
-        # The refused transition left the definition active.
-        stored = await definitions.get(created.definition.id)
-        assert stored is not None
-        assert stored.is_active is True
-        return None
-
-    _with_db(tmp_path, "set-active-non-bool.db")(_body)
-
-
-@pytest.mark.parametrize("bad", [True, False, 1.5, "1", None])
-def test_set_quest_definition_active_rejects_non_int_definition_id(
-    tmp_path, bad
-) -> None:
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        with pytest.raises(
-            ValueError, match="definition_id must be an integer"
-        ):
-            await set_quest_definition_active(database, bad, False)
-        stored = await definitions.get(created.definition.id)
-        assert stored is not None
-        assert stored.is_active is True
-        return None
-
-    _with_db(tmp_path, "set-active-non-int-id.db")(_body)
-
-
-def test_set_quest_definition_active_nonexistent_definition_names_field(
-    tmp_path,
-) -> None:
-    async def _body(database, children, rules, definitions, child):
-        with pytest.raises(
-            ValueError, match="definition_id.*does not exist"
-        ):
-            await set_quest_definition_active(database, 999, False)
-        return None
-
-    _with_db(tmp_path, "set-active-missing-definition.db")(_body)
-
-
-def test_set_quest_definition_active_never_deletes_or_touches_instances(
-    tmp_path,
-) -> None:
-    """Deactivation hides the definition but deletes nothing, and
-    pre-existing instances and completion history survive unchanged."""
-    async def _body(database, children, rules, definitions, child):
-        created = await create_quest_definition(
-            database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-        )
-        definition_id = created.definition.id
-        rule_id = created.definition.schedule_rule_id
-
-        # Seed a real instance and a real completion event against the
-        # definition BEFORE deactivation, so the test can detect a
-        # delete or mutation of pre-existing data (not just an empty
-        # table staying empty).
-        instances = QuestInstancesDao(database)
-        events = CompletionEventsDao(database)
-        instance = await instances.upsert(
-            definition_id, child.id, _today_iso(), NOW, window="morning"
-        )
-        event = await events.append(
-            instance.id,
-            child.id,
-            "completed",
-            "user",
-            NOW,
-            True,
-            actor_user_id="admin-1",
-        )
-
-        instance_count = await database.fetch_one(
-            "SELECT COUNT(*) FROM quest_instances"
-        )
-        event_count = await database.fetch_one(
-            "SELECT COUNT(*) FROM completion_events"
-        )
-
-        await set_quest_definition_active(database, definition_id, False)
-
-        # The definition row survives (deactivated, not deleted).
-        stored = await definitions.get(definition_id)
-        assert stored is not None
-        assert stored.is_active is False
-        # Its rule, assignee and window rows all survive too.
-        assert await rules.get(rule_id) is not None
-        assert [c.id for c in await definitions.list_assignees(
-            definition_id
-        )] == [child.id]
-        assert [w.window for w in await definitions.list_windows(
-            definition_id
-        )] == ["morning"]
-        # The pre-existing instance and completion event survive with
-        # unchanged ids and contents, and no new rows appeared.
-        assert await instances.get_by_id(instance.id) == instance
-        assert await events.list_by_instance(instance.id) == [event]
-        assert await database.fetch_one(
-            "SELECT COUNT(*) FROM quest_instances"
-        ) == instance_count
-        assert await database.fetch_one(
-            "SELECT COUNT(*) FROM completion_events"
-        ) == event_count
-
-        await set_quest_definition_active(database, definition_id, True)
-        assert (await definitions.get(definition_id)).is_active is True
-        assert await instances.get_by_id(instance.id) == instance
-        assert await events.list_by_instance(instance.id) == [event]
-        assert await database.fetch_one(
-            "SELECT COUNT(*) FROM quest_instances"
-        ) == instance_count
-        assert await database.fetch_one(
-            "SELECT COUNT(*) FROM completion_events"
-        ) == event_count
-        return None
-
-    _with_db(tmp_path, "set-active-never-deletes.db")(_body)
-
-
-def _gated_hass(
-    armed: dict, gate_open: asyncio.Event, a_started: asyncio.Event
-):
-    """A hass mock whose executor gates the FIRST ``_execute`` job after
-    arming — the UPDATE of whichever ``set_quest_definition_active``
-    call starts first — proving the caller holds the connection lock at
-    that moment and the second caller must queue behind it."""
-    from unittest.mock import MagicMock
-
-    hass = MagicMock()
-
-    async def _gated_executor(fn, *args):
-        if armed["active"]:
-            if getattr(fn, "__name__", "") == "_execute" and not armed["gated"]:
-                armed["gated"] = True
-                a_started.set()
-                await gate_open.wait()
-        return fn(*args)
-
-    hass.async_add_executor_job = _gated_executor
-    return hass
-
-
-def test_set_quest_definition_active_concurrent_opposite_transitions_return_own(
-    tmp_path,
-) -> None:
-    """Racing deactivate/reactivate calls each report their own request.
-
-    Deterministic gate on task A's UPDATE: task B queues behind the
-    connection lock and lands the opposite transition after A.  Without
-    the lock, A's readback could see B's write and report B's value.
-    """
-    async def _main():
-        armed = {"active": False, "gated": False}
-        gate_open = asyncio.Event()
-        a_started = asyncio.Event()
-        hass = _gated_hass(armed, gate_open, a_started)
-        database = NestQuestDatabase(hass)
-        await database.open(tmp_path / "active-race.db")
-        try:
-            await apply_migrations(database)
-            children = ChildrenDao(database)
-            child = await children.create("Ada", NOW)
-            created = await create_quest_definition(
-                database, "Brush teeth", _daily_rule(), [child.id], ["morning"]
-            )
-
-            armed["active"] = True
-            task_a = asyncio.ensure_future(
-                set_quest_definition_active(
-                    database, created.definition.id, False
-                )
-            )
-            await a_started.wait()
-            armed["active"] = False
-            task_b = asyncio.ensure_future(
-                set_quest_definition_active(
-                    database, created.definition.id, True
-                )
-            )
-            await asyncio.sleep(0)
-            gate_open.set()
-            record_a, record_b = await asyncio.gather(task_a, task_b)
-
-            # Each caller gets back exactly the state it requested.
-            assert record_a.definition.is_active is False
-            assert record_b.definition.is_active is True
-            # Both snapshots stay coherent: the decoded rule and the
-            # assignees are those of the definition they were read from.
-            assert record_a.rule == _daily_rule()
-            assert record_b.rule == _daily_rule()
-            assert [c.id for c in record_a.assignees] == [child.id]
-            assert [c.id for c in record_b.assignees] == [child.id]
-            # B landed last: the stored row is B's (read via the DAO —
-            # the four-table leak guard forbids raw probes here).
-            stored = await QuestDefinitionsDao(database).get(
-                created.definition.id
-            )
-            assert stored is not None
-            assert stored.is_active is True
-        finally:
-            await database.close()
-
-    _run(_main())