KasperskyLab#249. Reverted the previous behavior of ObjectAutoScrollP…

…roviderImpl
PaladiRka · Nov 14, 2021 · 44f3bbd · 44f3bbd
1 parent c8aa36b
commit 44f3bbd
Show file tree

Hide file tree

Showing 4 changed files with 36 additions and 16 deletions.
diff --git a/kaspresso/src/main/kotlin/com/kaspersky/kaspresso/autoscroll/ObjectAutoScrollProviderImpl.kt b/kaspresso/src/main/kotlin/com/kaspersky/kaspresso/autoscroll/ObjectAutoScrollProviderImpl.kt
@@ -47,16 +47,36 @@ class ObjectAutoScrollProviderImpl(
      * @return the result of [action] invocation.
      */
     override fun <T> scroll(interaction: UiObjectInteraction, action: () -> T, cachedError: Throwable): T {
-        return try {
-            // Looks for a scrollable content
-            val scrollable = UiScrollable(UiSelector().scrollable(true))
-            // Scrolls to the bottom and looks for the given view. Invokes the action if the view was found.
-            scrollable.scrollForward()
-            logger.i("UiObject autoscroll successfully performed.")
-            action.invoke()
-        } catch (error: Throwable) {
-            logger.i("UiObject autoscroll did not help. Throwing exception.")
-            throw cachedError
-        }
+        /**
+         *  Looks for a scrollable content
+         */
+        val scrollable = UiScrollable(UiSelector().scrollable(true))
+
+        /**
+         * Scrolls to the bottom and looks for the given view. Invokes the action if the view was found.
+         */
+        do {
+            if (interaction.uiObject2 != null) {
+                logger.i("UiObject autoscroll to the bottom successfully performed.")
+                return action.invoke()
+            } else {
+                interaction.reCalculateUiObject()
+            }
+        } while (scrollable.scrollForward())
+
+        /**
+         * Scrolls to the beginning and looks for the given view. Invokes the action if the view was found.
+         */
+        do {
+            if (interaction.uiObject2 != null) {
+                logger.i("UiObject autoscroll to the beginning successfully performed.")
+                return action.invoke()
+            } else {
+                interaction.reCalculateUiObject()
+            }
+        } while (scrollable.scrollBackward())
+
+        logger.i("UiObject autoscroll did not help. Throwing exception.")
+        throw cachedError
     }
 }
diff --git a/...erceptors/behaviorkautomator/impl/elementloader/ElementLoaderObjectBehaviorInterceptor.kt b/...erceptors/behaviorkautomator/impl/elementloader/ElementLoaderObjectBehaviorInterceptor.kt
@@ -28,7 +28,7 @@ class ElementLoaderObjectBehaviorInterceptor(
         assertion: UiObjectAssertion,
         activity: () -> T
     ): T = passAction(
-        elementLoader = { interaction.reFindUiObject() },
+        elementLoader = { interaction.reCalculateUiObject() },
         action = activity
     )
 
@@ -43,7 +43,7 @@ class ElementLoaderObjectBehaviorInterceptor(
         action: UiObjectAction,
         activity: () -> T
     ): T = passAction(
-        elementLoader = { interaction.reFindUiObject() },
+        elementLoader = { interaction.reCalculateUiObject() },
         action = activity
     )
 }
diff --git a/...so/interceptors/behaviorkautomator/impl/flakysafety/FlakySafeObjectBehaviorInterceptor.kt b/...so/interceptors/behaviorkautomator/impl/flakysafety/FlakySafeObjectBehaviorInterceptor.kt
@@ -31,7 +31,7 @@ class FlakySafeObjectBehaviorInterceptor(
         assertion: UiObjectAssertion,
         activity: () -> T
     ): T = flakySafely(action = {
-        interaction.reFindUiObject()
+        interaction.reCalculateUiObject()
         activity.invoke()
     })
 
@@ -47,7 +47,7 @@ class FlakySafeObjectBehaviorInterceptor(
         action: UiObjectAction,
         activity: () -> T
     ): T = flakySafely(action = {
-        interaction.reFindUiObject()
+        interaction.reCalculateUiObject()
         activity.invoke()
     })
 }
diff --git a/...n/kotlin/com/kaspersky/components/kautomator/intercept/interaction/UiObjectInteraction.kt b/...n/kotlin/com/kaspersky/components/kautomator/intercept/interaction/UiObjectInteraction.kt
@@ -35,7 +35,7 @@ class UiObjectInteraction(
         action.execute(uiObject2 ?: throw UnfoundedUiObjectException(selector))
     }
 
-    fun reFindUiObject() {
+    fun reCalculateUiObject() {
         uiObject2 = calculateUiObject()
     }