-
Notifications
You must be signed in to change notification settings - Fork 11
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
GH-84 - Some initial e2e tests #85
Conversation
Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
|
Note that one of the tests fails on webkit. This is because the checkbox widget is not tab-selectable there. I left the test failure in there for now as this might be something we really want to fix? |
I've limited the runs to be on Chromium and Firefox for now until the checkbox selection issue on Webkit is fixed. That way the playwright tests pass. |
What do you think about...
Not sure if |
unit tests -> test/unit e2e_test -> test/e2e
Good point, I relocated the tests now as follows (so used
|
@auniverseaway I think I addressed your concerns, let me know if you need any further changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One last small thing I'd recommend changing (but not required).
I don't think I have a strong opinion on this, but it's an interesting observation: Having another package.json in a sub-folder puts us squarely in
|
Description
Use Playwright to create an initial set of E2E tests.
This PR contributes the setup for Playwright which is in the
e2e_test
subdirectory.Tests can be run from the root using
npm run test:e2e
or using the variousnpm run test:...
scripts in thee2e_test
directory.Normally the tests are run on Chrome, Firefox and Webkit.
This PR contributes 3 tests:
This is only the start of the E2E test suite. Hopefully many more test cases will follow.
Related Issue
Contributes to #84
Motivation and Context
E2E tests are important to ensure product quality.
How Has This Been Tested?
It is a test 😉
Types of changes
Checklist: