-
Notifications
You must be signed in to change notification settings - Fork 14
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
chore: use ooni/probe-cli v3.16.6 #315
Conversation
I don't fully understand why the CI is failing here 🤷 Here's the full log: jest -c test/lib/integration.config.js --testRegex 'e2e.js' --runInBand PASS test/onboarding.e2e.js (29.971 s) Onboarding Story 1 ✓ App is running (132 ms) ✓ App launches with correct title (92 ms) ✓ First screen renders correctly (141 ms) ✓ App goes to the second screen (292 ms) ✓ App loads up Pop Quiz (275 ms) ✓ App accepts first Pop Quiz Answer (1644 ms) ✓ App accepts second Pop Quiz Answer and goes to Crash Reporting page (1673 ms) ✓ App lets user opt-in for Crash Reporting (214 ms) ✓ Finishing Onboarding process brings up the Dashboard (307 ms) ✓ Check if Crash Reporting is enabled in Settings (295 ms) Onboarding Story 2 ✓ App plays cross animation and shows warning if first pop quiz answer is wrong (4853 ms) ✓ App plays cross animation and shows warning if second pop quiz answer is wrong (1811 ms) ✓ App lets user opt out of Crash Reporting (250 ms) ✓ Finishing Onboarding process brings up the Dashboard (292 ms) ✓ Check if Crash Reporting is disabled in Settings (304 ms) Ah, maybe the issue is this:
Apparently some end-to-end tests fail. (Maybe I can run them locally? In the meanwhile I'll restart the build.) |
Okay, I am going to merge because the CI completed successfully. So, it seems it was just flaky before and some tests did not complete in the total amount of time they were given. FTR, this is the result of the run:
So, clearly 30 seconds of timeout is a bit border line for Performance and Experimental. I am surprised about the total runtime of Middleboxes in the previous run, though. 🤔 |
No description provided.