Idrinth / habit-evaluator / 22257586622

40%

push

github

Add animation wait steps to navigation flow test (#556)

* Add waitForAnimationToEnd before toolbar navigation in Maestro test 13

The statsTitle assertion was failing because the ViewPager animation
from navigating to the Emotions tab hadn't fully settled before the
statsButton tap triggered another setCurrentItem call. The element
can become visible during animation (fragment inflation), but the
ViewPager itself may still be animating, causing the subsequent
programmatic navigation to Stats (page 7) to not execute properly.

https://claude.ai/code/session_01ReJ6Ju7Tjy5M1PRpDUqStn

* Add waitForAnimationToEnd at all risky ViewPager transitions in test 13

The end-to-end navigation flow test makes many large ViewPager jumps
(12-17 pages) where cached fragments make elements visible before
animations fully settle. When a subsequent setCurrentItem call fires
during an unsettled animation, the navigation can silently fail.

Added waitForAnimationToEnd at:
- Section transitions (toolbar, diary sub-pages, fragment sub-pages)
- Before each diary sub-page return to Diary hub (12-16 page jumps)
- After each return to Diary before the next card tap
- Before button taps on cached parent pages (Home, Emotions, Sleep)

The command is lightweight — it returns immediately when no animation
is running, so it only adds delay when actually needed.

https://claude.ai/code/session_01ReJ6Ju7Tjy5M1PRpDUqStn

---------

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

3390 of 6877 branches covered (49.29%)

Branch coverage included in aggregate %.

12183 of 19640 relevant lines covered (62.03%)

3.13 hits per line