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

Email workflow test changes needed to pass in multiple browsers #1409

Open
rmunn opened this issue Jan 23, 2025 · 0 comments · May be fixed by #1410
Open

Email workflow test changes needed to pass in multiple browsers #1409

rmunn opened this issue Jan 23, 2025 · 0 comments · May be fixed by #1410
Assignees
Labels
🧑‍💻 DX Developer experience problem testing

Comments

@rmunn
Copy link
Contributor

rmunn commented Jan 23, 2025

Describe the bug
One email workflow test I was working on was passing in Chromium but failing in Firefox, and I believe I have figured out why.

The email subject line is: "Project join request: (user name) wants to join (project name)". These are randomly-generated test users with a GUID in their name. The email is being sent to the [email protected] email address.

Currently, our email testing code in frontend/tests/email/mailbox.ts has code to open an email inbox for a certain user, and click on the email whose subject is taken from an enum, i.e. "Project join request". This works when the mailbox belongs to the generated test user, because there will be only one "Verify Email" message in the inbox. But when the mailbox belongs to [email protected], the username being searched for needs to be part of the subject line searched, so we'll need to allow arbitrary strings to be passed in to the mailbox.ts openEmail function.

I'm recording this as a separate issue because I might not get to it right away, but I want to keep track of my findings.

@rmunn rmunn added testing 🧑‍💻 DX Developer experience problem labels Jan 23, 2025
@rmunn rmunn self-assigned this Jan 23, 2025
@rmunn rmunn linked a pull request Jan 23, 2025 that will close this issue
@rmunn rmunn linked a pull request Jan 23, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧑‍💻 DX Developer experience problem testing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant