Skip to content

Introduce E2E tests with Playwright#55

Open
pfferrari wants to merge 1 commit into
mainfrom
add-test-e2e
Open

Introduce E2E tests with Playwright#55
pfferrari wants to merge 1 commit into
mainfrom
add-test-e2e

Conversation

@pfferrari
Copy link
Copy Markdown
Contributor

@pfferrari pfferrari commented Mar 17, 2026

What I did

Introduced E2E tests with Playwright by:

  • Setting up the Playwright configuration from scratch
  • Adding command line scripts
  • Introducing a first spec to test if the app is just working properly with required URL parameters (defined via env file)
  • Introducing an app flow spec to test the customer signup and the customer login reusing the same credentials created once and shared between the two steps

Checklist

  • Make sure your changes are tested (stories and/or unit, integration, or end-to-end tests).
  • Make sure to add/update documentation regarding your changes.
  • You are NOT deprecating/removing a feature.

@pfferrari pfferrari force-pushed the add-test-e2e branch 2 times, most recently from 5d0c3f8 to 4b9cd4a Compare March 17, 2026 12:14
@pfferrari pfferrari self-assigned this Mar 17, 2026
@pfferrari pfferrari added the enhancement New feature or request label Mar 17, 2026
@pfferrari pfferrari force-pushed the add-test-e2e branch 2 times, most recently from 62a01c4 to 1b09bef Compare March 19, 2026 18:18
@pfferrari pfferrari requested a review from Copilot March 27, 2026 09:01

This comment was marked as outdated.

This comment was marked as outdated.

@pfferrari pfferrari force-pushed the add-test-e2e branch 2 times, most recently from f6289d4 to 1f5af40 Compare March 27, 2026 17:30
@pfferrari pfferrari requested a review from Copilot March 27, 2026 17:30

This comment was marked as outdated.

This comment was marked as resolved.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as resolved.

Copy link
Copy Markdown

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 12 out of 14 changed files in this pull request and generated 2 comments.


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

Comment thread packages/app/playwright.config.ts Outdated
Comment thread packages/app/specs/e2e/test-helpers.ts
Comment thread packages/app/specs/e2e/test-helpers.ts Outdated
@malessani
Copy link
Copy Markdown
Contributor

I suggest you to change the circleci image to be the one specified here: https://playwright.dev/docs/ci#circleci

@pfferrari pfferrari force-pushed the add-test-e2e branch 7 times, most recently from 6fd77de to dbe9798 Compare April 21, 2026 20:54
@pfferrari pfferrari closed this Apr 21, 2026
@pfferrari pfferrari reopened this Apr 21, 2026
@pfferrari pfferrari force-pushed the add-test-e2e branch 6 times, most recently from 0d6f67b to 87f3932 Compare April 21, 2026 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants