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

Add list block test #970

Merged
merged 11 commits into from
May 21, 2019
Merged

Conversation

hypest
Copy link
Contributor

@hypest hypest commented May 10, 2019

This PR builds on top of #821 and tries to add a List block test.

On the Android side of things, it also supports verifying that the block html output is consistent with what we expect for the list block.

By the way, the Appium tests run green locally when trying them on a Nexus 5X with Android 7.1.2, at least as of d65b6fa.

@hypest hypest requested a review from JavonDavis May 10, 2019 08:24
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented May 10, 2019

Warnings
⚠️ PR is not assigned to a milestone.

Generated by 🚫 dangerJS

@hypest hypest added Blocks Testing Anything related to automated tests labels May 10, 2019
@JavonDavis JavonDavis merged commit fabc73c into add/paragraph-flow-tests May 21, 2019
@JavonDavis JavonDavis deleted the tests/add-list-test branch May 21, 2019 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocks Testing Anything related to automated tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants