test(live): retain load context for timing failures (iter 264) - #259
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The three timing tests now report measured wall/internal timing and host load after the timed operation, so a failure retains useful saturation context. Their existing 2500 ms, 2000 ms and ±750 ms bounds remain unchanged; load never waives a failure.
Iteration 264 records ten isolated and ten controlled CPU-load samples per test (60 actual measurements), per-test dispositions, and a failing dishonest-elapsed mutation followed by byte-for-byte product restoration. Controlled CPU load does not reproduce every aspect of the historical concurrent-browser workload. All three original acceptance criteria are fulfilled.
Validation
FF_RDP_LIVE_TESTS=1 FF_RDP_LIVE_NETWORK_TESTS=1 cargo run -p xtask -- live-sweep: 346 unique named tests passed across all six tiers (335+1+3+3+2+2).All owned load workers and the raw sweep browser/profile were cleaned up; desktop Firefox was preserved. Evidence:
.git/ralph-loop/20260919-queue/iter264/validation-record.md, raw per-command logs/metadata,supervisor-accounting.json, anditer264-review/phase-result.json. Actual token usage is unavailable.Carry-over
None produced by this iteration. Historical timing observations remain in the plan; parked iteration 203 and unrelated daemon failures are not claimed fixed.