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

E2E: Write a separate test for accepting notifications #2930

Open
LeandraH opened this issue Sep 17, 2024 · 0 comments
Open

E2E: Write a separate test for accepting notifications #2930

LeandraH opened this issue Sep 17, 2024 · 0 comments
Labels
not-testable Not testable by our tester Task Test

Comments

@LeandraH
Copy link
Contributor

Is your feature request related to a problem? Please describe.
We currently check in e2e tests that the permission to send notifications is requested in each test because we don't know which one will be called first. If the permission request doesn't show up, an unrelated test fails, which isn't great. Additionally, with #2868, that check becomes optional, so we aren't testing the notifications at all anymore.

Describe the solution you'd like
I would like a new e2e test that does nothing other than open the app, wait for the permission popup to appear, and accepts it.

Describe alternatives you've considered
We could leave it untested but that's not great since we keep having problems with our notifications.

@LeandraH LeandraH added Task Test not-testable Not testable by our tester labels Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-testable Not testable by our tester Task Test
Projects
Status: No status
Development

No branches or pull requests

1 participant