Skip to content

Redaction tests: replace the two wall-time mutant tests with work-counter assertions #311

Open

Description

@ViperJuice

Follow-up from the review of #310, the implementation for #234.

test_bearer_value_lookahead_mutant and test_list_backtracking_mutant in tests/test_redaction_additive.py assert wall-clock durations. Their margins are generous (about 30x and 6x on dev0), and one is a lower bound (> 0.3 s), so a much faster runner could fail it. Everything else in that file measures linearity with the deterministic work counter and tracemalloc. Moving these two onto the same footing would make them independent of the hardware.

🤖 Generated with Claude Code

Activity

  1. ViperJuice commented on Oct 17, 2026

    @ViperJuice
    CollaboratorAuthor

    Reconciled after #399 merged: both named mutant tests still use perf_counter wall-clock assertions on main e222606, so this issue remains open. #400 overlaps the bearer mutant and tracks additional xdist-sensitive deadlines. They can be addressed together, preserving mutation sensitivity and deterministic work-based assertions; the xdist merge did not complete this work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions