Skip to content

feat(html-reporter): add step filter in test steps section#39595

Merged
dgozman merged 1 commit intomicrosoft:mainfrom
EmanueleMinotto:html-reporter-steps-filter
Mar 25, 2026
Merged

feat(html-reporter): add step filter in test steps section#39595
dgozman merged 1 commit intomicrosoft:mainfrom
EmanueleMinotto:html-reporter-steps-filter

Conversation

@EmanueleMinotto
Copy link
Copy Markdown
Contributor

Split from #39334 with a more clean code and the html reporter only.

Without filter (default state)

without-filter

With filter

with-filter

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@pavelfeldman
Copy link
Copy Markdown
Member

it does not reveal

image

@EmanueleMinotto
Copy link
Copy Markdown
Contributor Author

EmanueleMinotto commented Mar 11, 2026

@pavelfeldman in your screenshot "Close context" is part of a subtree of "After Hooks", so it has to be open but the open/closed status will remain when the filter is removed.

In this moment it should work exactly like the steps filter in the trace viewer, do you prefer a different behavior for the html report?

Edit: my mistake, I didn't have the branch updated, I'll update it in few hours.

Adds a search input to the Test Steps in the HTML reporter,
allowing users to filter the steps tree by title text. Steps that
don't match (and have no matching descendants) are hidden.
@EmanueleMinotto EmanueleMinotto force-pushed the html-reporter-steps-filter branch from d7a966c to f8d776b Compare March 11, 2026 19:36
@EmanueleMinotto
Copy link
Copy Markdown
Contributor Author

@pavelfeldman ready again for a review

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "tests 1"

1 failed
❌ [playwright-test] › ui-mode-test-network-tab.spec.ts:397 › should not preserve selection across test runs @windows-latest-node20

2 flaky ⚠️ [chromium-library] › library/popup.spec.ts:260 › should not throw when click closes popup `@ubuntu-22.04-chromium-tip-of-tree`
⚠️ [webkit-page] › page/page-set-input-files.spec.ts:38 › should upload a folder `@webkit-ubuntu-22.04-node20`

38714 passed, 848 skipped


Merge workflow run.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "MCP"

5244 passed, 164 skipped


Merge workflow run.

@dgozman dgozman merged commit 91e5e4a into microsoft:main Mar 25, 2026
38 of 39 checks passed
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.

3 participants