-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
tests crash when updated from v17 to v19. Signal 11 was raised #3123
Comments
we are also seeing this issue with xcode 12.5, with v17 the tests work fine. But after updating to v19 we get random test crashes. Is there anything further we can do to isolate the cause of the crash? Attached is the tests logs from Bitrise. |
@asafkorem |
@fatlinesofcode Are you using any Expo libraries? We're getting this Signal 11 error consistently with an Expo bare-workflow app after upgrading to Expo 43. |
yes we're using a few, we ejected originally from v37 but have since upgraded to to rn64 |
@curtismgt The issue showed up for us as well after upgrading to Expo 43. |
This workaround solves the issue described here: firebase/firebase-ios-sdk#9083 And some of the crashes that was mentioned here: - wix#3000 - wix#3123 - wix#2641 - wix#2802
I've created a companion issue in Expo Expo issue 15595. Hopefully we get some action out of these and work this out. We're blocked entirely with e2e on iOS by this and need to switch to another solution if this isn't resolved. |
I'm not sure if this is directly related to Expo but these changes fix the issue for us. #3135 |
Amazing news @christopherdro! |
@curtismgt are you using Firebase? I don't think this problem is from Expo. |
@shamilovtim We are not using Firebase at all. It's possible it's both with the same problem. If you create a new bare Expo app with |
thanks for the suggestions, we'll try and them out and let you know if it resolves our issue |
This workaround solves the issue described here: firebase/firebase-ios-sdk#9083 And some of the crashes that was mentioned here: - #3000 - #3123 - #2641 - #2802
So we managed to get ours working. Keep in mind we are not using the Firebase libraries but the solution may be similar, I'm not familiar with them. Our problem is a conflict of some kind with Detox and Expo Updates after the Expo 43 upgrade. Our solution is to disable the expo-updates native module by setting |
#3135 was merged, it should solve the issue of DetoxSync crashes when Firebase/Performance is integrated, it was tested on the example apps that reproduced this issue. Once you have updated a version (to the latest/next version), if any of you are still having Signal 11 crashes on DetoxSync when Firebase/Performance is integrated, please open a new bug report with the required details. As for the Expo issue, can you please check the next released version that will have this fix ( I hope to investigate those issue soon. For the meanwhile, in case it doesn't solve your crashes, can you please open a designated issue for this (Detox crashes on Expo apps)? |
@curtismgt sounds like you found a workaround. Do you think it worth noting in our docs? Maybe we should mention this on our expo doc page: https://wix.github.io/Detox/docs/guide/expo/ (feel free to open a PR if so). |
@asafkorem we confirmed our crash issue are fixed with the detox 19.4.0 update. In the latest version however 19.4.4 the issue has reappeared. |
See my comment here: #3167 (comment).
|
@fatlinesofcode |
I can confirm that we still have Signal 11 after updating both Firebase and Detox |
@shamilovtim thanks for the report, |
@asafkorem Yeah I'll definitely give my best shot after I've done more triage |
Description
updated project to xcode 13.1, updated detox to v19. tests crash on startup frequently.
always crashes with maxWorkers set to 2, sometimes passes with maxWorkers 1. We are using react-native-firebase too
Screenshots / Video
Environment
Logs
fail.log
pass.log
The text was updated successfully, but these errors were encountered: