Skip to content

feat(test): support per-project webServer configuration#40869

Open
yury-s wants to merge 3 commits into
microsoft:mainfrom
yury-s:feat-22496
Open

feat(test): support per-project webServer configuration#40869
yury-s wants to merge 3 commits into
microsoft:mainfrom
yury-s:feat-22496

Conversation

@yury-s
Copy link
Copy Markdown
Member

@yury-s yury-s commented May 15, 2026

Summary

  • Add webServer to TestProject so each project can declare its own dev server(s); a per-project server is launched only when the project is selected (directly via --project or through the dependency closure), in addition to any top-level webServer.

Fixes #22496

Add `webServer` to TestProject so each project can declare its own
dev server(s). A per-project server is launched only when the project
is selected (directly via --project or through the dependency closure),
in addition to any top-level `webServer`.

Fixes microsoft#22496
Comment thread packages/playwright/src/plugins/index.ts Outdated
Comment thread packages/playwright/src/runner/tasks.ts Outdated
Comment thread packages/playwright/src/runner/testRunner.ts Outdated
Plugin registration carries a single optional projectId. The task
builders no longer take a TestRun and no longer compute an active-plugin
list up front; each plugin task's setup gates itself with
\`!plugin.projectId || testRun.projectClosureIds.has(plugin.projectId)\`.
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Drop the title-based exclusion so config.webServer is shape- and
command-non-empty-checked at load time, same as project.webServer.

Also revert the testRunner.ts hoists from the previous follow-up
commit; the task builders no longer need the TestRun up front.
@github-actions
Copy link
Copy Markdown
Contributor

Test results for "MCP"

7099 passed, 1104 skipped


Merge workflow run.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "tests 1"

5 failed
❌ [playwright-test] › web-server.spec.ts:476 › should support self signed certificate @ubuntu-latest-node20
❌ [playwright-test] › web-server.spec.ts:476 › should support self signed certificate @macos-latest-node20
❌ [playwright-test] › web-server.spec.ts:476 › should support self signed certificate @ubuntu-latest-node24
❌ [playwright-test] › web-server.spec.ts:476 › should support self signed certificate @ubuntu-latest-node22
❌ [playwright-test] › web-server.spec.ts:476 › should support self signed certificate @windows-latest-node20

1 flaky ⚠️ [chromium-library] › library/video.spec.ts:719 › screencast › should work with video+trace `@chromium-ubuntu-22.04-arm-node20`

41928 passed, 850 skipped


Merge workflow run.

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.

[Feature] Configure web servers per project

2 participants