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

Fixes for firebase e2e still sometimes failing #6653

Merged
merged 3 commits into from
Aug 21, 2024

Conversation

niklasberglund
Copy link
Collaborator

@niklasberglund niklasberglund commented Aug 21, 2024

e2e tests on Firebase are still sometimes failing due to rate limiting. This PR include some improvements related to failing tests in Firebase:

  • Disable testLoginWithInvalidCredentials because the requests it generates are highly rate limited
  • Logging broke in some bases because the response object was null
  • Configure Firebase to re-run failing tests once

This change is Reviewable

@niklasberglund niklasberglund added the Android Issues related to Android label Aug 21, 2024
Copy link
Collaborator

@albin-mullvad albin-mullvad left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 4 of 4 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@niklasberglund niklasberglund force-pushed the fixes-for-firebase-e2e-still-sometimes-failing branch from 50cebba to e5d66dd Compare August 21, 2024 09:34
Copy link
Collaborator

@albin-mullvad albin-mullvad left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@niklasberglund niklasberglund force-pushed the fixes-for-firebase-e2e-still-sometimes-failing branch from e5d66dd to 78bef00 Compare August 21, 2024 12:16
@niklasberglund niklasberglund marked this pull request as ready for review August 21, 2024 12:16
@niklasberglund niklasberglund force-pushed the fixes-for-firebase-e2e-still-sometimes-failing branch from 78bef00 to 207cc54 Compare August 21, 2024 12:20
@albin-mullvad albin-mullvad merged commit b26659e into main Aug 21, 2024
25 checks passed
@albin-mullvad albin-mullvad deleted the fixes-for-firebase-e2e-still-sometimes-failing branch August 21, 2024 12:23
@niklasberglund niklasberglund self-assigned this Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android Issues related to Android
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants