Repository navigation
feat: list the tests behind a message in the test messages graph (#384) - #386
Open
timdegroot1996 wants to merge 1 commit into
Open
timdegroot1996 wants to merge 1 commit into
timdegroot1996 wants to merge 1 commit into
Conversation
Clicking a message in the Test Messages graph (a bar or its label in the bar view, the message label in the timeline view) opens a dialog with the full message and one row per test that produced it, with the number of runs it got the message in. The dialog stays open for selecting text, Copy All copies every test name, and with --uselogs a test name opens the log of the latest run with that message at that test. Messages grouped by a message config list the tests of the whole group. Chart.js only calls onClick inside the chart area, so the axis label clicks go through a small chart plugin that reads chart.data.messageTests, which update_chart refreshes on every rebuild. Escape in a fullscreen graph no longer also closes the fullscreen view when it closes a modal opened from that graph. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch has not been deployed
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.
Closes #384
Problem
The Test Messages graph shows the most common failure messages, but not which tests produced them.
Change
--uselogs, clicking a test name opens thelog.htmlof the latest run in which that test got the message, at that test.Chart.js only calls
onClickinside the chart area, so axis label clicks are handled by a small chart plugin readingchart.data.messageTests(refreshed on everyupdate_chart).Tests
get_message_tests(counts, latest run, same name in different suites), the message config merge,open_test_log. 549 passed.Clicking A Message Lists The Tests That Produced Itin03_dashboard.robot(timeline label + bar), fails without the click handling. Full suite in Docker: 146 tests, 146 passed, 0 failed.Docs
New section in
docs/graphs-tables.md.🤖 Generated with Claude Code