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: convert universal navigation test to Playwright #54873

Conversation

Sembauke
Copy link
Member

@Sembauke Sembauke commented May 20, 2024

Checklist:

Closes #XXXXX

@github-actions github-actions bot added the scope: tools/scripts Scripts for supporting dev work, generating config and build artifacts, etc. label May 20, 2024
@Sembauke
Copy link
Member Author

I will be adding the donation test for the menu in another pr, most likely in #54857

@Sembauke Sembauke marked this pull request as ready for review May 20, 2024 09:31
@Sembauke Sembauke requested a review from a team as a code owner May 20, 2024 09:31
@moT01 moT01 added the status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. label May 20, 2024
@naomi-lgbt naomi-lgbt changed the title feat: convert universal naviation test to Playwright feat: convert universal navigation test to Playwright May 20, 2024
e2e/header.spec.ts Outdated Show resolved Hide resolved
Co-authored-by: Naomi <accounts+github@nhcarrigan.com>
@Sembauke Sembauke mentioned this pull request May 21, 2024
61 tasks
@ahmaxed
Copy link
Member

ahmaxed commented May 22, 2024

Thanks for the pr.
The Donor Navigation Menu tests are missing.
Additionally the sign in/sign out and cancelling the sign out tests are missing as well.

@Sembauke
Copy link
Member Author

Sembauke commented May 22, 2024

@ahmaxed
#54873 (comment)

See the above comment for the donation tests.

The signout/signin tests are covered in signout-modal.spec.ts

@ahmaxed ahmaxed merged commit 1e83bed into freeCodeCamp:main May 23, 2024
22 checks passed
@Sembauke Sembauke deleted the feat/convert-universal-navigation-to-playwright branch May 23, 2024 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: tools/scripts Scripts for supporting dev work, generating config and build artifacts, etc. status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants