Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI Tests - Add Launch Argument to Disable Animations #623

Closed
rhsalas3 opened this issue May 2, 2022 · 1 comment · Fixed by #624
Closed

UI Tests - Add Launch Argument to Disable Animations #623

rhsalas3 opened this issue May 2, 2022 · 1 comment · Fixed by #624
Labels
automation Enhancement New feature or request Nice To Have Designates a feature/task that would be nice to implement but is not a priority Testing...

Comments

@rhsalas3
Copy link
Contributor

rhsalas3 commented May 2, 2022

Description:
Adding a launch argument for the app that allows us to disable animations can speed up our UI tests. It can also reduce potential areas of flakiness when it comes to waiting for screen transitions.

Acceptance Criteria:
A launch argument exists that, when declared at app startup, will disable animations on the device under test
Something like:

        app.configure {
            Arguments(.resetAppDataOnLaunch, .enableListeningMode, .disableAnimations)
        }
@rhsalas3 rhsalas3 added Enhancement New feature or request automation Nice To Have Designates a feature/task that would be nice to implement but is not a priority labels May 2, 2022
@rhsalas3 rhsalas3 linked a pull request May 2, 2022 that will close this issue
@rhsalas3
Copy link
Contributor Author

rhsalas3 commented May 3, 2022

After disabling animations we found that some tests are failing due to a known bug (which seems more exposed now w/ animations off). We opted to just disable the affected tests. Once the bug(s) are fixed then we will re-enable them.

Tests affected by the bugs:
CategoryPhrasesPaginationTests::testCanNavigatePages()
MainScreenPaginationTests::testCanScrollPagesWithPaginationArrows()

Bugs:
#597
#594

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation Enhancement New feature or request Nice To Have Designates a feature/task that would be nice to implement but is not a priority Testing...
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant