Skip to content

fix: use testName/testSubType filters instead of hidden testCategory column - #2307

Open
sravankumarkunadi wants to merge 1 commit into
elementary-data:masterfrom
sravankumarkunadi:fix/dashboard-test-category-filter
Open

fix: use testName/testSubType filters instead of hidden testCategory column#2307
sravankumarkunadi wants to merge 1 commit into
elementary-data:masterfrom
sravankumarkunadi:fix/dashboard-test-category-filter

Conversation

@sravankumarkunadi

@sravankumarkunadi sravankumarkunadi commented Jul 23, 2026

Copy link
Copy Markdown

Description

Fixes #2294

Clicking a test category tile on the Dashboard (e.g. "Anomalies", "Volume") navigates to the Test Results tab with a testCategory filter applied. However, testCategory is a hidden column that is never populated from row data, so the filter matches no rows and the tab appears empty.

This removes the testCategory shortcut from Y0n and falls back to the existing testName/testSubType filter logic, which correctly matches the data in the Test Results table.

Note on the changed file

The only changed file is elementary/monitor/data_monitoring/report/index.html, which is the built report bundle, so the diff is a one-token change in minified code. That means the next UI build will regenerate it and drop this fix unless the same change also lands in the UI source. Happy to open that PR too if you point me at the right repo — otherwise treat this as the stopgap for the shipped report.

Summary by CodeRabbit

  • Bug Fixes
    • Updated test filtering to rely on test name and optional subtype criteria.
    • Removed automatic filtering by test category to improve filter accuracy.

@github-actions

Copy link
Copy Markdown
Contributor

👋 @sravankumarkunadi
Thank you for raising your pull request.
Please make sure to add tests and document all user-facing changes.
You can do this by editing the docs files in this pull request.

@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The dashboard filter-construction helper now omits testCategory filters and uses testName with an optional testSubType constraint.

Changes

Dashboard filter correction

Layer / File(s) Summary
Test result filter mapping
elementary/monitor/data_monitoring/report/index.html
Y0n no longer emits testCategory equality filters and instead constructs filters from testName and optional testSubType.

Estimated code review effort: 2 (Simple) | ~5 minutes

Suggested reviewers: elazarlachkar

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The change addresses #2294 by removing the empty testCategory filter path and relying on testName/testSubType filters.
Out of Scope Changes check ✅ Passed No unrelated code changes are evident from the provided summary.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: switching filtering from testCategory to testName/testSubType.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sravankumarkunadi

Copy link
Copy Markdown
Author

This is a patch to the bundled index.html no source-level tests or docs apply here since the UI is built separately. Happy to add anything if needed though!

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.

Dashboard click-through filter uses hidden "testCategory" column instead of "testSubType", returning empty Test Results

1 participant