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

E2e protractor removal poc #740

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

pauluxxx
Copy link
Contributor

@pauluxxx pauluxxx commented Dec 23, 2024

Frontend Pull Request Description

This PR contains the examples for the different tests framework. Docs describing the comparison.

Progress from this PR

webdriver-io

https://webdriver.io/docs/gettingstarted

Great tool, best match in the test harness, suits, selenium based solution.
cd packages/bits && yarn run wdio - executes button e2e tests
cli builder from ng e2e - not done but can be worked out
visual/a11y - can be worked out later
ci/cd - not investigated

cypress

https://docs.cypress.io/app/get-started/why-cypress
NOT investigated

Nightwatch

https://nightwatchjs.org/guide/overview/what-is-nightwatch.html
cd packages/bits && yarn run nightwatch - executes dummy examples
no easy rewrite. has different syntax and we will need to rewrite some parts of test files from scratch and reuse some of the parts from the existing tests on protractor.

cli builder from ng e2e - not investigated
visual/a11y - can be worked out later
ci/cd - not investigated

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have updated change log
  • I have been following Definition of done
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new lint warnings
  • New and existing unit tests pass locally and on CI with my changes
  • Any dependent changes have been merged and published in downstream modules

Screenshots (if applicable)

Additional Context (if necessary)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants