Skip to content

feat/tree compare grouped tests - #2119

Open
alanpeixinho wants to merge 1 commit into
kernelci:mainfrom
profusion:feat/tree-compare-grouped-tests
Open

alanpeixinho wants to merge 1 commit into
kernelci:mainfrom
profusion:feat/tree-compare-grouped-tests

Conversation

@alanpeixinho

Copy link
Copy Markdown
Contributor

What it is

Compare Tests tab: path grouping + Tree Details–style summaries (config, arch, platform, change). Config column fixes duplicate-looking rows (#2105). Reuses tree builders + adaptColumnsForUnifiedTable, not
full TestsTable. Group rows: Side A/B status circles. Boots: config + prefix path ellipsis.

Closes #2105, #2101.

How to test

  1. Open tree A/B compare with real or mocked compare API data (Tests tab needs rows with config_name, path, etc.).
  2. Tests: config column, expand/collapse groups, search + status filters.
  3. Boots: config + path ellipsis.
  4. Spot-check Tree Details Tests still works.

@felipebergamin felipebergamin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good one. But some configs are overflowing their columns.
even on my larger external monitor

Image

@alanpeixinho
alanpeixinho force-pushed the feat/tree-compare-grouped-tests branch from 2143d84 to ae72481 Compare September 23, 2026 12:55
* Add CompareTestsGroupedTable using tree builders and adaptColumnsForUnifiedTable
* Show config, architecture, and platform; group rows use Tree Details summaries
* Side A/B status circles on collapsed groups; change shows uniform badge or (N)
* Pass compare fields through TestHistory into UnifiedTestRow group summaries
* Trim CompareFailuresTables; share search and side cells via compareTableShared
* Add boots config column and PathWithPrefixEllipsis for long paths
* Extend tree-compare e2e for Tests tab Config column

Closes kernelci#2105
Closes kernelci#2101
@alanpeixinho
alanpeixinho force-pushed the feat/tree-compare-grouped-tests branch from ae72481 to 07a5f61 Compare September 23, 2026 12:58

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[AB Comparison] Show config name in tests table

2 participants