appium-espresso-driver 2.13.1 → 2.13.3

package/espresso-server/app/src/androidTest/java/io/appium/espressoserver/lib/helpers/ComposeNodeFinder.kt

@@ -26,6 +26,7 @@ import io.appium.espressoserver.lib.handlers.exceptions.StaleElementException
 import io.appium.espressoserver.lib.model.Locator
 import io.appium.espressoserver.lib.model.SourceDocument
 import io.appium.espressoserver.lib.model.AttributesEnum
+import io.appium.espressoserver.lib.model.compileXpathExpression
 import io.appium.espressoserver.lib.viewmatcher.fetchIncludedAttributes
 
 /**
@@ -63,9 +64,12 @@ fun semanticsMatcherForLocator(locator: Locator): SemanticsMatcher =
     }
 
 private fun hasXpath(locator: Locator): SemanticsMatcher {
+    val xpath = locator.value!!
+    val expression = compileXpathExpression(xpath)
+    val attributes = fetchIncludedAttributes(xpath)
     val matchingIds = SourceDocument(
-        locator.elementId?.let { getSemanticsNode(it) }, fetchIncludedAttributes(locator.value!!)
-    ).matchingNodeIds(locator.value!!, AttributesEnum.RESOURCE_ID.toString())
+        locator.elementId?.let { getSemanticsNode(it) }, attributes
+    ).findMatchingNodeIds(expression, AttributesEnum.RESOURCE_ID.toString())
 
     return SemanticsMatcher("Matches Xpath ${locator.value}") {
         matchingIds.contains(it.id)

package/espresso-server/app/src/androidTest/java/io/appium/espressoserver/lib/helpers/ViewFinder.kt

@@ -168,8 +168,8 @@ object ViewFinder {
                 // If we're only looking for one item that matches xpath, pass it index 0 or else
                 // Espresso throws an AmbiguousMatcherException
                 return getViews(
-                    if (findOne) withXPath(parent, selector, 0) else withXPath(parent, selector),
-                    true,
+                    withXPath(parent, selector, if (findOne) 0 else null),
+                    findOne,
                     parent
                 ).map { ViewState(it) }
             Strategy.VIEW_TAG ->

package/espresso-server/app/src/androidTest/java/io/appium/espressoserver/lib/model/SourceDocument.kt

@@ -80,6 +80,12 @@ private fun toXmlNodeName(className: String?): String {
     return fixedName
 }
 
+fun compileXpathExpression(selector: String): XPathExpression =
+    try {
+        XPATH.compile(selector)
+    } catch (xe: XPathExpressionException) {
+        throw XPathLookupException(selector, xe.message!!)
+    }
 
 class SourceDocument constructor(
     private val root: Any? = null,
@@ -324,8 +330,8 @@ class SourceDocument constructor(
         throw AppiumException(lastError!!)
     }
 
-    private fun rootSemanticNodes(): List<SemanticsNode> {
-        return try {
+    private fun rootSemanticNodes(): List<SemanticsNode> =
+        try {
             listOf(EspressoServerRunnerTest.composeTestRule.onRoot(useUnmergedTree = true).fetchSemanticsNode())
         } catch (e: AssertionError) {
 //            Ideally there should be on `root` node but on some cases e.g:overlays screen, there can be more than 1 root.
@@ -337,7 +343,6 @@ class SourceDocument constructor(
             } as SelectionResult
             result.selectedNodes
         }
-    }
 
     private fun performCleanup() {
         tmpXmlName?.let {
@@ -346,8 +351,8 @@ class SourceDocument constructor(
         }
     }
 
-    fun toXMLString(): String {
-        return RESOURCES_GUARD.withPermit({
+    fun toXMLString(): String =
+        RESOURCES_GUARD.withPermit({
             toStream().use { xmlStream ->
                 val sb = StringBuilder()
                 val reader = BufferedReader(InputStreamReader(xmlStream, XML_ENCODING))
@@ -359,24 +364,17 @@ class SourceDocument constructor(
                 sb.toString()
             }
         }, { performCleanup() })
-    }
 
-    fun findViewsByXPath(xpathSelector: String): List<View> =
-        matchingNodeIds(xpathSelector, VIEW_INDEX).map { viewMap.get(it) }
+    fun findViewsByXPath(expression: XPathExpression): List<View> =
+        findMatchingNodeIds(expression, VIEW_INDEX).map { viewMap.get(it) }
 
-    fun matchingNodeIds(xpathSelector: String, attributeName: String): List<Int> {
-        val expr = try {
-            XPATH.compile(xpathSelector)
-        } catch (xe: XPathExpressionException) {
-            throw XPathLookupException(xpathSelector, xe.message!!)
-        }
-        return RESOURCES_GUARD.withPermit({
+    fun findMatchingNodeIds(expression: XPathExpression, attributeName: String): List<Int> =
+        RESOURCES_GUARD.withPermit({
             toStream().use { xmlStream ->
-                val list = expr.evaluate(InputSource(xmlStream), XPathConstants.NODESET) as NodeList
+                val list = expression.evaluate(InputSource(xmlStream), XPathConstants.NODESET) as NodeList
                 (0 until list.length).map { index ->
                     list.item(index).attributes.getNamedItem(attributeName).nodeValue.toInt()
                 }
             }
         }, { performCleanup() })
-    }
 }

package/espresso-server/app/src/androidTest/java/io/appium/espressoserver/lib/viewmatcher/WithXPath.kt

@@ -16,12 +16,15 @@
 package io.appium.espressoserver.lib.viewmatcher
 
 import android.view.View
+import io.appium.espressoserver.lib.helpers.extensions.withPermit
 import io.appium.espressoserver.lib.model.SourceDocument
 import io.appium.espressoserver.lib.model.AttributesEnum
 import io.appium.espressoserver.lib.model.EspressoAttributes
+import io.appium.espressoserver.lib.model.compileXpathExpression
 import org.hamcrest.Description
 import org.hamcrest.Matcher
 import org.hamcrest.TypeSafeMatcher
+import java.util.concurrent.Semaphore
 
 fun fetchIncludedAttributes(xpath: String): Set<AttributesEnum>? {
     if (xpath.contains("@*")) {
@@ -37,10 +40,26 @@ fun fetchIncludedAttributes(xpath: String): Set<AttributesEnum>? {
 }
 
 fun withXPath(root: View?, xpath: String, index: Int? = null): Matcher<View> {
-    // Get a list of the Views that match the provided xpath
-    val matchedXPathViews = SourceDocument(root, fetchIncludedAttributes(xpath)).findViewsByXPath(xpath)
+    val expression = compileXpathExpression(xpath)
+    val attributes = fetchIncludedAttributes(xpath)
+    val matchedXPathViews = mutableListOf<View>()
+    var didLookup = false
+    val lookupGuard = Semaphore(1)
     return object : TypeSafeMatcher<View>() {
         override fun matchesSafely(item: View): Boolean {
+            lookupGuard.withPermit {
+                if (!didLookup) {
+                    matchedXPathViews.addAll(
+                        SourceDocument(root ?: item.rootView, attributes).findViewsByXPath(expression)
+                    )
+                    didLookup = true
+                }
+            }
+
+            if (matchedXPathViews.isEmpty()) {
+                return false
+            }
+
             return if (index != null) {
                 // If index is not null, match it with the xpath in the list at the provided index
                 index < matchedXPathViews.size && matchedXPathViews[index] == item

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "appium-espresso-driver",
-  "version": "2.13.1",
+  "version": "2.13.3",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "appium-espresso-driver",
-      "version": "2.13.1",
+      "version": "2.13.3",
       "license": "Apache-2.0",
       "dependencies": {
         "@babel/runtime": "^7.4.3",
@@ -304,7 +304,7 @@
       }
     },
     "node_modules/@babel/parser": {
-      "version": "7.19.6",
+      "version": "7.20.0",
       "extraneous": true,
       "license": "MIT",
       "bin": {
@@ -315,10 +315,10 @@
       }
     },
     "node_modules/@babel/runtime": {
-      "version": "7.19.4",
+      "version": "7.20.0",
       "license": "MIT",
       "dependencies": {
-        "regenerator-runtime": "^0.13.4"
+        "regenerator-runtime": "^0.13.10"
       },
       "engines": {
         "node": ">=6.9.0"
@@ -1000,7 +1000,7 @@
       }
     },
     "node_modules/@types/node": {
-      "version": "18.11.5",
+      "version": "18.11.7",
       "license": "MIT"
     },
     "node_modules/@types/normalize-package-data": {
@@ -1252,7 +1252,7 @@
       }
     },
     "node_modules/appium-android-driver": {
-      "version": "5.6.0",
+      "version": "5.7.0",
       "license": "Apache-2.0",
       "dependencies": {
         "@babel/runtime": "^7.0.0",
@@ -1316,7 +1316,7 @@
       }
     },
     "node_modules/appium-chromedriver": {
-      "version": "5.2.4",
+      "version": "5.2.5",
       "hasInstallScript": true,
       "license": "Apache-2.0",
       "dependencies": {
@@ -2755,7 +2755,7 @@
       }
     },
     "node_modules/io.appium.settings": {
-      "version": "4.2.1",
+      "version": "4.2.2",
       "license": "Apache-2.0",
       "engines": {
         "node": ">=14",
@@ -5383,13 +5383,13 @@
       }
     },
     "@babel/parser": {
-      "version": "7.19.6",
+      "version": "7.20.0",
       "extraneous": true
     },
     "@babel/runtime": {
-      "version": "7.19.4",
+      "version": "7.20.0",
       "requires": {
-        "regenerator-runtime": "^0.13.4"
+        "regenerator-runtime": "^0.13.10"
       }
     },
     "@colors/colors": {
@@ -5863,7 +5863,7 @@
       }
     },
     "@types/node": {
-      "version": "18.11.5"
+      "version": "18.11.7"
     },
     "@types/normalize-package-data": {
       "version": "2.4.1"
@@ -6039,7 +6039,7 @@
       }
     },
     "appium-android-driver": {
-      "version": "5.6.0",
+      "version": "5.7.0",
       "requires": {
         "@babel/runtime": "^7.0.0",
         "appium-adb": "^9.10.9",
@@ -6087,7 +6087,7 @@
       }
     },
     "appium-chromedriver": {
-      "version": "5.2.4",
+      "version": "5.2.5",
       "requires": {
         "@babel/runtime": "^7.0.0",
         "@xmldom/xmldom": "^0.x",
@@ -7035,7 +7035,7 @@
       }
     },
     "io.appium.settings": {
-      "version": "4.2.1"
+      "version": "4.2.2"
     },
     "ipaddr.js": {
       "version": "1.9.1",

package/package.json

@@ -7,7 +7,7 @@
     "automated testing",
     "android"
   ],
-  "version": "2.13.1",
+  "version": "2.13.3",
   "author": "Appium Contributors",
   "license": "Apache-2.0",
   "repository": {