test(ios): defuzz iOS e2e workflow a bit more #8679
Merged
Conversation
This file contains hidden or 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
there were two places this was defined, and I just added 429 / quota-exceeded retries to one, both could use it though so focus all cloud helper function usage through a single code path
also add some continue-on-error's into other, hasn't been a problem, but the steps that will now continue-on-error should not stop the workflow
perhaps we can isolate and handle Simulator boot issues
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Separating the iOS Simulator boot from Detox run resulted in a drop from 33% failure to 10%, which is a pretty big improvement, going to merge. https://github.com/invertase/react-native-firebase/actions/runs/17351439598 The 10% all share a pattern where the app went to get something external (analytics.sessionId or remoteConfig onConfigUpdate or remoteConfig activation status) and appeared to hang. Very strange. Next thought is verbose jest and detox logging I suppose |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Description
Still thinking about how to get e2e working more reliably for iOS runs, and this PR collects those ideas
Still not handled but needs to be as it happens a lot:
Related issues
Release Summary
all tests, no release will trigger
Checklist
AndroidiOSOther(macOS, web)e2etests added or updated inpackages/\*\*/e2ejesttests added or updated inpackages/\*\*/__tests__Test Plan
It's all test work
Think
react-native-firebaseis great? Please consider supporting the project with any of the below:React Native FirebaseandInvertaseon Twitter