-
Notifications
You must be signed in to change notification settings - Fork 61
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
Horizontal swipe with gesture Plugin #238
Horizontal swipe with gesture Plugin #238
Conversation
…_gesture # Conflicts: # src/main/java/com/znsio/teswiz/runner/Driver.java
src/test/java/com/znsio/teswiz/screen/android/jiocinema/JioCinemaScreenAndroid.java
Outdated
Show resolved
Hide resolved
src/test/java/com/znsio/teswiz/screen/android/jiocinema/JioCinemaScreenAndroid.java
Outdated
Show resolved
Hide resolved
…th_gesture # Conflicts: # src/main/java/com/znsio/teswiz/runner/Driver.java
@@ -40,4 +40,9 @@ public void iSwipeLeftOnTrendingInIndiaSection() { | |||
public void iAmAbleToViewNumberTrendingCinema(int movieNumberOnScreen) { | |||
new JioCinemaBL(SAMPLE_TEST_CONTEXT.ME, Runner.getPlatform()).verifyMovieNumberVisibleOnScreen(movieNumberOnScreen); | |||
} | |||
|
|||
@When("I swipe {string} trending no {int} on trending in india section") | |||
public void iSwipeRightTrendingNoOnTrendingInIndiaSection(String direction, int movieNumberOnScreen) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
method name has "right" in it, where as the direction is a parameter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the method name
|
||
@When("I swipe {string} trending no {int} on trending in india section") | ||
public void iSwipeRightTrendingNoOnTrendingInIndiaSection(String direction, int movieNumberOnScreen) { | ||
new JioCinemaBL(SAMPLE_TEST_CONTEXT.ME, Runner.getPlatform()).swipeMovieTrendingInIndiaSection(direction,movieNumberOnScreen); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
instead of passing the direction as a string, convert it to the Direction enum here itself, and pass that as a parameter - Direction.valueOf(direction.toUpperCase()
See this implementation in AutoScrollSteps.java - iShouldBeAbleToScrollInDynamicLayer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated with Direction object in method parameters
This PR covers implementation for Horizontal swipe action using gesture plugin in teswiz framework