Idrinth / habit-evaluator / 22227815975 / 2

40%

the-one: 40%

push

github

Fix Maestro CRUD test failures by adding scrollUntilVisible waits (#535)

Replace bare assertVisible and direct tapOn with scrollUntilVisible for
elements that may not be immediately accessible after ViewPager2 smooth
scroll navigation or keyboard interactions. Two root causes:

1. Premature assertion: After tapping a diary navigation card, the
   assertVisible matched the card's own text (still visible during the
   ViewPager2 animation) instead of waiting for the target fragment.
   Fix: wait for a unique element in the target fragment instead.

2. Off-screen buttons: In NestedScrollView fragments, buttons at the
   bottom of forms can be below the visible area after text input and
   keyboard dismissal. Maestro's tapOn by ID fails when the element
   is not in the visible portion of a scrollable container.
   Fix: scrollUntilVisible before each tapOn for form buttons.

Tests fixed: 14 (diary), 16 (food log), 17 (sport log),
18 (medication), 19 (medication log), 20 (emergency plan step).

https://claude.ai/code/session_01DQeiKNFT9xpoVFNWEsocXe

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

607 of 2714 branches covered (22.37%)

Branch coverage included in aggregate %.

1857 of 7759 relevant lines covered (23.93%)

1.13 hits per line