appium-remote-debugger 15.0.0 → 15.0.1

package/lib/mixins/connect.js

@@ -173,11 +173,11 @@ export async function selectPage (appIdKey, pageIdKey, skipReadyCheck = false) {
 
   const timer = new timing.Timer().start();
 
-  await this.requireRpcClient().selectPage(fullAppIdKey, pageIdKey);
-
-  if (!skipReadyCheck) {
-    await this.waitForDom();
-  }
+  const pageReadinessDetector = skipReadyCheck ? undefined : {
+    timeoutMs: this.pageLoadMs,
+    readinessDetector: (/** @type {string} */ readyState) => this.isPageLoadingCompleted(readyState),
+  };
+  await this.requireRpcClient().selectPage(fullAppIdKey, pageIdKey, pageReadinessDetector);
 
   this.log.debug(`Selected page after ${timer.getDuration().asMilliSeconds.toFixed(0)}ms`);
 }

package/lib/mixins/execute.js

@@ -2,7 +2,7 @@ import { errors } from '@appium/base-driver';
 import {
   checkParams,
   simpleStringify,
-  convertResult,
+  convertJavascriptEvaluationResult,
   RESPONSE_LOG_LENGTH,
 } from '../utils';
 import { getScriptForAtom } from '../atoms';
@@ -123,7 +123,7 @@ export async function executeAtomAsync (atom, args = [], frames = []) {
       );
     } catch {}
   }
-  return convertResult(res);
+  return convertJavascriptEvaluationResult(res);
 }
 
 /**
@@ -155,7 +155,7 @@ export async function execute (command, override) {
     appIdKey,
     pageIdKey,
   });
-  return convertResult(res);
+  return convertJavascriptEvaluationResult(res);
 }
 
 /**
@@ -184,7 +184,7 @@ export async function callFunction (objectId, fn, args) {
     pageIdKey,
   });
 
-  return convertResult(res);
+  return convertJavascriptEvaluationResult(res);
 }
 
 /**

package/lib/mixins/navigate.js

@@ -20,11 +20,11 @@ const PAGE_READINESS_CHECK_INTERVAL_MS = 50;
 /**
  * pageLoadStrategy in WebDriver definitions.
  */
-const PAGE_LOAD_STRATEGY = {
+const PAGE_LOAD_STRATEGY = Object.freeze({
   EAGER: 'eager',
   NONE: 'none',
   NORMAL: 'normal'
-};
+});
 
 /**
  * @this {RemoteDebugger}
@@ -53,18 +53,17 @@ export function cancelPageLoad () {
  * @returns {boolean}
  */
 export function isPageLoadingCompleted (readyState) {
-  const _pageLoadStrategy = _.toLower(getPageLoadStartegy(this));
-  if (_pageLoadStrategy === PAGE_LOAD_STRATEGY.NONE) {
-    return true;
+  const pageLoadStrategy = _.toLower(getPageLoadStartegy(this));
+  switch (pageLoadStrategy) {
+    case PAGE_LOAD_STRATEGY.EAGER:
+      // This could include 'interactive' or 'complete'
+      return readyState !== 'loading';
+    case PAGE_LOAD_STRATEGY.NONE:
+      return true;
+    case PAGE_LOAD_STRATEGY.NORMAL:
+    default:
+      return readyState === 'complete';
   }
-
-  if (_pageLoadStrategy === PAGE_LOAD_STRATEGY.EAGER) {
-    // This could include 'interactive' or 'complete'
-    return readyState !== 'loading';
-  }
-
-  // Default behavior. It includes pageLoadStrategy is 'normal' as well.
-  return readyState === 'complete';
 }
 
 /**

package/lib/rpc/rpc-client.js

@@ -7,6 +7,7 @@ import RpcMessageHandler from './rpc-message-handler';
 import { util, timing } from '@appium/support';
 import { EventEmitter } from 'node:events';
 import AsyncLock from 'async-lock';
+import { convertJavascriptEvaluationResult } from '../utils';
 
 const DATA_LOG_LENGTH = {length: 200};
 const MIN_WAIT_FOR_TARGET_TIMEOUT_MS = 30000;
@@ -83,7 +84,7 @@ export class RpcClient {
   /** @type {EventEmitter} */
   _targetSubscriptions;
 
-  /** @type {[import('../types').AppIdKey, import('../types').PageIdKey] | undefined} */
+  /** @type {PendingTargetNotification | undefined} */
   _pendingTargetNotification;
 
   /**
@@ -454,7 +455,7 @@ export class RpcClient {
       return;
     }
 
-    const [appIdKey, pageIdKey] = this._pendingTargetNotification;
+    const [appIdKey, pageIdKey, pageReadinessDetector] = this._pendingTargetNotification;
     if (appIdKey !== app) {
       log.info(
         `Received 'Target.targetCreated' event for app '${app}' with no pending request: ${JSON.stringify(targetInfo)}`
@@ -470,6 +471,24 @@ export class RpcClient {
     }
     const timer = new timing.Timer().start();
 
+    const adjustPageReadinessDetector = () => {
+      if (!pageReadinessDetector) {
+        return;
+      }
+
+      const elapsedMs = timer.getDuration().asMilliSeconds;
+      if (elapsedMs >= pageReadinessDetector.timeoutMs) {
+        log.warn(
+          `Page '${pageIdKey}' took too long to initialize, skipping readiness check`
+        );
+        return;
+      }
+      return {
+        timeoutMs: pageReadinessDetector.timeoutMs - elapsedMs,
+        readinessDetector: pageReadinessDetector.readinessDetector,
+      };
+    };
+
     if (targetInfo.isProvisional) {
       log.debug(
         `Provisional target created for app '${appIdKey}' and page '${pageIdKey}': '${JSON.stringify(targetInfo)}'`
@@ -478,8 +497,9 @@ export class RpcClient {
       this._provisionedPages.add(pageIdKey);
       try {
         await this.targets[appIdKey].lock.acquire(pageIdKey, async () => {
+          let wasInitialized = false;
           try {
-            await this._initializePage(appIdKey, pageIdKey, targetInfo.targetId);
+            wasInitialized = await this._initializePage(appIdKey, pageIdKey, targetInfo.targetId);
           } finally {
             if (targetInfo.isPaused) {
               await this._resumeTarget(appIdKey, pageIdKey, targetInfo.targetId);
@@ -489,6 +509,11 @@ export class RpcClient {
               );
             }
           }
+          if (wasInitialized) {
+            await this._waitForPageReadiness(
+              appIdKey, pageIdKey, targetInfo.targetId, adjustPageReadinessDetector()
+            );
+          }
         });
       } catch (e) {
         log.warn(
@@ -522,18 +547,24 @@ export class RpcClient {
 
     try {
       await this.targets[appIdKey].lock.acquire(pageIdKey, async () => {
+        let wasInitialized = false;
         try {
           if (this._provisionedPages.has(pageIdKey)) {
             log.debug(`Page '${pageIdKey}' has been already provisioned`);
             this._provisionedPages.delete(pageIdKey);
           } else {
-            await this._initializePage(appIdKey, pageIdKey);
+            wasInitialized = await this._initializePage(appIdKey, pageIdKey);
           }
         } finally {
           if (targetInfo.isPaused) {
             await this._resumeTarget(appIdKey, pageIdKey, targetInfo.targetId);
           }
         }
+        if (wasInitialized) {
+          await this._waitForPageReadiness(
+            appIdKey, pageIdKey, targetInfo.targetId, adjustPageReadinessDetector()
+          );
+        }
       });
     } catch (e) {
       log.warn(e.message);
@@ -633,10 +664,11 @@ export class RpcClient {
   /**
    * @param {import('../types').AppIdKey} appIdKey
    * @param {import('../types').PageIdKey} pageIdKey
+   * @param {PageReadinessDetector} [pageReadinessDetector]
    * @returns {Promise<void>}
    */
-  async selectPage (appIdKey, pageIdKey) {
-    this._pendingTargetNotification = [appIdKey, pageIdKey];
+  async selectPage (appIdKey, pageIdKey, pageReadinessDetector) {
+    this._pendingTargetNotification = [appIdKey, pageIdKey, pageReadinessDetector];
     this._provisionedPages.clear();
 
     // go through the steps that the Desktop Safari system
@@ -900,6 +932,56 @@ export class RpcClient {
     }
   }
 
+  /**
+   *
+   * @param {import('../types').AppIdKey} appIdKey
+   * @param {import('../types').PageIdKey} pageIdKey
+   * @param {import('../types').TargetId} targetId
+   * @param {PageReadinessDetector} [pageReadinessDetector]
+   * @returns {Promise<void>}
+   */
+  async _waitForPageReadiness(appIdKey, pageIdKey, targetId, pageReadinessDetector) {
+    if (!pageReadinessDetector) {
+      return;
+    }
+
+    log.debug(`Waiting up to ${pageReadinessDetector.timeoutMs}ms for page readiness`);
+    const timer = new timing.Timer().start();
+    while (pageReadinessDetector.timeoutMs - timer.getDuration().asMilliSeconds > 0) {
+      /** @type {string} */
+      let readyState;
+      try {
+        const commandTimeoutMs = Math.max(
+          100,
+          Math.trunc((pageReadinessDetector.timeoutMs - timer.getDuration().asMilliSeconds) * 0.8)
+        );
+        const rawResult = await B.resolve(this.send('Runtime.evaluate', {
+          expression: 'document.readyState;',
+          returnByValue: true,
+          appIdKey,
+          pageIdKey,
+          targetId,
+        })).timeout(commandTimeoutMs);
+        readyState = convertJavascriptEvaluationResult(rawResult);
+      } catch (e) {
+        log.debug(`Cannot determine page readiness: ${e.message}`);
+        continue;
+      }
+      if (pageReadinessDetector.readinessDetector(readyState)) {
+        log.info(
+          `Page '${pageIdKey}' for app '${appIdKey}' is ready after ` +
+          `${timer.getDuration().asMilliSeconds}ms`
+        );
+        return;
+      }
+      await B.delay(100);
+    }
+    log.warn(
+      `Page '${pageIdKey}' for app '${appIdKey}' is not ready after ` +
+      `${timer.getDuration().asMilliSeconds}ms. Continuing anyway`
+    );
+  }
+
   /**
    *
    * @param {import('../types').AppIdKey} appIdKey
@@ -943,4 +1025,15 @@ export default RpcClient;
 /**
  * @typedef {PageDict & {provisional?: import('../types').ProvisionalTargetInfo, lock: AsyncLock}} PagesToTargets
  * @typedef {{[key: import('../types').AppIdKey]: PagesToTargets}} AppToTargetsMap
- */
+ */
+
+/**
+ * @typedef {Object} PageReadinessDetector
+ * @property {number} timeoutMs
+ * @property {(readyState: string) => boolean} readinessDetector
+ */
+
+/**
+ * @typedef {[import('../types').AppIdKey, import('../types').PageIdKey, PageReadinessDetector | undefined]} PendingTargetNotification
+ */
+

package/lib/utils.js

@@ -157,7 +157,7 @@ export function deferredPromise () {
  * @param {any} res
  * @returns {any}
  */
-export function convertResult (res) {
+export function convertJavascriptEvaluationResult (res) {
   if (_.isUndefined(res)) {
     throw new Error(`Did not get OK result from remote debugger. Result was: ${_.truncate(simpleStringify(res), {length: RESPONSE_LOG_LENGTH})}`);
   } else if (_.isString(res)) {

package/package.json

@@ -4,7 +4,7 @@
   "keywords": [
     "appium"
   ],
-  "version": "15.0.0",
+  "version": "15.0.1",
   "author": "Appium Contributors",
   "license": "Apache-2.0",
   "repository": {