super3 / water / 26670038952

82%

push

github

Fix instrumented tests: best-effort Pebble send + Espresso 3.7.0

Two CI failures in the instrumented job, both surfaced by the watch-sync work:

1. pushStateToPebble() called DefaultPebbleSender.close() in a finally block.
   When no Pebble app is connected (every CI emulator), the service bind never
   happens, so close() throws "Service not registered" and crashes the coroutine
   that the WaterAppTest cases drive via persist(). Make the whole send
   best-effort: swallow send errors and guard close() with runCatching, so a
   missing/disconnected watch never crashes the caller. (On real hardware the
   Core app binds, so this path always worked there.)

2. Bump Espresso 3.6.1 -> 3.7.0 (and ext:junit 1.2.1 -> 1.3.0): 3.6.1's event
   injector reflects on the removed InputManager.getInstance() and crashes on
   API 35/36, so the suite can't run on modern emulators. 3.7.0 fixes it and is
   backward compatible.

Verified: createDebugAndroidTestCoverageReport runs green (7/7) on an API 36
emulator.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

184 of 318 branches covered (57.86%)

Branch coverage included in aggregate %.

1 of 2 new or added lines in 1 file covered. (50.0%)

605 of 864 relevant lines covered (70.02%)

10.16 hits per line