appium-xcuitest-driver 7.24.1 → 7.24.2

package/lib/commands/context.js

@@ -35,7 +35,7 @@ const extensions = {
      */
     let ctxs = [{id: NATIVE_WIN, view: {}}];
     this.contexts = [NATIVE_WIN];
-    for (let view of webviews) {
+    for (const view of webviews) {
       ctxs.push({id: `${WEBVIEW_BASE}${view.id}`, view});
       this.contexts.push(view.id.toString());
     }
@@ -75,7 +75,7 @@ const extensions = {
         if (contextId) {
           this.log.info(`Picking webview '${contextId}' after ${timer.getDuration().asMilliSeconds.toFixed(0)}ms`);
           await this.setContext(contextId);
-          await this.remote.cancelPageLoad();
+          await (/** @type {RemoteDebugger} */ (this.remote)).cancelPageLoad();
           return true;
         }
 
@@ -112,13 +112,15 @@ const extensions = {
 
     if (!this.remote) {
       await this.connectToRemoteDebugger();
+      // @ts-ignore TS does not understand property mutation
       if (!this.remote.appIdKey) {
+        // @ts-ignore TS does not understand property mutation
         await this.remote.setConnectionKey();
       }
     }
     const getWebviewPages = async () => {
       try {
-        return await this.remote.selectApp(
+        return await (/** @type {RemoteDebugger} */ (this.remote)).selectApp(
           useUrl ? this.getCurrentUrl() : undefined,
           this.opts.webviewConnectRetries,
           this.opts.ignoreAboutBlankUrl,
@@ -141,7 +143,9 @@ const extensions = {
   async connectToRemoteDebugger() {
     this.remote = await this.getNewRemoteDebugger();
 
+    // @ts-ignore static is fine
     this.remote.on(RemoteDebugger.EVENT_PAGE_CHANGE, this.onPageChange.bind(this));
+    // @ts-ignore static is fine
     this.remote.on(RemoteDebugger.EVENT_FRAMES_DETACHED, () => {
       if (isNonEmptyArray(this.curWebFrames)) {
         const curWebFrames = this.curWebFrames;
@@ -333,7 +337,7 @@ const extensions = {
 
       if (needsPageLoad) {
         this.log.debug('Page load needed. Loading...');
-        await this.remote.pageLoad();
+        await this.remote.waitForDom();
       }
 
       this.log.debug('New page listing is same as old, doing nothing');
@@ -371,7 +375,7 @@ const helpers = {
    */
   async stopRemote(closeWindowBeforeDisconnecting = false) {
     if (!this.remote) {
-      this.log.errorAndThrow('Tried to leave a web frame but were not in one');
+      throw this.log.errorWithException('Tried to leave a web frame but were not in one');
     }
 
     if (closeWindowBeforeDisconnecting) {
@@ -435,6 +439,7 @@ const helpers = {
   },
   /**
    * @this {XCUITestDriver}
+   * @returns {Promise<RemoteDebugger>}
    */
   async getNewRemoteDebugger() {
     let socketPath;
@@ -547,7 +552,7 @@ const commands = {
     const [appIdKey, pageIdKey] = _.map(contextId.split('.'), (id) => parseInt(id, 10));
     try {
       this.selectingNewPage = true;
-      await this.remote.selectPage(appIdKey, pageIdKey, skipReadyCheck);
+      await (/** @type {RemoteDebugger} */ (this.remote)).selectPage(appIdKey, pageIdKey, skipReadyCheck);
     } catch (err) {
       this.curContext = this.curWindowHandle = oldContext;
       throw err;
@@ -568,12 +573,12 @@ const commands = {
     // start safari logging if the logs handlers are active
     if (name && name !== NATIVE_WIN && this.logs) {
       if (this.logs.safariConsole) {
-        await this.remote.startConsole(
+        (/** @type {RemoteDebugger} */ (this.remote)).startConsole(
           this.logs.safariConsole.onConsoleLogEvent.bind(this.logs.safariConsole),
         );
       }
       if (this.logs.safariNetwork) {
-        await this.remote.startNetwork(
+        (/** @type {RemoteDebugger} */ (this.remote)).startNetwork(
           this.logs.safariNetwork.onNetworkEvent.bind(this.logs.safariNetwork),
         );
       }

package/lib/commands/execute.js

@@ -135,7 +135,7 @@ export default {
 
     args = this.convertElementsForAtoms(args);
     this.asyncWaitMs = this.asyncWaitMs || 0;
-    const promise = this.remote.executeAtomAsync(
+    const promise = (/** @type {import('appium-remote-debugger').RemoteDebugger} */ (this.remote)).executeAtomAsync(
       'execute_async_script',
       [script, args, this.asyncWaitMs],
       this.curWebFrames,

package/lib/commands/general.js

@@ -162,7 +162,7 @@ const commands = {
       this.setCurrentUrl(url);
       // make sure to clear out any leftover web frames
       this.curWebFrames = [];
-      await this.remote.navToUrl(url);
+      await (/** @type {import('appium-remote-debugger').RemoteDebugger} */ (this.remote)).navToUrl(url);
       return;
     }
 

package/lib/commands/screenshots.js

@@ -14,8 +14,8 @@ export default {
       switch (_.toLower(webScreenshotMode)) {
         case 'page':
         case 'viewport':
-          return await this.remote.captureScreenshot({
-            coordinateSystem: _.capitalize(webScreenshotMode),
+          return await (/** @type {import('appium-remote-debugger').RemoteDebugger} */ (this.remote)).captureScreenshot({
+            coordinateSystem: /** @type {'Viewport'|'Page'} */ (_.capitalize(webScreenshotMode)),
           });
         case 'native':
         case undefined:
@@ -94,7 +94,7 @@ export default {
    */
   async getViewportScreenshot() {
     if (this.isWebContext()) {
-      return await this.remote.captureScreenshot();
+      return await (/** @type {import('appium-remote-debugger').RemoteDebugger} */ (this.remote)).captureScreenshot();
     }
 
     let statusBarHeight = await this.getStatusBarHeight();

package/lib/commands/web.js

@@ -188,7 +188,7 @@ const commands = {
       throw new errors.NotImplementedError();
     }
 
-    await this.remote.execute('window.location.reload()');
+    await (/** @type {RemoteDebugger} */ (this.remote)).execute('window.location.reload()');
   },
   /**
    * @this {XCUITestDriver}
@@ -199,7 +199,7 @@ const commands = {
       throw new errors.NotImplementedError();
     }
 
-    return await this.remote.execute('window.location.href');
+    return await (/** @type {RemoteDebugger} */ (this.remote)).execute('window.location.href');
   },
   /**
    * @this {XCUITestDriver}
@@ -210,7 +210,7 @@ const commands = {
       throw new errors.NotImplementedError();
     }
 
-    return await this.remote.execute('window.document.title');
+    return await (/** @type {RemoteDebugger} */ (this.remote)).execute('window.document.title');
   },
   /**
    * @this {XCUITestDriver}
@@ -222,7 +222,7 @@ const commands = {
     }
 
     // get the cookies from the remote debugger, or an empty object
-    const {cookies} = await this.remote.getCookies();
+    const {cookies} = await (/** @type {RemoteDebugger} */ (this.remote)).getCookies();
 
     // the value is URI encoded, so decode it safely
     return cookies.map((cookie) => {
@@ -306,7 +306,7 @@ const helpers = {
    */
   async _deleteCookie(cookie) {
     const url = `http${cookie.secure ? 's' : ''}://${cookie.domain}${cookie.path}`;
-    return await this.remote.deleteCookie(cookie.name, url);
+    return await (/** @type {RemoteDebugger} */ (this.remote)).deleteCookie(cookie.name, url);
   },
   /**
    * @this {XCUITestDriver}
@@ -351,18 +351,20 @@ const helpers = {
    */
   async executeAtom(atom, args, alwaysDefaultFrame = false) {
     let frames = alwaysDefaultFrame === true ? [] : this.curWebFrames;
-    let promise = this.remote.executeAtom(atom, args, frames);
+    let promise = (/** @type {RemoteDebugger} */ (this.remote)).executeAtom(atom, args, frames);
     return await this.waitForAtom(promise);
   },
   /**
    * @this {XCUITestDriver}
+   * @param {string} atom
+   * @param {any[]} args
    */
-  async executeAtomAsync(atom, args, responseUrl) {
+  async executeAtomAsync(atom, args) {
     // save the resolve and reject methods of the promise to be waited for
     let promise = new B((resolve, reject) => {
       this.asyncPromise = {resolve, reject};
     });
-    await this.remote.executeAtomAsync(atom, args, this.curWebFrames, responseUrl);
+    await (/** @type {RemoteDebugger} */ (this.remote)).executeAtomAsync(atom, args, this.curWebFrames);
     return await this.waitForAtom(promise);
   },
   /**
@@ -673,7 +675,7 @@ const extensions = {
       const { offsetX, offsetY, pixelRatioX, pixelRatioY } = this.webviewCalibrationResult;
       const cmd = '(function () {return {innerWidth: window.innerWidth, innerHeight: window.innerHeight, ' +
         'outerWidth: window.outerWidth, outerHeight: window.outerHeight}; })()';
-      const wvDims = await this.remote.execute(cmd);
+      const wvDims = await (/** @type {RemoteDebugger} */ (this.remote)).execute(cmd);
       // https://tripleodeon.com/2011/12/first-understand-your-screen/
       const shouldApplyPixelRatio = wvDims.innerWidth > wvDims.outerWidth
         || wvDims.innerHeight > wvDims.outerHeight;
@@ -713,7 +715,7 @@ const extensions = {
     const realDims = {w: rect.width, h: rect.height};
 
     const cmd = '(function () { return {w: window.innerWidth, h: window.innerHeight}; })()';
-    const wvDims = await this.remote.execute(cmd);
+    const wvDims = await (/** @type {RemoteDebugger} */ (this.remote)).execute(cmd);
 
     // keep track of implicit wait, and set locally to 0
     // https://github.com/appium/appium/issues/14988
@@ -854,11 +856,11 @@ const extensions = {
    * @this {XCUITestDriver}
    */
   async mobileWebNav(navType) {
-    this.remote.allowNavigationWithoutReload = true;
+    (/** @type {RemoteDebugger} */ (this.remote)).allowNavigationWithoutReload = true;
     try {
       await this.executeAtom('execute_script', [`history.${navType}();`, null]);
     } finally {
-      this.remote.allowNavigationWithoutReload = false;
+      (/** @type {RemoteDebugger} */ (this.remote)).allowNavigationWithoutReload = false;
     }
   },
 
@@ -1006,3 +1008,7 @@ export default {...helpers, ...extensions, ...commands};
  * @template {string} [S=string]
  * @typedef {import('@appium/types').Element<S>} Element
  */
+
+/**
+ * @typedef {import('appium-remote-debugger').RemoteDebugger} RemoteDebugger
+ */

package/lib/driver.js

@@ -262,6 +262,9 @@ export class XCUITestDriver extends BaseDriver {
   /** @type {WebDriverAgent} */
   wda;
 
+  /** @type {import('appium-remote-debugger').RemoteDebugger|null} */
+  remote;
+
   /**
    *
    * @param {XCUITestDriverOpts} opts
@@ -310,6 +313,7 @@ export class XCUITestDriver extends BaseDriver {
     this.lifecycleData = {};
     this._audioRecorder = null;
     this.appInfosCache = new AppInfosCache(this.log);
+    this.remote = null;
   }
 
   async onSettingsUpdate(key, value) {

package/npm-shrinkwrap.json

@@ -1,19 +1,19 @@
 {
   "name": "appium-xcuitest-driver",
-  "version": "7.24.1",
+  "version": "7.24.2",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "appium-xcuitest-driver",
-      "version": "7.24.1",
+      "version": "7.24.2",
       "license": "Apache-2.0",
       "dependencies": {
         "@colors/colors": "^1.6.0",
         "appium-idb": "^1.6.13",
         "appium-ios-device": "^2.5.4",
         "appium-ios-simulator": "^6.1.7",
-        "appium-remote-debugger": "^11.3.0",
+        "appium-remote-debugger": "^11.5.3",
         "appium-webdriveragent": "^8.7.8",
         "appium-xcode": "^5.1.4",
         "async-lock": "^1.4.0",
@@ -192,6 +192,18 @@
         "url": "https://github.com/sponsors/sindresorhus"
       }
     },
+    "node_modules/@appium/docutils/node_modules/typescript": {
+      "version": "5.5.3",
+      "resolved": "https://registry.npmjs.org/typescript/-/typescript-5.5.3.tgz",
+      "integrity": "sha512-/hreyEujaB0w76zKo6717l3L0o/qEUtRgdvUBvlkhoWeOVMjMuHNHk0BRBzikzuGDqNmPQbg5ifMEqsHLiIUcQ==",
+      "bin": {
+        "tsc": "bin/tsc",
+        "tsserver": "bin/tsserver"
+      },
+      "engines": {
+        "node": ">=14.17"
+      }
+    },
     "node_modules/@appium/logger": {
       "version": "1.6.0",
       "resolved": "https://registry.npmjs.org/@appium/logger/-/logger-1.6.0.tgz",
@@ -741,9 +753,9 @@
       }
     },
     "node_modules/@types/node": {
-      "version": "20.14.11",
-      "resolved": "https://registry.npmjs.org/@types/node/-/node-20.14.11.tgz",
-      "integrity": "sha512-kprQpL8MMeszbz6ojB5/tU8PLN4kesnN8Gjzw349rDlNgsSzg90lAVj3llK99Dh7JON+t9AuscPPFW6mPbTnSA==",
+      "version": "20.14.12",
+      "resolved": "https://registry.npmjs.org/@types/node/-/node-20.14.12.tgz",
+      "integrity": "sha512-r7wNXakLeSsGT0H1AU863vS2wa5wBOK4bWMjZz2wj+8nBx+m5PeIn0k8AloSLpRuiwdRQZwarZqHE4FNArPuJQ==",
       "dependencies": {
         "undici-types": "~5.26.4"
       }
@@ -958,9 +970,9 @@
       }
     },
     "node_modules/appium-remote-debugger": {
-      "version": "11.3.2",
-      "resolved": "https://registry.npmjs.org/appium-remote-debugger/-/appium-remote-debugger-11.3.2.tgz",
-      "integrity": "sha512-usVaIU6KHUtc6CJDObctiBiAWjykG1fM0VfoTToHa5cCANVxC1slIU7OWH2EXOqNzV4+3yl3rfHY7EObY3nIDQ==",
+      "version": "11.5.3",
+      "resolved": "https://registry.npmjs.org/appium-remote-debugger/-/appium-remote-debugger-11.5.3.tgz",
+      "integrity": "sha512-O0A5gxXQBgl+KvdHhVgjTiNm2ns1pHRbk0Op4RCAAq2Op7a32IPGjULQSaI6fa7Um26PBBy3HmglPeLi265H3A==",
       "dependencies": {
         "@appium/base-driver": "^9.0.0",
         "@appium/support": "^5.0.3",
@@ -980,9 +992,9 @@
       }
     },
     "node_modules/appium-webdriveragent": {
-      "version": "8.7.8",
-      "resolved": "https://registry.npmjs.org/appium-webdriveragent/-/appium-webdriveragent-8.7.8.tgz",
-      "integrity": "sha512-OPyyspJD1VIhYUzjcZsXV7LiTfwwjF8oD9mDcuak6zl6NIGpA/dCwzx8XKE1NQtdy+alGqOx6N0n+IUUWjf4Fg==",
+      "version": "8.7.9",
+      "resolved": "https://registry.npmjs.org/appium-webdriveragent/-/appium-webdriveragent-8.7.9.tgz",
+      "integrity": "sha512-SvGZpM126rfUpIoWEZciDQ53Ek2WsdNtt0cJZtBDW1Cv4Zws4gqjrJd3BgB6uZj9kjhS4OGXYSzXSvEcCzb2cw==",
       "dependencies": {
         "@appium/base-driver": "^9.0.0",
         "@appium/strongbox": "^0.x",
@@ -3854,9 +3866,9 @@
       }
     },
     "node_modules/typescript": {
-      "version": "5.5.3",
-      "resolved": "https://registry.npmjs.org/typescript/-/typescript-5.5.3.tgz",
-      "integrity": "sha512-/hreyEujaB0w76zKo6717l3L0o/qEUtRgdvUBvlkhoWeOVMjMuHNHk0BRBzikzuGDqNmPQbg5ifMEqsHLiIUcQ==",
+      "version": "5.5.4",
+      "resolved": "https://registry.npmjs.org/typescript/-/typescript-5.5.4.tgz",
+      "integrity": "sha512-Mtq29sKDAEYP7aljRgtPOpTvOfbwRWlS6dPRzwjdE+C0R4brX/GUyhHSecbHMFLNBLcJIPt9nl9yG5TZ1weH+Q==",
       "bin": {
         "tsc": "bin/tsc",
         "tsserver": "bin/tsserver"

package/package.json

@@ -8,7 +8,7 @@
     "xcuitest",
     "xctest"
   ],
-  "version": "7.24.1",
+  "version": "7.24.2",
   "author": "Appium Contributors",
   "license": "Apache-2.0",
   "repository": {
@@ -80,7 +80,7 @@
     "appium-idb": "^1.6.13",
     "appium-ios-device": "^2.5.4",
     "appium-ios-simulator": "^6.1.7",
-    "appium-remote-debugger": "^11.3.0",
+    "appium-remote-debugger": "^11.5.3",
     "appium-webdriveragent": "^8.7.8",
     "appium-xcode": "^5.1.4",
     "async-lock": "^1.4.0",