Skip to content

test(ci): unique-ify secondary app names in tests - #9139

Merged
russellwheatley merged 3 commits into
mainfrom
e2e-deflake-20260731
Aug 3, 2026
Merged

test(ci): unique-ify secondary app names in tests#9139
russellwheatley merged 3 commits into
mainfrom
e2e-deflake-20260731

Conversation

@mikehardy

Copy link
Copy Markdown
Collaborator

Summary

  • Soft-fail Jet transformFailure when Metro symbolicate is unavailable so Release e2e keeps the original error (D1).
  • Retry Jet only when reconnect_recovered coincides with a mid-suite desync signal (suite re-entry / coverage 0/0 / server stopped), not on bare reconnect (D2).
  • Uniquify the functions modular secondary initializeApp app name with Date.now() and clean up via deleteApp so suite re-entry cannot collide (D4).

Test plan

  • D1/D2/D4 area-focused app+functions e2e on iOS and Android (74 passing / 1 pending)
  • CI e2e matrix on this PR

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@codecov

codecov Bot commented Aug 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.47%. Comparing base (a1e9ffe) to head (5cff790).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #9139      +/-   ##
============================================
- Coverage     66.13%   63.47%   -2.65%     
============================================
  Files           497      389     -108     
  Lines         39249    30763    -8486     
  Branches       5905     4870    -1035     
============================================
- Hits          25953    19524    -6429     
+ Misses        11861    10481    -1380     
+ Partials       1435      758     -677     
Flag Coverage Δ
android-native ?
e2e-ts-android ?
e2e-ts-ios 57.94% <ø> (-5.05%) ⬇️
e2e-ts-macos 49.73% <ø> (-0.03%) ⬇️
ios-native 57.94% <ø> (-5.05%) ⬇️
jest 49.69% <ø> (-0.67%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@russellwheatley
russellwheatley merged commit 2d73cdc into main Aug 3, 2026
27 checks passed
@russellwheatley
russellwheatley deleted the e2e-deflake-20260731 branch August 3, 2026 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants