appium-xcuitest-driver 10.13.2 → 10.13.4

package/lib/commands/{recordscreen.js → recordscreen.ts}

@@ -5,6 +5,9 @@ import {encodeBase64OrUpload} from '../utils';
 import {WDA_BASE_URL} from 'appium-webdriveragent';
 import {waitForCondition} from 'asyncbox';
 import url from 'url';
+import type {XCUITestDriver} from '../driver';
+import type {StartRecordingScreenOptions, StopRecordingScreenOptions} from './types';
+import type {WDASettings} from 'appium-webdriveragent';
 
 /**
  * Set max timeout for 'reconnect_delay_max' ffmpeg argument usage.
@@ -20,14 +23,22 @@ const DEFAULT_VCODEC = 'mjpeg';
 const MP4_EXT = '.mp4';
 const FFMPEG_BINARY = 'ffmpeg';
 const ffmpegLogger = logger.getLogger(FFMPEG_BINARY);
-const QUALITY_MAPPING = {
+const QUALITY_MAPPING: Record<string, number> = {
   low: 10,
   medium: 25,
   high: 75,
   photo: 100,
 };
 
-const HARDWARE_ACCELERATION_PARAMETERS = {
+const HARDWARE_ACCELERATION_PARAMETERS: Record<
+  string,
+  {
+    hwaccel: string;
+    hwaccelOutputFormat: string;
+    scaleFilterHWAccel: string;
+    videoTypeHWAccel: string;
+  }
+> = {
   /* https://trac.ffmpeg.org/wiki/HWAccelIntro#VideoToolbox */
   videoToolbox: {
     hwaccel: 'videotoolbox',
@@ -54,21 +65,28 @@ const HARDWARE_ACCELERATION_PARAMETERS = {
     hwaccel: 'qsv',
     hwaccelOutputFormat: '',
     scaleFilterHWAccel: 'scale_qsv',
-    videoTypeHWAccel: 'h264_qsv'
+    videoTypeHWAccel: 'h264_qsv',
   },
   /* https://trac.ffmpeg.org/wiki/Hardware/VAAPI */
   vaapi: {
     hwaccel: 'vaapi',
     hwaccelOutputFormat: 'vaapi',
     scaleFilterHWAccel: 'scale_vaapi',
-    videoTypeHWAccel: 'h264_vaapi'
-  }
+    videoTypeHWAccel: 'h264_vaapi',
+  },
 };
 
 const CAPTURE_START_MARKER = /^\s*frame=/;
 
 export class ScreenRecorder {
-  constructor(udid, log, videoPath, opts = {}) {
+  private readonly videoPath: string;
+  private readonly log: any;
+  private readonly opts: ScreenRecorderOptions;
+  private readonly udid: string;
+  private mainProcess: SubProcess | null;
+  private timeoutHandler: NodeJS.Timeout | null;
+
+  constructor(udid: string, log: any, videoPath: string, opts: ScreenRecorderOptions = {}) {
     this.videoPath = videoPath;
     this.log = log;
     this.opts = opts;
@@ -77,7 +95,7 @@ export class ScreenRecorder {
     this.timeoutHandler = null;
   }
 
-  async start(timeoutMs) {
+  async start(timeoutMs: number): Promise<void> {
     try {
       await fs.which(FFMPEG_BINARY);
     } catch {
@@ -98,7 +116,7 @@ export class ScreenRecorder {
       pixelFormat,
     } = this.opts;
 
-    const args = [
+    const args: string[] = [
       '-f',
       'mjpeg',
       // https://github.com/appium/appium/issues/16294
@@ -115,8 +133,8 @@ export class ScreenRecorder {
       hwaccel,
       hwaccelOutputFormat,
       scaleFilterHWAccel,
-      videoTypeHWAccel
-    } = HARDWARE_ACCELERATION_PARAMETERS[hardwareAcceleration] ?? {};
+      videoTypeHWAccel,
+    } = HARDWARE_ACCELERATION_PARAMETERS[hardwareAcceleration || ''] ?? {};
 
     if (hwaccel) {
       args.push('-hwaccel', hwaccel);
@@ -127,10 +145,10 @@ export class ScreenRecorder {
     }
 
     //Parameter `-r` is optional. See details: https://github.com/appium/appium/issues/12067
-    if (videoFps && videoType === 'libx264' || videoTypeHWAccel) {
-      args.push('-r', videoFps);
+    if ((videoFps && videoType === 'libx264') || videoTypeHWAccel) {
+      args.push('-r', String(videoFps));
     }
-    const {protocol, hostname} = url.parse(remoteUrl);
+    const {protocol, hostname} = url.parse(remoteUrl || '');
     args.push('-i', `${protocol}//${hostname}:${remotePort}`);
 
     if (videoFilters || videoScale) {
@@ -141,13 +159,13 @@ export class ScreenRecorder {
     if (pixelFormat) {
       args.push('-pix_fmt', pixelFormat);
     }
-    args.push('-vcodec', videoTypeHWAccel || videoType);
+    args.push('-vcodec', videoTypeHWAccel || videoType || DEFAULT_VCODEC);
     args.push('-y');
     args.push(this.videoPath);
 
     this.mainProcess = new SubProcess(FFMPEG_BINARY, args);
     let isCaptureStarted = false;
-    this.mainProcess.on('line-stderr', (line) => {
+    this.mainProcess.on('line-stderr', (line: string) => {
       if (CAPTURE_START_MARKER.test(line)) {
         if (!isCaptureStarted) {
           isCaptureStarted = true;
@@ -175,9 +193,8 @@ export class ScreenRecorder {
       );
     }
     this.log.info(
-      `Starting screen capture on the device '${
-        this.udid
-      }' with command: '${FFMPEG_BINARY} ${args.join(' ')}'. ` + `Will timeout in ${timeoutMs}ms`,
+      `Starting screen capture on the device '${this.udid}' with command: '${FFMPEG_BINARY} ${args.join(' ')}'. ` +
+        `Will timeout in ${timeoutMs}ms`,
     );
 
     this.timeoutHandler = setTimeout(async () => {
@@ -189,7 +206,7 @@ export class ScreenRecorder {
     }, timeoutMs);
   }
 
-  async interrupt(force = false) {
+  async interrupt(force = false): Promise<boolean> {
     let result = true;
 
     if (this.timeoutHandler) {
@@ -202,7 +219,7 @@ export class ScreenRecorder {
       this.mainProcess = null;
       try {
         await interruptPromise;
-      } catch (e) {
+      } catch (e: any) {
         this.log.warn(
           `Cannot ${force ? 'terminate' : 'interrupt'} ${FFMPEG_BINARY}. ` +
             `Original error: ${e.message}`,
@@ -214,12 +231,12 @@ export class ScreenRecorder {
     return result;
   }
 
-  async finish() {
+  async finish(): Promise<string> {
     await this.interrupt();
     return this.videoPath;
   }
 
-  async cleanup() {
+  async cleanup(): Promise<void> {
     if (await fs.exists(this.videoPath)) {
       await fs.rimraf(this.videoPath);
     }
@@ -235,13 +252,15 @@ export class ScreenRecorder {
  * If screen recording has been already started then the command will stop it forcefully and start a new one.
  * The previously recorded video file will be deleted.
  *
- * @param {import('./types').StartRecordingScreenOptions} [options] - The available options.
- * @returns {Promise<string>} Base64-encoded content of the recorded media file if
+ * @param options - The available options.
+ * @returns Base64-encoded content of the recorded media file if
  *                   any screen recording is currently running or an empty string.
  * @throws {Error} If screen recording has failed to start.
- * @this {XCUITestDriver}
  */
-export async function startRecordingScreen(options = {}) {
+export async function startRecordingScreen(
+  this: XCUITestDriver,
+  options: StartRecordingScreenOptions = {},
+): Promise<string> {
   const {
     videoType = DEFAULT_VCODEC,
     timeLimit = DEFAULT_RECORDING_TIME_SEC,
@@ -251,21 +270,21 @@ export async function startRecordingScreen(options = {}) {
     videoScale,
     forceRestart,
     pixelFormat,
-    hardwareAcceleration
+    hardwareAcceleration,
   } = options;
 
   let result = '';
   if (!forceRestart) {
     this.log.info(
       `Checking if there is/was a previous screen recording. ` +
-      `Set 'forceRestart' option to 'true' if you'd like to skip this step.`
+        `Set 'forceRestart' option to 'true' if you'd like to skip this step.`,
     );
     result = (await this.stopRecordingScreen(options)) ?? result;
   }
 
   const videoPath = await tempDir.path({
     prefix: `appium_${Math.random().toString(16).substring(2, 8)}`,
-    suffix: MP4_EXT
+    suffix: MP4_EXT,
   });
 
   const wdaBaseUrl = this.opts.wdaBaseUrl || WDA_BASE_URL;
@@ -275,9 +294,9 @@ export async function startRecordingScreen(options = {}) {
     videoType,
     videoFilters,
     videoScale,
-    videoFps,
+    videoFps: typeof videoFps === 'string' ? parseInt(videoFps, 10) : videoFps,
     pixelFormat,
-    hardwareAcceleration
+    hardwareAcceleration,
   });
   if (!(await screenRecorder.interrupt(true))) {
     throw this.log.errorWithException('Unable to stop screen recording process');
@@ -295,10 +314,10 @@ export async function startRecordingScreen(options = {}) {
     );
   }
 
-  let {mjpegServerScreenshotQuality, mjpegServerFramerate} =
-    /** @type {import('appium-webdriveragent').WDASettings} */ (
-    await this.proxyCommand('/appium/settings', 'GET')
-  );
+  let {mjpegServerScreenshotQuality, mjpegServerFramerate} = (await this.proxyCommand(
+    '/appium/settings',
+    'GET',
+  )) as WDASettings;
   if (videoQuality) {
     const quality = _.isInteger(videoQuality)
       ? videoQuality
@@ -306,11 +325,12 @@ export async function startRecordingScreen(options = {}) {
     if (!quality) {
       throw new Error(
         `videoQuality value should be one of ${JSON.stringify(
-          _.keys(QUALITY_MAPPING)
-        )} or a number in range 1..100. ` + `'${videoQuality}' is given instead`
+          _.keys(QUALITY_MAPPING),
+        )} or a number in range 1..100. ` + `'${videoQuality}' is given instead`,
       );
     }
-    mjpegServerScreenshotQuality = mjpegServerScreenshotQuality !== quality ? quality : undefined;
+    mjpegServerScreenshotQuality =
+      mjpegServerScreenshotQuality !== quality ? (quality as number) : undefined;
   } else {
     mjpegServerScreenshotQuality = undefined;
   }
@@ -319,7 +339,7 @@ export async function startRecordingScreen(options = {}) {
     if (isNaN(fps)) {
       throw new Error(
         `videoFps value should be a valid number in range 1..60. ` +
-        `'${videoFps}' is given instead`
+          `'${videoFps}' is given instead`,
       );
     }
     mjpegServerFramerate = mjpegServerFramerate !== fps ? fps : undefined;
@@ -330,8 +350,8 @@ export async function startRecordingScreen(options = {}) {
     await this.proxyCommand('/appium/settings', 'POST', {
       settings: {
         mjpegServerScreenshotQuality,
-        mjpegServerFramerate
-      }
+        mjpegServerFramerate,
+      },
     });
   }
 
@@ -355,16 +375,17 @@ export async function startRecordingScreen(options = {}) {
  * active screen recording processes are running then the method returns an
  * empty string.
  *
- * @param {import('./types').StopRecordingScreenOptions} options - The available
- * options.
- * @returns {Promise<string?>} Base64-encoded content of the recorded media
+ * @param options - The available options.
+ * @returns Base64-encoded content of the recorded media
  * file if `remotePath` parameter is empty or null or an empty string.
  * @throws {Error} If there was an error while getting the name of a media
  *                 file or the file content cannot be uploaded to the remote
  *                 location.
- * @this {XCUITestDriver}
  */
-export async function stopRecordingScreen(options = {}) {
+export async function stopRecordingScreen(
+  this: XCUITestDriver,
+  options: StopRecordingScreenOptions = {},
+): Promise<string | null> {
   if (!this._recentScreenRecorder) {
     this.log.info('Screen recording is not running. There is nothing to stop.');
     return '';
@@ -375,7 +396,7 @@ export async function stopRecordingScreen(options = {}) {
     if (!(await fs.exists(videoPath))) {
       throw this.log.errorWithException(
         `The screen recorder utility has failed ` +
-        `to store the actual screen recording at '${videoPath}'`
+          `to store the actual screen recording at '${videoPath}'`,
       );
     }
     return await encodeBase64OrUpload(videoPath, options.remotePath, options);
@@ -386,6 +407,13 @@ export async function stopRecordingScreen(options = {}) {
   }
 }
 
-/**
- * @typedef {import('../driver').XCUITestDriver} XCUITestDriver
- */
+interface ScreenRecorderOptions {
+  hardwareAcceleration?: string;
+  remotePort?: number;
+  remoteUrl?: string;
+  videoFps?: number;
+  videoType?: string;
+  videoScale?: string;
+  videoFilters?: string;
+  pixelFormat?: string;
+}

package/lib/commands/{xctest.js → xctest.ts}

@@ -2,34 +2,42 @@ import B from 'bluebird';
 import {logger} from 'appium/support';
 import _ from 'lodash';
 import {errors} from 'appium/driver';
+import type {XCUITestDriver} from '../driver';
+import type {XCTestResult, RunXCTestResult} from './types';
+import type {StringRecord} from '@appium/types';
+import type IDB from 'appium-idb';
 
 const XCTEST_TIMEOUT = 360000; // 60 minute timeout
 
 const xctestLog = logger.getLogger('XCTest');
 
 /**
- * Asserts that IDB is present and that launchWithIDB was used
+ * Asserts that IDB is present and that launchWithIDB was used.
  *
- * @param {XCUITestDriver['opts']} opts Opts object from the driver instance
+ * @param opts - Opts object from the driver instance
+ * @returns The IDB instance
+ * @throws {Error} If IDB is not available or launchWithIDB is not enabled
  */
-export function assertIDB(opts) {
-  if (!this.device?.idb || !opts.launchWithIDB) {
+export function assertIDB(this: XCUITestDriver, opts: XCUITestDriver['opts']): IDB {
+  const device = this.device as any;
+  if (!device?.idb || !opts.launchWithIDB) {
     throw new Error(
       `To use XCTest runner, IDB (https://github.com/facebook/idb) must be installed ` +
         `and sessions must be run with the "launchWithIDB" capability`,
     );
   }
-  return this.device.idb;
+  return device.idb;
 }
 
 /**
- * Parse the stdout of XC test log
- * @param {string} stdout A line of standard out from `idb xctest run ...`
- * @returns {XCTestResult[]|string[]} results The final output of the XCTest run
+ * Parse the stdout of XC test log.
+ *
+ * @param stdout - A line of standard out from `idb xctest run ...`
+ * @returns The final output of the XCTest run
  */
-export function parseXCTestStdout(stdout) {
+export function parseXCTestStdout(stdout: string): XCTestResult[] | string[] {
   // Parses a 'key' into JSON format
-  function parseKey(name) {
+  function parseKey(name: string): string {
     const words = name.split(' ');
     let out = '';
     for (const word of words) {
@@ -39,7 +47,7 @@ export function parseXCTestStdout(stdout) {
   }
 
   // Parses a 'value' into JSON format
-  function parseValue(value) {
+  function parseValue(value: string): any {
     value = value || '';
     switch (value.toLowerCase()) {
       case 'true':
@@ -51,7 +59,7 @@ export function parseXCTestStdout(stdout) {
       default:
         break;
     }
-    if (!isNaN(value)) {
+    if (!isNaN(Number(value))) {
       if (!_.isString(value)) {
         return 0;
       } else if (value.indexOf('.') > 0) {
@@ -73,16 +81,14 @@ export function parseXCTestStdout(stdout) {
     return [lines[0]];
   }
 
-  /** @type {XCTestResult[]} */
-  const results = [];
+  const results: XCTestResult[] = [];
   for (const line of lines) {
     // The properties are split up by pipes and each property
     // has the format "Some Key : Some Value"
     const properties = line.split('|');
 
     // Parse each property
-    /** @type {XCTestResult} */
-    const output = /** @type {any} */ ({});
+    const output: any = {};
     let entryIndex = 0;
     for (const prop of properties) {
       if (entryIndex === 0) {
@@ -94,7 +100,7 @@ export function parseXCTestStdout(stdout) {
         // e.g. Location /path/to/XCTesterAppUITests/XCTesterAppUITests.swift:36
         output.location = prop.substring(prop.indexOf('Location') + 8).trim();
       } else {
-        let [key, value] = prop.split(':');
+        const [key, value] = prop.split(':');
         output[parseKey(key.trim())] = parseValue(value ? value.trim() : '');
       }
       entryIndex++;
@@ -123,12 +129,13 @@ export function parseXCTestStdout(stdout) {
 }
 
 /**
- * @typedef {Error} XCUITestError
- *
- * @property {number} code Subprocess exit code
- * @property {string} signal The signal (SIG*) that caused the process to fail
- * @property {XCTestResult[]} results The output of the failed test (if there is output)
+ * Error thrown when XCTest subprocess returns non-zero exit code.
  */
+export interface XCUITestError extends Error {
+  code: number;
+  signal?: string;
+  result?: XCTestResult[];
+}
 
 /**
  * Run a native XCTest script.
@@ -137,36 +144,36 @@ export function parseXCTestStdout(stdout) {
  *
  * **Facebook's [IDB](https://github.com/facebook/idb) tool is required** to run such tests; see [the idb docs](https://fbidb.io/docs/test-execution/) for reference.
  *
- * @param {string} testRunnerBundleId - Test app bundle (e.g.: `io.appium.XCTesterAppUITests.xctrunner`)
- * @param {string} appUnderTestBundleId - App-under-test bundle
- * @param {string} xcTestBundleId - XCTest bundle ID
- * @param {string[]} args - Launch arguments to start the test with (see [reference documentation](https://developer.apple.com/documentation/xctest/xcuiapplication/1500477-launcharguments))
- * @param {'app'|'ui'|'logic'} testType - XC test type
- * @param {import('@appium/types').StringRecord} [env] - Environment variables passed to test
- * @param {number} timeout - Timeout (in ms) for session completion.
+ * @param testRunnerBundleId - Test app bundle (e.g.: `io.appium.XCTesterAppUITests.xctrunner`)
+ * @param appUnderTestBundleId - App-under-test bundle
+ * @param xcTestBundleId - XCTest bundle ID
+ * @param args - Launch arguments to start the test with (see [reference documentation](https://developer.apple.com/documentation/xctest/xcuiapplication/1500477-launcharguments))
+ * @param testType - XC test type
+ * @param env - Environment variables passed to test
+ * @param timeout - Timeout (in ms) for session completion
+ * @returns The array of test results
  * @throws {XCUITestError} Error thrown if subprocess returns non-zero exit code
- * @returns {Promise<import('./types').RunXCTestResult>} The array of test results
- * @this {XCUITestDriver}
  */
 export async function mobileRunXCTest(
-  testRunnerBundleId,
-  appUnderTestBundleId,
-  xcTestBundleId,
-  args = [],
-  testType = 'ui',
-  env,
+  this: XCUITestDriver,
+  testRunnerBundleId: string,
+  appUnderTestBundleId: string,
+  xcTestBundleId: string,
+  args: string[] = [],
+  testType: 'app' | 'ui' | 'logic' = 'ui',
+  env?: StringRecord,
   timeout = XCTEST_TIMEOUT,
-) {
-  const subproc = await assertIDB(this.opts).runXCUITest(
+): Promise<RunXCTestResult> {
+  const subproc = await assertIDB.call(this, this.opts).runXCUITest(
     testRunnerBundleId,
     appUnderTestBundleId,
     xcTestBundleId,
     {env, args, testType},
   );
   return await new B((resolve, reject) => {
-    let mostRecentLogObject = null;
-    let xctestTimeout;
-    let lastErrorMessage = null;
+    let mostRecentLogObject: XCTestResult[] | string[] | null = null;
+    let xctestTimeout: NodeJS.Timeout | undefined;
+    let lastErrorMessage: string | null = null;
     if (timeout > 0) {
       xctestTimeout = setTimeout(
         () =>
@@ -179,11 +186,11 @@ export async function mobileRunXCTest(
       );
     }
 
-    subproc.on('output', (stdout, stderr) => {
+    subproc.on('output', (stdout: string, stderr: string) => {
       if (stdout) {
         try {
           mostRecentLogObject = parseXCTestStdout(stdout);
-        } catch (err) {
+        } catch (err: any) {
           // Fails if log parsing fails.
           // This is in case IDB changes the way that logs are formatted and
           // it breaks 'parseXCTestStdout'. If that happens we still want the process
@@ -202,23 +209,25 @@ export async function mobileRunXCTest(
       }
     });
 
-    subproc.on('exit', (code, signal) => {
-      clearTimeout(xctestTimeout);
+    subproc.on('exit', (code: number | null, signal: string | null) => {
+      if (xctestTimeout) {
+        clearTimeout(xctestTimeout);
+      }
       if (code !== 0) {
-        const err = /** @type {any} */ (new Error(lastErrorMessage || mostRecentLogObject));
-        err.code = code;
+        const err = new Error(lastErrorMessage || String(mostRecentLogObject)) as XCUITestError;
+        err.code = code ?? -1;
         if (signal != null) {
           err.signal = signal;
         }
         if (mostRecentLogObject) {
-          err.result = mostRecentLogObject;
+          err.result = mostRecentLogObject as XCTestResult[];
         }
         return reject(err);
       }
       resolve({
-        code,
-        signal,
-        results: mostRecentLogObject,
+        code: code ?? 0,
+        signal: signal ?? null,
+        results: mostRecentLogObject as XCTestResult[],
         passed: true,
       });
     });
@@ -230,11 +239,12 @@ export async function mobileRunXCTest(
  *
  * **Facebook's [IDB](https://github.com/facebook/idb) tool is required** for this command to work.
  *
- * @param {string} xctestApp - Path of the XCTest app (URL or filename with extension `.app`)
- * @returns {Promise<void>}
- * @this {XCUITestDriver}
+ * @param xctestApp - Path of the XCTest app (URL or filename with extension `.app`)
  */
-export async function mobileInstallXCTestBundle(xctestApp) {
+export async function mobileInstallXCTestBundle(
+  this: XCUITestDriver,
+  xctestApp: string,
+): Promise<void> {
   if (!_.isString(xctestApp)) {
     throw new errors.InvalidArgumentError(
       `'xctestApp' is a required parameter for 'installXCTestBundle' and ` +
@@ -242,7 +252,7 @@ export async function mobileInstallXCTestBundle(xctestApp) {
     );
   }
   xctestLog.info(`Installing bundle '${xctestApp}'`);
-  const idb = assertIDB(this.opts);
+  const idb = assertIDB.call(this, this.opts);
   const res = await this.helpers.configureApp(xctestApp, '.xctest');
   await idb.installXCTestBundle(res);
 }
@@ -252,34 +262,31 @@ export async function mobileInstallXCTestBundle(xctestApp) {
  *
  * **Facebook's [IDB](https://github.com/facebook/idb) tool is required** for this command to work.
  *
- * @returns {Promise<string[]>} List of XCTest bundles (e.g.: `XCTesterAppUITests.XCTesterAppUITests/testLaunchPerformance`)
- * @this {XCUITestDriver}
+ * @returns List of XCTest bundles (e.g.: `XCTesterAppUITests.XCTesterAppUITests/testLaunchPerformance`)
  */
-export async function mobileListXCTestBundles() {
-  return await assertIDB(this.opts).listXCTestBundles();
+export async function mobileListXCTestBundles(this: XCUITestDriver): Promise<string[]> {
+  return await assertIDB.call(this, this.opts).listXCTestBundles();
 }
 
 /**
- * List XCTests in a test bundle
+ * List XCTests in a test bundle.
  *
  * **Facebook's [IDB](https://github.com/facebook/idb) tool is required** for this command to work.
- * @param {string} bundle - Bundle ID of the XCTest
  *
- * @returns {Promise<string[]>} The list of xctests in the test bundle (e.g., `['XCTesterAppUITests.XCTesterAppUITests/testExample', 'XCTesterAppUITests.XCTesterAppUITests/testLaunchPerformance']`)
- * @this {XCUITestDriver}
+ * @param bundle - Bundle ID of the XCTest
+ * @returns The list of xctests in the test bundle (e.g., `['XCTesterAppUITests.XCTesterAppUITests/testExample', 'XCTesterAppUITests.XCTesterAppUITests/testLaunchPerformance']`)
  */
-export async function mobileListXCTestsInTestBundle(bundle) {
+export async function mobileListXCTestsInTestBundle(
+  this: XCUITestDriver,
+  bundle: string,
+): Promise<string[]> {
   if (!_.isString(bundle)) {
     throw new errors.InvalidArgumentError(
       `'bundle' is a required parameter for 'listXCTestsInTestBundle' and ` +
         `must be a string. Found '${bundle}'`,
     );
   }
-  const idb = assertIDB(this.opts);
+  const idb = assertIDB.call(this, this.opts);
   return await idb.listXCTestsInTestBundle(bundle);
 }
 
-/**
- * @typedef {import('../driver').XCUITestDriver} XCUITestDriver
- * @typedef {import('./types').XCTestResult} XCTestResult
- */

package/lib/execute-method-map.ts

@@ -215,7 +215,6 @@ export const executeMethodMap = {
     command: 'getViewportRect',
   },
   'mobile: startPerfRecord': {
-    // @ts-expect-error Class field assignment - method exists on XCUITestDriver
     command: 'mobileStartPerfRecord',
     params: {
       optional: ['timeout', 'profileName', 'pid'],
@@ -253,7 +252,6 @@ export const executeMethodMap = {
     command: 'mobileListCertificates',
   },
   'mobile: startLogsBroadcast': {
-    // @ts-expect-error Class field assignment - method exists on XCUITestDriver
     command: 'mobileStartLogsBroadcast',
   },
   'mobile: stopLogsBroadcast': {

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "appium-xcuitest-driver",
-  "version": "10.13.2",
+  "version": "10.13.4",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "appium-xcuitest-driver",
-      "version": "10.13.2",
+      "version": "10.13.4",
       "license": "Apache-2.0",
       "dependencies": {
         "@appium/strongbox": "^1.0.0-rc.1",
@@ -773,9 +773,9 @@
       }
     },
     "node_modules/appium-webdriveragent": {
-      "version": "11.0.1",
-      "resolved": "https://registry.npmjs.org/appium-webdriveragent/-/appium-webdriveragent-11.0.1.tgz",
-      "integrity": "sha512-pVrwRadjw4kP8RDlIwDsEWqWn0G1IRBok2+HgOFdXsoEIDL0O7DQrlVMSLmD5eiHTCWUjSqCCLpDSE6c7QaYzw==",
+      "version": "11.1.0",
+      "resolved": "https://registry.npmjs.org/appium-webdriveragent/-/appium-webdriveragent-11.1.0.tgz",
+      "integrity": "sha512-Ve9n+QIM4JUwTvKKeuaVvyra1k02JAcNsmgTtbx3fNItEzuC/s4LjEvqoxdr5KAjGAIz/Np5Efqn6S1WyvEiZQ==",
       "license": "Apache-2.0",
       "dependencies": {
         "@appium/base-driver": "^10.0.0-rc.1",
@@ -3550,9 +3550,9 @@
       "license": "MIT"
     },
     "node_modules/qs": {
-      "version": "6.14.0",
-      "resolved": "https://registry.npmjs.org/qs/-/qs-6.14.0.tgz",
-      "integrity": "sha512-YWWTjgABSKcvs/nWBi9PycY/JiPJqOD4JA6o9Sej2AtvSGarXxKC3OQSk4pAarbdQlKAh5D4FCQkJNkW+GAn3w==",
+      "version": "6.14.1",
+      "resolved": "https://registry.npmjs.org/qs/-/qs-6.14.1.tgz",
+      "integrity": "sha512-4EK3+xJl8Ts67nLYNwqw/dsFVnCf+qR7RgXSK9jEEm9unao3njwMDdmsdvoKBKHzxd7tCYz5e5M+SnMjdtXGQQ==",
       "license": "BSD-3-Clause",
       "dependencies": {
         "side-channel": "^1.1.0"