Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

POC: Compare Playwright with existing cypress test workflow #8714

Closed
wants to merge 2 commits into from

Conversation

nihal467
Copy link
Member

@nihal467 nihal467 commented Oct 4, 2024

  • Command to Start the Test

npx playwright test --ui

Copy link

netlify bot commented Oct 4, 2024

Deploy Preview for care-ohc ready!

Name Link
🔨 Latest commit 5f5b471
🔍 Latest deploy log https://app.netlify.com/sites/care-ohc/deploys/6702c23e062ef50008a6911a
😎 Deploy Preview https://deploy-preview-8714--care-ohc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

cloudflare-workers-and-pages bot commented Oct 4, 2024

Deploying care-fe with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5f5b471
Status: ✅  Deploy successful!
Preview URL: https://e4f89626.care-fe.pages.dev
Branch Preview URL: https://migrate-to-playwright.care-fe.pages.dev

View logs

Copy link

cypress bot commented Oct 4, 2024

CARE    Run #3541

Run Properties:  status check passed Passed #3541  •  git commit 5f5b471772: POC: Compare Playwright with existing cypress test workflow
Project CARE
Branch Review migrate-to-playwright
Run status status check passed Passed #3541
Run duration 03m 10s
Commit git commit 5f5b471772: POC: Compare Playwright with existing cypress test workflow
Committer Mohammed Nihal
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 124
View all changes introduced in this branch ↗︎

@nihal467 nihal467 linked an issue Oct 4, 2024 that may be closed by this pull request
@nihal467
Copy link
Member Author

nihal467 commented Oct 6, 2024

Closing the POC of Playwright migration because of the following reasons:

  • Flaky tests are still occurring even after using the await functionality.
  • We cannot freeze a particular step in the test video replay to understand which APIs were called during a specific action.
  • Migrating all the files, even with a specifically configured chatbot, can be energy- and time-consuming, which doesn’t provide real value as debugging the code might take time.
  • For integrating BrowserStack and commenting on each URL PR preview, we need to run separate GitHub actions and maintain all configurations ourselves, whereas Cypress offers single-click cloud integration, which is easier.

@nihal467 nihal467 closed this Oct 6, 2024
@rithviknishad rithviknishad deleted the migrate-to-playwright branch December 2, 2024 03:28
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.

POC: Migrate Cypress to Playwright for Initial Evaluation
1 participant