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
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:
@@ -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"
|
||||
)
|
||||
|
||||
|
||||
Reference in New Issue
Block a user