Skip to content

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

Open
EmanueleMinotto wants to merge 1 commit intomicrosoft:mainfrom
EmanueleMinotto:html-reporter-steps-filter
Open

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

Conversation

@EmanueleMinotto
Copy link
Contributor

@EmanueleMinotto EmanueleMinotto commented Mar 10, 2026

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
Copy link
Contributor

github-actions bot commented Mar 11, 2026

Test results for "MCP"

4 failed
❌ [chrome] › mcp/http.spec.ts:207 › http transport browser lifecycle (persistent) @mcp-macos-latest
❌ [chrome] › mcp/sse.spec.ts:158 › sse transport browser lifecycle (persistent) @mcp-macos-latest
❌ [chromium] › mcp/http.spec.ts:207 › http transport browser lifecycle (persistent) @mcp-macos-latest
❌ [chromium] › mcp/sse.spec.ts:158 › sse transport browser lifecycle (persistent) @mcp-macos-latest

5149 passed, 164 skipped

Merge workflow run.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 11, 2026

Test results for "tests 1"

1 failed
❌ [webkit-library] › library/proxy.spec.ts:127 › should allow bypassing [::1] requests @webkit-ubuntu-22.04-node20

8 flaky ⚠️ [chromium-library] › library/popup.spec.ts:260 › should not throw when click closes popup `@ubuntu-22.04-chromium-tip-of-tree`
⚠️ [chromium-page] › page/page-request-continue.spec.ts:754 › propagate headers cross origin redirect after interception `@ubuntu-22.04-chromium-tip-of-tree`
⚠️ [chromium-library] › library/trace-viewer.spec.ts:1230 › should display language-specific locators `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/popup.spec.ts:260 › should not throw when click closes popup `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/inspector/recorder-api.spec.ts:120 › should type `@chromium-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/beforeunload.spec.ts:188 › does not get stalled by beforeUnload `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/browsertype-connect.spec.ts:758 › run-server › should upload a folder `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-library] › library/inspector/cli-codegen-1.spec.ts:1080 › cli codegen › should not throw csp directive violation errors `@firefox-ubuntu-22.04-node20`

38935 passed, 842 skipped

Merge workflow run.

@pavelfeldman
Copy link
Member

pavelfeldman commented Mar 11, 2026

it does not reveal

image

@EmanueleMinotto
Copy link
Contributor Author

EmanueleMinotto commented Mar 11, 2026 •
edited
Loading

@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.

@EmanueleMinotto
feat(html-reporter): add step filter in test steps section
f8d776b 
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
Contributor Author

EmanueleMinotto commented Mar 11, 2026

@pavelfeldman ready again for a review

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

Reviewers

@pavelfeldman pavelfeldman pavelfeldman approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@EmanueleMinotto @pavelfeldman