Repository navigation
GUI: transient tests, per-fixture results, passed panels that go - #6
Merged
Merged
Conversation
A test started with `htf.Test(..., transient=True)` (record metadata) only serves the station while it runs, such as one that asks which fixture to start next: its panel goes as soon as it completes. A new test now replaces only the completed tests of its own name, so with several fixtures on one station each keeps its result in view until it starts again. A station with one test name behaves as before. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Coverage Report for CI Build 38723482313Coverage increased (+1.3%) to 67.593%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsNo coverage regressions found. Coverage Stats
💛 - Coveralls |
A transient test tested nothing; its final record was listed in the history as an UNKNOWN_DUT pass each time it ended. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
`StationServer(..., hide_passed_after_s=120)` passes the delay to the frontend in its config. With several fixtures on one station, a passed test's panel then goes that long after it completes, while one that failed, errored or was aborted stays until a test of its name starts again: fixtures that run for hours mostly finish with nobody watching, and a failure must still be on screen when someone comes back. Unset, nothing changes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
For the battery station's new start flow (BluEye-Robotics/P2-functional-tests#244). Its first column is a test that asks which slot to start next. When all four slots are running, that column should go, so the station shows at most four columns.
Changes
Transient tests. A test started with
htf.Test(..., transient=True)(it ends up in the record's metadata) has its panel removed as soon as it completes.TestState.transientcomes fromtest_record.metadata.transient.Each fixture keeps its result. A new test now replaces only the completed tests of its own name. Before, any new test cleared every completed panel. The battery station restarts its first column whenever a slot frees up, and under the old rule a slot's PASS/FAIL would have vanished at that moment. Now a slot's result stays until that slot starts again. A station with a single test name (every other station) behaves exactly as before.
Passed panels can go after a delay.
StationServer(..., hide_passed_after_s=120)passes the delay to the GUI in its config (ConfigService.hidePassedAfterMs). With it set, a passed test's panel is removed that long after it completes. A test that failed, errored or was aborted keeps its panel until a test of its name starts again. When it isn't set, nothing changes.Transient tests stay out of History. A transient test's final record isn't added there; otherwise every one showed up as an
UNKNOWN_DUTpass.Testing
station.service.spec.ts:station.service.spec.ts:ng testin headless Chrome: 43 of 43 pass.test/output/servers/station_server_test.py: the server rendershide_passed_after_sinto its page, asnullwhen it's not set. 6 of 6 pass.dist/rebuilt with Node 24; only the main bundle changes.🤖 Generated with Claude Code