Skip to content

add wait for idle and disable scroll and textview cursor in tests - #7

Merged
dnieto-r merged 4 commits into
task/16401-publish-testfrom
task/wait-for-idle
Sep 10, 2025
Merged

add wait for idle and disable scroll and textview cursor in tests#7
dnieto-r merged 4 commits into
task/16401-publish-testfrom
task/wait-for-idle

Conversation

@dnieto-r

@dnieto-r dnieto-r commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

🎟️ Jira ticket

ANDROID-16401

🥅 What's the goal?

After testing the app for several days, we noticed that sometimes some screenshots were regenerated because it didn't wait for the ripple animations to finish, or the scrollbar appeared halfway through, etc. The goal of this PR is to fix these issues.

🚧 How do we do it?

To fix this, we analyzed the shot library and saw that it used the disableFlakyComponentsAndWaitForIdle() function, which hides the scroll view and waits for animations to finish, so we added that same code with some adaptations.

@dnieto-r
dnieto-r requested review from a team, hjorrod, jeprubio and jeslat and removed request for a team September 4, 2025 07:54

@jeprubio jeprubio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@dnieto-r
dnieto-r requested a review from dpastor September 4, 2025 11:24
@dnieto-r dnieto-r changed the title add wait for idle and disable animations add wait for idle and disable scroll and textview cursor in tests Sep 4, 2025

@jeslat jeslat left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

@dpastor dpastor left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok!

@dnieto-r
dnieto-r merged commit 2c4de8d into task/16401-publish-test Sep 10, 2025
1 check passed
@dnieto-r
dnieto-r deleted the task/wait-for-idle branch September 10, 2025 06:01
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.

4 participants