Skip to content

Add [TIMING] instrumentation to integration test runner - #17

Closed
ofalvai wants to merge 1 commit into
masterfrom
push-xynkovqyvutx
Closed

Add [TIMING] instrumentation to integration test runner#17
ofalvai wants to merge 1 commit into
masterfrom
push-xynkovqyvutx

Conversation

@ofalvai

@ofalvai ofalvai commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@ofalvai
ofalvai force-pushed the push-xynkovqyvutx branch 11 times, most recently from de6f999 to bb0eab9 Compare August 5, 2026 07:37
…witches, and fix Android's fixed 10s teardown delay

- Instrument the mocha test runner with [TIMING] log lines across shell
  exec calls, emulator boot, per-test device prep, per-scenario builds,
  and update-archive creation, to find what dominates test suite runtime.
- RNIOS.buildApp now only runs a real xcodebuild for the first build of
  a project; subsequent scenario switches invoke react-native-xcode.sh
  directly (the same script Xcode's build phase runs) to re-bundle JS
  into the already-built .app, since native code never changes between
  scenarios. Confirmed on CI: 33% cut in build cost, ~2 min off the job,
  no regressions.
- AndroidEmulatorManager.endRunningApplication no longer sleeps a fixed
  10000ms after every force-stop (a zero-variance artificial delay that
  measured ~44% of the entire Android fast-test phase on CI). It now
  polls `adb shell pidof` every 200ms until the process is confirmed
  gone, capped at the old 10000ms as a safety ceiling so reliability
  can't regress below prior behavior. Validating locally before this
  push; pushing now to get a real CI read in parallel.
@ofalvai
ofalvai force-pushed the push-xynkovqyvutx branch from bb0eab9 to 2246c57 Compare August 5, 2026 08:29
@ofalvai ofalvai closed this Aug 5, 2026
@ofalvai
ofalvai deleted the push-xynkovqyvutx branch August 11, 2026 18:25
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.

1 participant