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

Fix native Latest Posts e2e device tests #3945

Merged
merged 2 commits into from
Sep 9, 2021

Conversation

dcalhoun
Copy link
Member

@dcalhoun dcalhoun commented Sep 9, 2021

Related PRs

Description

Increase the accuracy of the swipe distance required to scroll one page of blocks within the block inserter. We now have more than two pages of blocks. The previous implementation would scroll too far (to the bottom), which caused failures when the test scrolled past the subject block.

To test: Verify CI checks pass.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

@dcalhoun dcalhoun added [Type] Technical Debt Testing Anything related to automated tests labels Sep 9, 2021
@dcalhoun dcalhoun self-assigned this Sep 9, 2021
@dcalhoun dcalhoun added this to the 1.62.0 (18.3) milestone Sep 9, 2021
@dcalhoun dcalhoun marked this pull request as ready for review September 9, 2021 21:26
@jd-alexander jd-alexander self-requested a review September 9, 2021 21:48
Copy link
Contributor

@jd-alexander jd-alexander left a comment

Choose a reason for hiding this comment

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

@peril-wordpress-mobile
Copy link

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@dcalhoun dcalhoun enabled auto-merge September 9, 2021 22:58
@dcalhoun dcalhoun merged commit 9d75129 into develop Sep 9, 2021
@dcalhoun dcalhoun deleted the bug/fix-latest-posts-e2e-test branch September 9, 2021 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Testing Anything related to automated tests [Type] Technical Debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants