fix: appium tests

Author: kobenguyent

.github/workflows/appium_Android.yml

@@ -4,6 +4,7 @@ on:
   push:
     branches:
       - 3.x
+      - 5228-fix-appium-tests
 
 env:
   CI: true

lib/helper/Appium.js

@@ -1523,7 +1523,39 @@ class Appium extends Webdriver {
    */
   async dontSeeElement(locator) {
     if (this.isWeb) return super.dontSeeElement(locator)
-    return super.dontSeeElement(parseLocator.call(this, locator))
+    
+    // For mobile native apps, handle isDisplayed error gracefully
+    const parsedLocator = parseLocator.call(this, locator)
+    const res = await this._locate(parsedLocator, false)
+    const { truth } = require('../assert/truth')
+    const Locator = require('../locator')
+    
+    if (!res || res.length === 0) {
+      return truth(`elements of ${new Locator(parsedLocator)}`, 'to be seen').negate(false)
+    }
+    
+    const selected = []
+    for (const el of res) {
+      try {
+        const displayed = await el.isDisplayed()
+        if (displayed) selected.push(true)
+      } catch (err) {
+        // If isDisplayed fails due to execute/sync not being supported,
+        // fall back to checking if element exists (which we already verified)
+        if (err.message && err.message.includes('Method is not implemented')) {
+          // Element exists, assume it's displayed
+          selected.push(true)
+        } else {
+          throw err
+        }
+      }
+    }
+    
+    try {
+      return truth(`elements of ${new Locator(parsedLocator)}`, 'to be seen').negate(selected)
+    } catch (err) {
+      throw err
+    }
   }
 
   /**
@@ -1656,7 +1688,43 @@ class Appium extends Webdriver {
    */
   async seeElement(locator) {
     if (this.isWeb) return super.seeElement(locator)
-    return super.seeElement(parseLocator.call(this, locator))
+    
+    // For mobile native apps, we need to handle isDisplayed differently
+    // because webdriverio's isDisplayed() tries to execute JavaScript which isn't supported
+    const parsedLocator = parseLocator.call(this, locator)
+    const res = await this._locate(parsedLocator, true)
+    const ElementNotFound = require('./errors/ElementNotFound')
+    const { truth } = require('../assert/truth')
+    const { dontSeeElementError } = require('./errors/ElementAssertion')
+    const Locator = require('../locator')
+    
+    if (!res || res.length === 0) {
+      throw new ElementNotFound(parsedLocator)
+    }
+    
+    // Use a safer approach for mobile: check if element exists and try isDisplayed with error handling
+    const selected = []
+    for (const el of res) {
+      try {
+        const displayed = await el.isDisplayed()
+        if (displayed) selected.push(true)
+      } catch (err) {
+        // If isDisplayed fails due to execute/sync not being supported,
+        // fall back to checking if element exists (which we already verified)
+        if (err.message && err.message.includes('Method is not implemented')) {
+          // Element exists, assume it's displayed since we found it
+          selected.push(true)
+        } else {
+          throw err
+        }
+      }
+    }
+    
+    try {
+      return truth(`elements of ${new Locator(parsedLocator)}`, 'to be seen').assert(selected)
+    } catch (e) {
+      dontSeeElementError(parsedLocator)
+    }
   }
 
   /**

test/helper/Appium_test.js

@@ -31,6 +31,7 @@ describe('Appium', function () {
         platformName: 'Android',
         platformVersion: '7.0',
         deviceName: 'Android GoogleAPI Emulator',
+        automationName: 'UiAutomator2',
         androidInstallTimeout: 90000,
         appWaitDuration: 300000,
       },