Experiment/create ephemeral UI test accts #1898
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…int on the API to create a UI testing account with it
…tore the email address while cypress is running
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
🧪 Review environmenthttps://awsm5zbzpkf7rxpmqpt3oey4gy0ytieq.lambda-url.ca-central-1.on.aws/ |
…tions are always closed and not dangling
…word since API already has this secret
🧪 Review environmenthttps://hu4pteypukhnutyk2w6ancyfde0kjhja.lambda-url.ca-central-1.on.aws/ |
🧪 Review environmenthttps://b2xc3okljybta7biudmui2keti0hffli.lambda-url.ca-central-1.on.aws/ |
… for language to change
…ithub.com/cds-snc/notification-admin into experiment/create-ephemeral-ui-test-accts
…ithub.com/cds-snc/notification-admin into experiment/create-ephemeral-ui-test-accts
…ithub.com/cds-snc/notification-admin into experiment/create-ephemeral-ui-test-accts
andrewleith
commented
Nov 15, 2024
| }, | ||
| deleteAllEmails() { | ||
| return emailAccount.deleteAllEmails() | ||
| }, | ||
| fetchEmail(acct) { | ||
| return emailAccount.fetchEmail(acct) | ||
| }, | ||
| createEmailAccount() { | ||
| return emailAccount.createEmailAccount(); | ||
| createAccount({ baseUrl, username, secret }) { |
There was a problem hiding this comment.
The result of this call is "cached" in the global object here, which means only one set of test users is generated for every test run or every time you launch the cypress UI. To get a new user, just quit cypress and launch it again.
andrewleith
commented
Nov 15, 2024
| @@ -82,7 +82,7 @@ const pages = [ | |||
| describe(`A11Y - App pages [${config.CONFIG_NAME}]`, () => { | |||
| for (const page of pages) { | |||
| it(`${page.name}`, () => { | |||
| cy.login(Cypress.env("NOTIFY_USER"), Cypress.env("NOTIFY_PASSWORD")); | |||
| cy.login(); | |||
There was a problem hiding this comment.
The login command has been simplified to use the generated user and the password automatically.
andrewleith
commented
Nov 15, 2024
| const mail = await simpleParser(message.parts[0].body); | ||
| const to_address = mail.to.value[0].address; | ||
|
|
||
| if (to_address == emailAddress) { |
There was a problem hiding this comment.
Only get 2FA codes for the specific email alias we are testing with 🎉
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary | Résumé
This PR hooks into the new
/cypressendpoints introduced in #2286 in order to create UI testing alias accounts on the fly.The end result is that test runs will no longer interfere with each other, meaning we can run as many suites concurrently as we want on the same environment.
Notes
/cypressendpoint on the backend to get new accounts createdlogin(regular user) and loginAsPlatformAdmin (platform admin user)