Skip to content

Critical state tests are skipped instead of made deterministic #47

@nficano

Description

@nficano

Three state tests are currently skipped in areas that carry important correctness guarantees. tests/state/test_resume.py:48 skips full resume replay because it is not implemented end-to-end, tests/state/test_subscriber_no_cancel.py:9 skips the subscriber-cannot-cancel authorization path because the dual-session interleaving occasionally hangs, and tests/state/test_subscribe_fanout_seq.py:9 skips subscriber-scoped sequence counter validation because subscribe-during-running races on the memory transport. The pytest run reports these as skipped, not flaky failures, so regressions in resume, cross-principal cancellation, and subscriber fan-out can ship without automated signal.

Fix prompt: Replace the timing-sensitive memory-transport interleavings with deterministic synchronization using explicit events, barriers, or a controllable fake transport, then re-enable the skipped tests. For resume, implement or narrow the expected behavior and assert replay ordering, duplicate suppression, and live continuation. For subscription fan-out and cancel authorization, assert both success and denial paths without relying on sleeps or manual coverage, and keep the existing timeout plugin as a guard rather than a reason to skip.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions