Skip to content

Commit b8edd47

Browse files
committed
Minor fixes:
- TouchActions: -- Updating imports -- beautifying code - ITouchActions -- Adding detailed description for swipeWithLongPress() - ElementTouchActions -- Updating imports -- Updating references to ITouchActionsConfiguration - ViewControlsScreen -- updating imports - AndroidBasicInteractionsTest -- Adding new call to isDisplayed()
1 parent 7930c56 commit b8edd47

File tree

5 files changed

+21
-20
lines changed

5 files changed

+21
-20
lines changed

src/main/java/aquality/appium/mobile/actions/ITouchActions.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ public interface ITouchActions {
1616
void swipe(Point startPoint, Point endPoint);
1717

1818
/**
19-
* Swipes from start point to end point using LongPress.
19+
* Performs long press action and moves cursor from a start point to an end end point imitating the swipe action
2020
*
2121
* @param startPoint point on screen to swipe from.
2222
* @param endPoint point on screen to swipe to.

src/main/java/aquality/appium/mobile/actions/TouchActions.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
package aquality.appium.mobile.actions;
22

33
import aquality.appium.mobile.application.AqualityServices;
4-
import aquality.appium.mobile.configuration.ISwipeConfiguration;
4+
import aquality.appium.mobile.configuration.ITouchActionsConfiguration;
55
import aquality.selenium.core.utilities.IElementActionRetrier;
66
import io.appium.java_client.TouchAction;
77
import io.appium.java_client.touch.offset.PointOption;
@@ -19,8 +19,10 @@ public void swipe(Point startPoint, Point endPoint) {
1919
startPoint.getY(),
2020
endPoint.getX(),
2121
endPoint.getY());
22-
performTouchAction(touchAction -> touchAction.press(PointOption.point(startPoint))
23-
.waitAction(waitOptions(AqualityServices.get(ISwipeConfiguration.class).getTimeout())), endPoint);
22+
performTouchAction(touchAction -> touchAction
23+
.press(PointOption.point(startPoint))
24+
.waitAction(waitOptions(AqualityServices.get(ITouchActionsConfiguration.class).getSwipeTimeout())),
25+
endPoint);
2426
}
2527

2628
@Override

src/main/java/aquality/appium/mobile/elements/actions/ElementTouchActions.java

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
import aquality.appium.mobile.actions.ITouchActions;
44
import aquality.appium.mobile.actions.SwipeDirection;
55
import aquality.appium.mobile.application.AqualityServices;
6-
import aquality.appium.mobile.configuration.ISwipeConfiguration;
6+
import aquality.appium.mobile.configuration.ITouchActionsConfiguration;
77
import aquality.appium.mobile.elements.interfaces.IElement;
88
import org.openqa.selenium.Dimension;
99
import org.openqa.selenium.Point;
@@ -23,20 +23,20 @@ public ElementTouchActions(IElement element) {
2323
this.element = element;
2424
this.scrollDownStartPoint = recalculatePointCoordinates(
2525
getBottomRightCornerPoint(),
26-
getISwipeConfiguration().getHorizontalSwipeBottomPointXCoefficient(),
27-
getISwipeConfiguration().getHorizontalSwipeBottomPointYCoefficient());
26+
getITouchActionsConfiguration().getHorizontalSwipeBottomPointXCoefficient(),
27+
getITouchActionsConfiguration().getHorizontalSwipeBottomPointYCoefficient());
2828
this.scrollDownEndPoint = recalculatePointCoordinates(
2929
getBottomRightCornerPoint(),
30-
getISwipeConfiguration().getHorizontalSwipeTopPointXCoefficient(),
31-
getISwipeConfiguration().getHorizontalSwipeTopPointYCoefficient());
30+
getITouchActionsConfiguration().getHorizontalSwipeTopPointXCoefficient(),
31+
getITouchActionsConfiguration().getHorizontalSwipeTopPointYCoefficient());
3232
this.swipeLeftStartPoint = recalculatePointCoordinates(
3333
getBottomRightCornerPoint(),
34-
getISwipeConfiguration().getVerticalSwipeRightPointXCoefficient(),
35-
getISwipeConfiguration().getVerticalSwipeRightPointYCoefficient());
34+
getITouchActionsConfiguration().getVerticalSwipeRightPointXCoefficient(),
35+
getITouchActionsConfiguration().getVerticalSwipeRightPointYCoefficient());
3636
this.swipeLeftEndPoint = recalculatePointCoordinates(
3737
getBottomRightCornerPoint(),
38-
getISwipeConfiguration().getVerticalSwipeLeftPointXCoefficient(),
39-
getISwipeConfiguration().getVerticalSwipeLeftPointYCoefficient());
38+
getITouchActionsConfiguration().getVerticalSwipeLeftPointXCoefficient(),
39+
getITouchActionsConfiguration().getVerticalSwipeLeftPointYCoefficient());
4040
this.scrollUpStartPoint = this.scrollDownEndPoint;
4141
this.scrollUpEndPoint = this.scrollDownStartPoint;
4242
this.swipeRightStartPoint = this.swipeLeftEndPoint;
@@ -55,7 +55,7 @@ public void swipeWithLongPress(Point endPoint) {
5555

5656
@Override
5757
public void scrollToElement(SwipeDirection direction) {
58-
int numberOfRetries = AqualityServices.get(ISwipeConfiguration.class).getRetries();
58+
int numberOfRetries = AqualityServices.get(ITouchActionsConfiguration.class).getSwipeRetries();
5959
ITouchActions touchActions = AqualityServices.getTouchActions();
6060
while (numberOfRetries > 0) {
6161
if (!element.state().isDisplayed()) {
@@ -104,11 +104,11 @@ private Point recalculatePointCoordinates(Point point, double coefficientX, doub
104104
}
105105

106106
/**
107-
* Returns ISwipeConfiguration class.
107+
* Returns ITouchActionsConfiguration class.
108108
*
109-
* @return ISwipeConfiguration class
109+
* @return ITouchActionsConfiguration class
110110
*/
111-
private ISwipeConfiguration getISwipeConfiguration() {
112-
return AqualityServices.get(ISwipeConfiguration.class);
111+
private ITouchActionsConfiguration getITouchActionsConfiguration() {
112+
return AqualityServices.get(ITouchActionsConfiguration.class);
113113
}
114114
}

src/test/java/samples/android/nativeapp/AndroidBasicInteractionsTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ public void testVerticalSwipeToElement() {
101101
@Test
102102
public void testHorizontalSwipeToElement() {
103103
ViewTabsScrollableScreen viewTabsScrollableScreen = ApplicationActivity.VIEW_TABS_SCROLLABLE.open();
104-
Assert.assertTrue(viewTabsScrollableScreen.isDisplayed(),
104+
Assert.assertTrue(viewTabsScrollableScreen.state().isDisplayed(),
105105
String.format("%s screen should be opened", viewTabsScrollableScreen.getName()));
106106
viewTabsScrollableScreen.swipeTab(4, 1);
107107
viewTabsScrollableScreen.selectTab(7);

src/test/java/samples/android/nativeapp/apidemos/screens/ViewControlsScreen.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
package samples.android.nativeapp.apidemos.screens;
22

33
import aquality.appium.mobile.actions.SwipeDirection;
4-
import aquality.appium.mobile.elements.actions.ElementTouchActions;
54
import aquality.appium.mobile.elements.interfaces.IButton;
65
import aquality.appium.mobile.elements.interfaces.ICheckBox;
76
import aquality.appium.mobile.elements.interfaces.ILabel;

0 commit comments

Comments
 (0)