Обновление скролла для элемента. Совмещение условных операторов в цикле while с if. Удаление лишней стркои с декрементированием, т.к. можно делать аналогичное постдекрементирование непосредственно в цикле. (#24)

Исправлен баг, когда скролл продолжал выполнение не смотря на то, что элемент уже видимо. Достигнуто путем совмещения условий в while. Аналогично добавлению break в блоке else, если бы его добавляли

Co-authored-by: artem <[email protected]>

Author: shketok

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

@@ -57,27 +57,24 @@ public void swipeWithLongPress(Point endPoint) {
     public void scrollToElement(SwipeDirection direction) {
         int numberOfRetries = AqualityServices.get(ITouchActionsConfiguration.class).getSwipeRetries();
         ITouchActions touchActions = AqualityServices.getTouchActions();
-        while (numberOfRetries > 0) {
-            if (!element.state().isDisplayed()) {
-                switch (direction) {
-                    case DOWN:
-                        touchActions.swipe(scrollDownStartPoint, scrollDownEndPoint);
-                        break;
-                    case UP:
-                        touchActions.swipe(scrollUpStartPoint, scrollUpEndPoint);
-                        break;
-                    case LEFT:
-                        touchActions.swipe(swipeLeftStartPoint, swipeLeftEndPoint);
-                        break;
-                    case RIGHT:
-                        touchActions.swipe(swipeRightStartPoint, swipeRightEndPoint);
-                        break;
-                    default:
-                        throw new IllegalArgumentException(
-                                String.format("'%s' direction does not exist", direction.toString()));
-                }
+        while (numberOfRetries-- > 0 && !element.state().isDisplayed()) {
+            switch (direction) {
+                case DOWN:
+                    touchActions.swipe(scrollDownStartPoint, scrollDownEndPoint);
+                    break;
+                case UP:
+                    touchActions.swipe(scrollUpStartPoint, scrollUpEndPoint);
+                    break;
+                case LEFT:
+                    touchActions.swipe(swipeLeftStartPoint, swipeLeftEndPoint);
+                    break;
+                case RIGHT:
+                    touchActions.swipe(swipeRightStartPoint, swipeRightEndPoint);
+                    break;
+                default:
+                    throw new IllegalArgumentException(
+                            String.format("'%s' direction does not exist", direction.toString()));
             }
-            numberOfRetries--;
         }
     }