appium-xcuitest-driver 9.3.1 → 9.4.0

package/lib/commands/gesture.js

@@ -107,19 +107,16 @@ const commands = {
     // This is mandatory, since WDA only supports TOUCH pointer type
     // and Selenium API uses MOUSE as the default one
     const preprocessedActions = actions
-      .map((action) =>
-        Object.assign(
-          {},
-          action,
-          action.type === 'pointer'
-            ? {
-                parameters: {
-                  pointerType: 'touch',
-                },
-              }
-            : {},
-        ),
-      )
+      .map((action) => ({
+        ...action,
+        ...(action.type === 'pointer'
+          ? {
+              parameters: {
+                pointerType: 'touch',
+              },
+            }
+          : {}),
+      }))
       .map((action) => {
         const modifiedAction = _.clone(action) || {};
         // Selenium API unexpectedly inserts zero pauses, which are not supported by WDA
@@ -170,14 +167,14 @@ const helpers = {
    *
    * Use this command to emulate precise scrolling in tables or collection views where it is already known to which element the scrolling should be performed.
    *
-   * The arguments define the choosen strategy: one of `name`, `direction`, `predicateString` or `toVisible`.
+   * The arguments define the chosen strategy: one of `name`, `direction`, `predicateString` or `toVisible`.
    *
    * **All strategies are exclusive**; only one strategy can be used at one time.
    *
    * **Known Limitations:**
    *
    * - If it is necessary to perform many scroll gestures on parent container to reach the necessary child element (tens of them), then the method call may fail.  *
-   * - The implemntation of this extension relies on several undocumented XCTest features, which might not always be reliable.
+   * - The implementation of this extension relies on several undocumented XCTest features, which might not always be reliable.
    *
    * @param {string} [name] - The internal element identifier (as hexadecimal hash string) to scroll on (e.g. the container). The Application element will be used if this argument is not provided.
    * @param {import('./types').Direction} [direction] - The main difference between this command and a `mobile: swipe` command using the same direction is that `mobile: scroll` will attempt to move the current viewport exactly to the next or previous page (the term "page" means the content, which fits into a single device screen).
@@ -473,7 +470,7 @@ const helpers = {
    * @throws If the target device does not support the "force press" gesture.
    * @param {number} [x] - The _x_ coordinate of the gesture. If `elementId` is set, this is calculated relative to its position; otherwise it's calculated relative to the active Application.
    * @param {number} [y] - The _y_ coordinate of the gesture. If `elementId` is set, this is calculated relative to its position; otherwise it's calculated relative to the active Application.
-   * @param {number} [duration] - The duraiton (in seconds) of the force press. If this is provided, `pressure` must also be provided.
+   * @param {number} [duration] - The duration (in seconds) of the force press. If this is provided, `pressure` must also be provided.
    * @param {number} [pressure] - A float value defining the pressure of the force press. If this is provided, `duration` must also be provided.
    * @param {string|Element} [elementId] - The internal element identifier (as hexadecimal hash string) to perform one or more taps.
    * The Application element will be used if this parameter is not provided.

package/lib/commands/hid-event.ts

@@ -1128,7 +1128,7 @@ export enum HIDUsagePowerDeviceEvent {
   /* Reserved 0x48 - 0x4F */
   kHIDUsage_PD_SwitchOnControl = 0x50 /* DV F- Power Device Switch On Control */,
   kHIDUsage_PD_SwitchOffControl = 0x51 /* DV F- Power Device Switch Off Control */,
-  kHIDUsage_PD_ToggleControl = 0x52 /* DV F- Power Device Toogle Sequence Control */,
+  kHIDUsage_PD_ToggleControl = 0x52 /* DV F- Power Device Toggle Sequence Control */,
   kHIDUsage_PD_LowVoltageTransfer = 0x53 /* DV F- Power Device Min Transfer Voltage */,
   kHIDUsage_PD_HighVoltageTransfer = 0x54 /* DV F- Power Device Max Transfer Voltage */,
   kHIDUsage_PD_DelayBeforeReboot = 0x55 /* DV F- Power Device Delay Before Reboot */,

package/lib/commands/increase-contrast.js

@@ -14,7 +14,7 @@ export default {
    * Sets the increase contrast configuration for the given simulator.
    *
    * @since Xcode 15 (but lower xcode could have this command)
-   * @param {IncreaseContrastAction} increaseContrast valid increase constrast configuration value.
+   * @param {IncreaseContrastAction} increaseContrast valid increase contrast configuration value.
    *                                                  Acceptable value is 'enabled' or 'disabled' with Xcode 16.2.
    * @throws {Error} if the current platform does not support content size appearance changes
    * @this {XCUITestDriver}

package/lib/commands/localization.js

@@ -8,7 +8,7 @@ export default {
    * Change localization settings on the currently booted simulator
    *
    * The changed settings are only applied for _newly started_ applications and activities.
-   * Currently running applications will be unchanged. This means, for example, that the keyboard should be hidden and shown again in order to observe the changed layout, and curresponding apps must be restarted in order to observe their interface using the newly set locale/language.
+   * Currently running applications will be unchanged. This means, for example, that the keyboard should be hidden and shown again in order to observe the changed layout, and corresponding apps must be restarted in order to observe their interface using the newly set locale/language.
    *
    * The driver performs no strict checking of the arguments (such as locale names). Be aware that an incorrect or invalid string may cause unexpected behavior.
    * @param {import('./types').KeyboardOptions} [keyboard] - Keyboard options

package/lib/commands/location.js

@@ -118,7 +118,7 @@ export default {
     } catch (err) {
       throw this.log.errorWithException(
         `Failed to reset the location on the device on device '${this.opts.udid}'. ` +
-          `Origianl error: ${err.message}`,
+          `Original error: ${err.message}`,
       );
     } finally {
       service.close();

package/lib/commands/recordscreen.js

@@ -138,7 +138,7 @@ export class ScreenRecorder {
       this.timeoutHandler = null;
     }
 
-    if (this.mainProcess && this.mainProcess.isRunning) {
+    if (this.mainProcess?.isRunning) {
       const interruptPromise = this.mainProcess.stop(force ? 'SIGTERM' : 'SIGINT');
       this.mainProcess = null;
       try {

package/lib/device-log/ios-simulator-log.ts

@@ -71,7 +71,7 @@ export class IOSSimulatorLog extends LineConsumingLog {
   }
 
   override get isCapturing(): boolean {
-    return Boolean(this.proc && this.proc.isRunning);
+    return Boolean(this.proc?.isRunning);
   }
 
   private onOutput(logRow: string, prefix: string = ''): void {
@@ -131,7 +131,7 @@ export class IOSSimulatorLog extends LineConsumingLog {
       await exec('kill', pids.map(String));
     } catch (e) {
       this.log.warn(
-        `Cound not terminate one or more obsolete log streams: ${e.stderr || e.message}`
+        `Could not terminate one or more obsolete log streams: ${e.stderr || e.message}`
       );
     }
   }

package/lib/driver.js

@@ -429,7 +429,7 @@ export class XCUITestDriver extends BaseDriver {
       await this.start();
 
       // merge server capabilities + desired capabilities
-      caps = Object.assign({}, defaultServerCaps, caps);
+      caps = { ...defaultServerCaps, ...caps };
       // update the udid with what is actually used
       caps.udid = this.opts.udid;
       // ensure we track nativeWebTap capability as a setting as well
@@ -1025,8 +1025,8 @@ export class XCUITestDriver extends BaseDriver {
       await simulatorDevice.delete();
     }
 
-    const shouldResetLocationServivce = this.isRealDevice() && !!this.opts.resetLocationService;
-    if (shouldResetLocationServivce) {
+    const shouldResetLocationService = this.isRealDevice() && !!this.opts.resetLocationService;
+    if (shouldResetLocationService) {
       try {
         await this.mobileResetLocationService();
       } catch {
@@ -1055,7 +1055,7 @@ export class XCUITestDriver extends BaseDriver {
     this.jwpProxyActive = false;
     this.proxyReqRes = null;
 
-    if (this.wda && this.wda.fullyStarted) {
+    if (this.wda?.fullyStarted) {
       if (this.wda.jwproxy) {
         try {
           await this.proxyCommand(`/session/${this.sessionId}`, 'DELETE');
@@ -1065,7 +1065,7 @@ export class XCUITestDriver extends BaseDriver {
         }
       }
       // The former could cache the xcodebuild, so should not quit the process.
-      // If the session skiped the xcodebuild (this.wda.canSkipXcodebuild), the this.wda instance
+      // If the session skipped the xcodebuild (this.wda.canSkipXcodebuild), the this.wda instance
       // should quit properly.
       if ((!this.wda.webDriverAgentUrl && this.opts.useNewWDA) || this.wda.canSkipXcodebuild) {
         await this.wda.quit();
@@ -1666,7 +1666,7 @@ export class XCUITestDriver extends BaseDriver {
     } catch (e) {
       throw this.log.errorWithException(`Could not parse "otherApps" capability: ${e.message}`);
     }
-    if (!appsList || !appsList.length) {
+    if (!appsList?.length) {
       this.log.info(`Got zero apps from 'otherApps' capability value. Doing nothing`);
       return;
     }
@@ -1764,7 +1764,7 @@ export class XCUITestDriver extends BaseDriver {
     );
 
     // Note: The CFBundleVersion in the test bundle was always 1.
-    // It may not be able to compare with the installed versio.
+    // It may not be able to compare with the installed version.
     if (this.isRealDevice()) {
       await installToRealDevice.bind(this)(
         this.opts.prebuiltWDAPath,

package/lib/utils.js

@@ -402,7 +402,7 @@ async function encodeBase64OrUpload(localPath, remotePath = null, uploadOptions
 
 /**
  * Stops and removes all web socket handlers that are listening
- * in scope of the currect session.
+ * in scope of the current session.
  *
  * @param {Object} server - The instance of NodeJs HTTP server,
  * which hosts Appium