fix(tests): remove dead step_assert_no_deadlocks and fix ruff formatting
CI / lint (pull_request) Successful in 44s
CI / typecheck (pull_request) Successful in 1m16s
CI / security (pull_request) Successful in 1m21s
CI / benchmark-publish (pull_request) Has been skipped
CI / helm (pull_request) Successful in 26s
CI / quality (pull_request) Successful in 1m6s
CI / build (pull_request) Successful in 51s
CI / benchmark-regression (pull_request) Failing after 37s
CI / push-validation (pull_request) Successful in 20s
CI / e2e_tests (pull_request) Failing after 3m34s
CI / integration_tests (pull_request) Successful in 6m44s
CI / unit_tests (pull_request) Successful in 8m34s
CI / docker (pull_request) Successful in 1m28s
CI / coverage (pull_request) Successful in 13m26s
CI / status-check (pull_request) Failing after 3s

Removed the unreachable step_assert_no_deadlocks function from
invariant_service_thread_safety_mixed_steps.py. The step was never
referenced by any Gherkin scenario and contained a broken reference
to context.elapsed_time which is never set, causing an AttributeError
if the step were ever invoked. Applied ruff auto-formatting to fix
trailing blank lines left after the removal.

ISSUES CLOSED: #7524
This commit is contained in:
2026-05-05 08:01:17 +00:00
parent 2e1d230a2c
commit 6eb3e9022e
@@ -219,12 +219,3 @@ def step_assert_results_consistent(context: Any) -> None:
assert len(context.errors) == 0, (
f"Expected no errors, got {len(context.errors)}: {context.errors}"
)
@then("no deadlocks should occur")
def step_assert_no_deadlocks(context: Any) -> None:
"""Assert that no deadlocks occurred by verifying the test completed in time."""
assert context.elapsed_time < 30, (
f"Test took {context.elapsed_time:.2f}s — possible deadlock or starvation"
)