Idrinth / habit-evaluator / 22103195071

40%

push

github

Fix Maestro coverage collection timeout by removing run-as dependency (#457)

The coverage broadcast receiver was writing files only to internal
storage, requiring `adb shell run-as` for extraction. On some API 30
emulator images, `run-as` is unreliable — silent failures cause the
script to never see the status file, reporting "receiver has not
responded" even when the receiver actually ran.

Changes to CoverageBroadcastReceiver:
- Write coverage and status files to both external storage
  (getExternalFilesDir) and internal storage for redundancy
- Add structured logcat messages (COVERAGE_RESULT:OK/EMPTY/NO_CLASS/etc)
  so the script can check receiver status via logcat without file access
- Add entry log (COVERAGE_RECEIVER_ALIVE) for diagnostics

Changes to run-maestro-tests.sh:
- Primary status check via logcat (no run-as needed)
- Primary file extraction via adb pull from external storage (no run-as)
- Keep run-as as fallback for both status and extraction
- Add FLAG_INCLUDE_STOPPED_PACKAGES (0x20) to broadcast for reliable
  delivery after crash + restart
- Increase timeouts (5s after restart, process stability check, longer
  per-attempt waits)
- Add diagnostic output on final failure (receiver registration check,
  run-as test, broadcast output)

https://claude.ai/code/session_017G3eEUe4sudPfFtSLcbGZP

Co-authored-by: Claude <noreply@anthropic.com>

2782 of 6320 branches covered (44.02%)

Branch coverage included in aggregate %.

9427 of 18130 relevant lines covered (52.0%)

2.5 hits per line