Skip to content

Playwright Setup#60

Open
kumaradityaraj wants to merge 10 commits intoserverlessworkflow:mainfrom
kumaradityaraj:playwright
Open

Playwright Setup#60
kumaradityaraj wants to merge 10 commits intoserverlessworkflow:mainfrom
kumaradityaraj:playwright

Conversation

@kumaradityaraj
Copy link
Copy Markdown
Contributor

@kumaradityaraj kumaradityaraj commented Apr 13, 2026

Close Set up Playwright end-to-end testing

For testing :-

  1. Navigate to the package
  2. run pnpm test:e2e or pnpm test:e2e:ui

Signed-off-by: kumaradityaraj <sedulous.0007@gmail.com>
Signed-off-by: kumaradityaraj <sedulous.0007@gmail.com>
Copilot AI review requested due to automatic review settings April 13, 2026 06:59
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a Playwright-based E2E testing package to the monorepo and includes an initial test validating the Diagram Editor Storybook render.

Changes:

  • Introduces a new @serverlessworkflow/playwright workspace package with Playwright config and scripts.
  • Adds an initial E2E spec for the Diagram Editor (Storybook iframe route).
  • Ignores Playwright output artifacts in .gitignore.

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
packages/playwright/tests/diagram-editor.spec.ts Adds an initial E2E test against the Diagram Editor Storybook iframe.
packages/playwright/playwright.config.ts Configures Playwright base URL and webServer startup for local runs.
packages/playwright/package.json Adds a new workspace package with Playwright dev dependency and scripts.
packages/playwright/README.md Documents how to run the Playwright tests in the monorepo.
.gitignore Ignores Playwright reports/results directories.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: kumaradityaraj <sedulous.0007@gmail.com>
@kumaradityaraj
Copy link
Copy Markdown
Contributor Author

@fantonangeli @lornakelly @handreyrc Please review this PR. Thank you.

Signed-off-by: kumaradityaraj <sedulous.0007@gmail.com>
Copilot AI review requested due to automatic review settings April 14, 2026 09:05
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 11 changed files in this pull request and generated 7 comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: kumaradityaraj <sedulous.0007@gmail.com>
Signed-off-by: kumaradityaraj <sedulous.0007@gmail.com>
Copilot AI review requested due to automatic review settings April 14, 2026 09:20
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 7 changed files in this pull request and generated 3 comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: kumaradityaraj <sedulous.0007@gmail.com>
Signed-off-by: kumaradityaraj <sedulous.0007@gmail.com>
Copilot AI review requested due to automatic review settings April 14, 2026 09:29
Signed-off-by: kumaradityaraj <sedulous.0007@gmail.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 6 changed files in this pull request and generated no new comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: kumaradityaraj <sedulous.0007@gmail.com>
@kumaradityaraj
Copy link
Copy Markdown
Contributor Author

@fantonangeli @lornakelly @handreyrc Please review this PR.

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.

Set up Playwright end-to-end testing

2 participants