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 user flows to test automation starter #46

Merged
merged 5 commits into from Sep 28, 2018
Merged

Conversation

thywen
Copy link
Contributor

@thywen thywen commented Sep 7, 2018

The user flow pattern

The user flow pattern is a level of abstraction between the actual tests and the page objects. It is used to reduce the duplications of commonly used flows as well as the numbers of lines in the test methods to make them easier to understand.

This pr adds the usage of these User flows and the page object chaining to the framework. I also changed the method names of the checks to make the purpose clearer.

@@ -19,13 +19,13 @@
import static android.support.test.espresso.matcher.ViewMatchers.isDisplayed;
import static android.support.test.espresso.matcher.ViewMatchers.withId;

public class EspressoJavaTestExample {
public class EspressoJavaTestExampleWithoutFramework {
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@@ -19,7 +19,7 @@ import org.junit.Rule
import org.junit.Test
import java.io.InputStream

class MockServerTestExample {
class EspressoTestExampleWithMockServer {
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

val category = "HATS"
val product = "hat white"

productUserFlow {
Copy link
Contributor

Choose a reason for hiding this comment

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

💯 nice improvement !

@thywen thywen merged commit a17dbf2 into master Sep 28, 2018
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.

None yet

2 participants