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

feat: add an initial spike of playwright UI tests #1620

Merged
merged 2 commits into from
Nov 17, 2023

Conversation

chirino
Copy link
Member

@chirino chirino commented Nov 9, 2023

Lets see if we can figure out a nice workflow to use these for e2e and qa testing.

Copy link

netlify bot commented Nov 9, 2023

Deploy Preview for nexodus-docs canceled.

Name Link
🔨 Latest commit 769ae5b
🔍 Latest deploy log https://app.netlify.com/sites/nexodus-docs/deploys/655505a2b301100008cdd0fd

@mergify mergify bot added missing-docs-label missing has-docs or no-docs-needed missing-design-label missing has-design or no-design-needed label missing-tests-label This PR is missing a tests label, has-tests or no-tests-required labels Nov 9, 2023
@chirino chirino added no-design-required This PR does not require a design document has-tests PR includes test coverage and removed missing-design-label missing has-design or no-design-needed label missing-tests-label This PR is missing a tests label, has-tests or no-tests-required labels Nov 9, 2023
Copy link
Contributor

mergify bot commented Nov 13, 2023

@chirino, this pull request is now in conflict and requires a rebase.

@mergify mergify bot added the needs-rebase label Nov 13, 2023
@chirino chirino force-pushed the playwright branch 3 times, most recently from e357dca to d31ddef Compare November 14, 2023 22:46
@chirino chirino added no-docs-required This PR does not require user facing docs. and removed missing-docs-label missing has-docs or no-docs-needed needs-rebase labels Nov 14, 2023
They are called from the feature e2e tests, this lets us use cucumber statements to set up UI test scenarios.

Signed-off-by: Hiram Chirino <[email protected]>
Copy link
Member

@nerdalert nerdalert left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify mergify bot merged commit 4417c8a into nexodus-io:main Nov 17, 2023
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has-tests PR includes test coverage no-design-required This PR does not require a design document no-docs-required This PR does not require user facing docs.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants