Skip to content

feat(skills): add component-testing agent skill#41738

Open
pavelfeldman wants to merge 2 commits into
microsoft:mainfrom
pavelfeldman:component-testing-skill
Open

feat(skills): add component-testing agent skill#41738
pavelfeldman wants to merge 2 commits into
microsoft:mainfrom
pavelfeldman:component-testing-skill

Conversation

@pavelfeldman

Copy link
Copy Markdown
Member

Summary

  • Adds a playwright-component-testing skill shipped in the playwright package, replacing @playwright/experimental-ct-* with plain e2e tests against a story gallery: stories colocated with components, a gallery page served by the app's dev server, and a mount(storyId) fixture returning a locator. Installed via npx playwright init-skills.
  • Skill templates live in top-level packages/skills, copied into playwright/lib/skills at build time.
  • Includes React and Vue templates, a migration guide, and two example projects under tests/components that instantiate the templates byte-for-byte (enforced by a sync test).

Adds a playwright-component-testing skill shipped in the playwright
package, replacing @playwright/experimental-ct-* with plain e2e tests
against a story gallery: stories colocated with components, a gallery
page served by the app's dev server, and a `mount(storyId)` fixture
returning a locator. Installed via `npx playwright init-skills`.

Skill templates live in top-level packages/skills, outside of package
sources, so they need no check-deps exception; they are copied into
playwright/lib/skills at build time.

Includes React and Vue templates, a migration guide, and two example
projects under tests/components that instantiate the templates
byte-for-byte (enforced by a sync test).
Removed deprecation warning for Playwright component testing.
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

8 flaky ⚠️ [chromium-library] › library/chromium/oopif.spec.ts:282 › should click `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/video.spec.ts:717 › screencast › should work with video+trace `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/chromium/chromium.spec.ts:434 › should produce network events, routing, and annotations for Service Worker (advanced) `@realtime-time-library-chromium-linux`
⚠️ [chromium-library] › library/video.spec.ts:476 › screencast › should capture static page in persistent context @smoke `@realtime-time-library-chromium-linux`
⚠️ [chromium-library] › library/popup.spec.ts:260 › should not throw when click closes popup `@chromium-ubuntu-22.04-node24`
⚠️ [chromium-library] › library/video.spec.ts:337 › screencast › should work for popups `@chromium-ubuntu-22.04-node22`
⚠️ [firefox-page] › page/page-emulate-media.spec.ts:144 › should keep reduced motion and color emulation after reload `@firefox-ubuntu-22.04-node20`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:388 › should reveal errors in the sourcetab `@windows-latest-node22`

49516 passed, 1161 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

2 failed
❌ [firefox] › mcp/http.spec.ts:104 › http transport browser lifecycle (isolated) @mcp-ubuntu-latest-firefox
❌ [webkit] › mcp/annotate.spec.ts:110 › should abort annotation when last screenshot is removed @mcp-ubuntu-latest-webkit

7758 passed, 1249 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.

1 participant