logview_new: adds tests overview page #2219
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
There was a problem hiding this comment.
Pull Request Overview
This PR introduces a new tests overview page that aggregates test results across multiple runs. The page shows test statistics including failure counts and allows filtering by branch and search terms. When viewing the page, the system fetches run details with a concurrency of 15, which drops to 5 when navigating away to maintain client performance.
Key Changes:
- Adds a new
/testsroute displaying aggregated test analysis data - Implements dynamic concurrency control (15 concurrent requests while viewing, 5 when navigating away)
- Creates test data aggregation logic that maps test names to run information
Reviewed Changes
Copilot reviewed 7 out of 11 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| petri/logview_new/src/tests.tsx | Main tests page component with filtering, search, and dynamic concurrency control |
| petri/logview_new/src/table_defs/tests.tsx | Table column definitions for displaying test data with status, counts, and links |
| petri/logview_new/src/styles/tests.css | CSS styling for test and architecture name display |
| petri/logview_new/src/menu.tsx | Enables navigation to the tests page by uncommenting the onClick handler |
| petri/logview_new/src/main.tsx | Adds the /tests route to the application routing |
| petri/logview_new/src/fetch/fetch_runs_data.tsx | Implements fetchTestAnalysis and convertToTestData functions for aggregating test results |
| petri/logview_new/src/data_defs.tsx | Adds TestRunInfo and TestData type definitions |
axelandrejs
previously approved these changes
Oct 21, 2025
Co-authored-by: Copilot <[email protected]>
…S/openvmm into logview-new-test-data
mattkur
approved these changes
Oct 21, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Adds and links the tests overview page. Concurrency when viewing the page is 15, if the user navigates away, concurrency drops to 5 to keep the client performant.
Sample Site:
https://gurasinghms.github.io/openvmm/petri/logview_new/dist/#/runs