appium-xcuitest-driver 10.14.13 → 10.16.0

package/lib/commands/certificate.ts

@@ -7,8 +7,8 @@ import path from 'path';
 import http from 'http';
 import {exec} from 'teen_process';
 import {findAPortNotInUse, checkPortStatus} from 'portscanner';
-import {Pyidevice} from '../device/clients/py-ios-device-client';
-import {requireRealDevice} from '../utils';
+import {CertificateClient} from '../device/certificate-client';
+import {requireRealDevice, isIos18OrNewer} from '../utils';
 import type {Simulator} from 'appium-ios-simulator';
 import type {XCUITestDriver} from '../driver';
 import type {CertificateList} from './types';
@@ -122,18 +122,23 @@ export async function mobileInstallCertificate(
     if (!this.opts.udid) {
       throw new Error('udid capability is required');
     }
-    const client = new Pyidevice({
-      udid: this.opts.udid,
-      log: this.log,
-    });
-    if (await client.assertExists(false)) {
-      await client.installProfile({payload: Buffer.from(content, 'base64')});
-      return;
-    } else {
-      this.log.info(
-        'pyidevice is not installed on your system. ' +
-          'Falling back to the (slow) UI-based installation',
+
+    let client: CertificateClient | null = null;
+    try {
+      client = await CertificateClient.create(
+        this.opts.udid,
+        this.log,
+        isIos18OrNewer(this.opts)
       );
+      await client.installCertificate({payload: Buffer.from(content, 'base64')});
+      return;
+    } catch (err) {
+      this.log.error(`Failed to install the certificate: ${err.message}`);
+      this.log.info('Falling back to the (slow) UI-based installation');
+    } finally {
+      if (client) {
+        await client.close();
+      }
     }
   }
 
@@ -256,12 +261,20 @@ export async function mobileRemoveCertificate(this: XCUITestDriver, name: string
   if (!this.opts.udid) {
     throw new Error('udid capability is required');
   }
-  const client = new Pyidevice({
-    udid: this.opts.udid,
-    log: this.log,
-  });
-  await client.assertExists(true);
-  return await client.removeProfile(name);
+
+  let client: CertificateClient | null = null;
+  try {
+    client = await CertificateClient.create(
+      this.opts.udid,
+      this.log,
+      isIos18OrNewer(this.opts)
+    );
+    return await client.removeCertificate(name);
+  } finally {
+    if (client) {
+      await client.close();
+    }
+  }
 }
 
 /**
@@ -278,14 +291,21 @@ export async function mobileListCertificates(this: XCUITestDriver): Promise<Cert
   if (!this.opts.udid) {
     throw new Error('udid capability is required');
   }
-  const client = new Pyidevice({
-    udid: this.opts.udid,
-    log: this.log,
-  });
-  await client.assertExists(true);
-  return await client.listProfiles();
-}
 
+  let client: CertificateClient | null = null;
+  try {
+    client = await CertificateClient.create(
+      this.opts.udid,
+      this.log,
+      isIos18OrNewer(this.opts)
+    );
+    return await client.listCertificates();
+  } finally {
+    if (client) {
+      await client.close();
+    }
+  }
+}
 
 /**
  * Extracts the common name of the certificate from the given buffer.
@@ -501,4 +521,3 @@ async function installPost122Certificate(driver: XCUITestDriver, name: string):
 
   return true;
 }
-

package/lib/commands/file-movement.ts

@@ -1,7 +1,6 @@
 import _ from 'lodash';
 import {fs, tempDir, mkdirp, zip, util} from 'appium/support';
 import path from 'path';
-import {services} from 'appium-ios-device';
 import {
   pullFile as realDevicePullFile,
   pullFolder as realDevicePullFolder,
@@ -11,6 +10,24 @@ import {errors} from 'appium/driver';
 import type {Simulator} from 'appium-ios-simulator';
 import type {XCUITestDriver} from '../driver';
 import type {ContainerObject, ContainerRootSupplier} from './types';
+import {isIos18OrNewer} from '../utils';
+import {AfcClient} from '../device/afc-client';
+
+//#region Type Definitions
+
+interface CreateServiceResult {
+  client: AfcClient;
+  relativePath: string;
+}
+
+interface CreateAfcClientOptions {
+  bundleId?: string | null;
+  containerType?: string | null;
+}
+
+//#endregion
+
+//#region Constants
 
 const CONTAINER_PATH_MARKER = '@';
 // https://regex101.com/r/PLdB0G/2
@@ -19,6 +36,10 @@ const CONTAINER_TYPE_SEPARATOR = ':';
 const CONTAINER_DOCUMENTS_PATH = 'Documents';
 const OBJECT_NOT_FOUND_ERROR_MESSAGE = 'OBJECT_NOT_FOUND';
 
+//#endregion
+
+//#region Public Exported Functions
+
 /**
  * Parses the actual path and the bundle identifier from the given path string.
  *
@@ -44,9 +65,12 @@ export async function parseContainerPath(
   const typeSeparatorPos = bundleId.indexOf(CONTAINER_TYPE_SEPARATOR);
   // We only consider container type exists if its length is greater than zero
   // not counting the colon
-  if (typeSeparatorPos > 0 && typeSeparatorPos < bundleId.length - 1) {
-    containerType = bundleId.substring(typeSeparatorPos + 1);
-    this.log.debug(`Parsed container type: ${containerType}`);
+  if (typeSeparatorPos > 0) {
+    if (typeSeparatorPos < bundleId.length - 1) {
+      containerType = bundleId.substring(typeSeparatorPos + 1);
+      this.log.debug(`Parsed container type: ${containerType}`);
+    }
+    // Always strip the colon and everything after it
     bundleId = bundleId.substring(0, typeSeparatorPos);
   }
   if (_.isNil(containerRootSupplier)) {
@@ -202,6 +226,29 @@ export async function mobilePullFolder(this: XCUITestDriver, remotePath: string)
   return await this.pullFolder(remotePath);
 }
 
+//#endregion
+
+//#region Private Helper Functions
+
+/**
+ * Delete file or folder helper
+ */
+async function deleteFileOrFolder(this: XCUITestDriver, remotePath: string): Promise<void> {
+  return this.isSimulator()
+    ? await deleteFromSimulator.bind(this)(remotePath)
+    : await deleteFromRealDevice.bind(this)(remotePath);
+}
+
+/**
+ * Check if container type refers to documents container
+ */
+function isDocumentsContainer(containerType?: string | null): boolean {
+  return _.toLower(containerType ?? '') === _.toLower(CONTAINER_DOCUMENTS_PATH);
+}
+
+/**
+ * Verify that a path is a subpath of a root directory
+ */
 function verifyIsSubPath(originalPath: string, root: string): void {
   const normalizedRoot = path.normalize(root);
   const normalizedPath = path.normalize(path.dirname(originalPath));
@@ -211,50 +258,49 @@ function verifyIsSubPath(originalPath: string, root: string): void {
   }
 }
 
+/**
+ * Create AFC client for file operations
+ */
 async function createAfcClient(
   this: XCUITestDriver,
-  bundleId?: string | null,
-  containerType?: string | null,
-): Promise<any> {
+  opts: CreateAfcClientOptions = {}
+): Promise<AfcClient> {
+  const {bundleId, containerType} = opts;
   const udid = this.device.udid as string;
+  const useIos18 = isIos18OrNewer(this.opts);
 
-  if (!bundleId) {
-    return await services.startAfcService(udid);
-  }
-  const service = await services.startHouseArrestService(udid);
-
-  const {
-    skipDocumentsContainerCheck = false,
-  } = await this.settings.getSettings();
-
-  if (skipDocumentsContainerCheck) {
-    return service.vendContainer(bundleId);
+  if (bundleId) {
+    const skipDocumentsCheck = this.settings.getSettings().skipDocumentsContainerCheck ?? false;
+    return await AfcClient.createForApp(udid, bundleId, useIos18, {
+      containerType: containerType ?? null,
+      skipDocumentsCheck,
+    });
   }
 
-  return isDocumentsContainer(containerType)
-    ? await service.vendDocuments(bundleId)
-    : await service.vendContainer(bundleId);
-}
-
-function isDocumentsContainer(containerType?: string | null): boolean {
-  return _.toLower(containerType ?? '') === _.toLower(CONTAINER_DOCUMENTS_PATH);
+  return await AfcClient.createForDevice(udid, useIos18);
 }
 
+/**
+ * Create service for file operations
+ */
 async function createService(
   this: XCUITestDriver,
   remotePath: string,
-): Promise<{service: any; relativePath: string}> {
+): Promise<CreateServiceResult> {
   if (CONTAINER_PATH_PATTERN.test(remotePath)) {
-    const {bundleId, pathInContainer, containerType} = await parseContainerPath.bind(this)(remotePath);
-    const service = await createAfcClient.bind(this)(bundleId, containerType);
-    const relativePath = isDocumentsContainer(containerType)
+    const {bundleId, pathInContainer, containerType}: ContainerObject = await parseContainerPath.bind(this)(remotePath);
+    const client: AfcClient = await createAfcClient.bind(this)({bundleId, containerType});
+    let relativePath = isDocumentsContainer(containerType)
       ? path.join(CONTAINER_DOCUMENTS_PATH, pathInContainer)
       : pathInContainer;
-    return {service, relativePath};
-  } else {
-    const service = await createAfcClient.bind(this)();
-    return {service, relativePath: remotePath};
+    // Ensure path starts with / for AFC operations
+    if (!relativePath.startsWith('/')) {
+      relativePath = `/${relativePath}`;
+    }
+    return {client, relativePath};
   }
+  const client: AfcClient = await createAfcClient.bind(this)({});
+  return {client, relativePath: remotePath};
 }
 
 /**
@@ -312,23 +358,17 @@ async function pushFileToRealDevice(
   remotePath: string,
   base64Data: string,
 ): Promise<void> {
-  const {service, relativePath} = await createService.bind(this)(remotePath);
+  const {client, relativePath} = await createService.bind(this)(remotePath);
   try {
-    await realDevicePushFile(service, Buffer.from(base64Data, 'base64'), relativePath);
+    await realDevicePushFile(client, Buffer.from(base64Data, 'base64'), relativePath);
   } catch (e) {
-    this.log.debug(e.stack);
-    throw new Error(`Could not push the file to '${remotePath}'. Original error: ${e.message}`);
+    this.log.debug((e as Error).stack);
+    throw new Error(`Could not push the file to '${remotePath}'. Original error: ${(e as Error).message}`);
   } finally {
-    service.close();
+    await client.close();
   }
 }
 
-async function deleteFileOrFolder(this: XCUITestDriver, remotePath: string): Promise<void> {
-  return this.isSimulator()
-    ? await deleteFromSimulator.bind(this)(remotePath)
-    : await deleteFromRealDevice.bind(this)(remotePath);
-}
-
 /**
  * Get the content of given file or folder from iOS Simulator and return it as base-64 encoded string.
  * Folder content is recursively packed into a zip archive.
@@ -402,21 +442,23 @@ async function pullFromRealDevice(
   remotePath: string,
   isFile: boolean,
 ): Promise<string> {
-  const {service, relativePath} = await createService.bind(this)(remotePath);
+  const {client, relativePath} = await createService.bind(this)(remotePath);
   try {
-    const fileInfo = await service.getFileInfo(relativePath);
-    if (isFile && fileInfo.isDirectory()) {
+    // Check if path is a directory
+    const isDirectory = await client.isDirectory(relativePath);
+
+    if (isFile && isDirectory) {
       throw new Error(`The requested path is not a file. Path: '${remotePath}'`);
     }
-    if (!isFile && !fileInfo.isDirectory()) {
+    if (!isFile && !isDirectory) {
       throw new Error(`The requested path is not a folder. Path: '${remotePath}'`);
     }
 
-    return fileInfo.isFile()
-      ? (await realDevicePullFile(service, relativePath)).toString('base64')
-      : (await realDevicePullFolder(service, relativePath)).toString();
+    return isDirectory
+      ? (await realDevicePullFolder(client, relativePath)).toString()
+      : (await realDevicePullFile(client, relativePath)).toString('base64');
   } finally {
-    service.close();
+    await client.close();
   }
 }
 
@@ -476,16 +518,18 @@ async function deleteFromSimulator(this: XCUITestDriver, remotePath: string): Pr
  * @returns Nothing
  */
 async function deleteFromRealDevice(this: XCUITestDriver, remotePath: string): Promise<void> {
-  const {service, relativePath} = await createService.bind(this)(remotePath);
+  const {client, relativePath} = await createService.bind(this)(remotePath);
   try {
-    await service.deleteDirectory(relativePath);
+    await client.deleteDirectory(relativePath);
   } catch (e) {
-    if (e.message.includes(OBJECT_NOT_FOUND_ERROR_MESSAGE)) {
+    if ((e as Error).message.includes(OBJECT_NOT_FOUND_ERROR_MESSAGE)) {
       throw new Error(`Path '${remotePath}' does not exist on the device`);
     }
     throw e;
   } finally {
-    service.close();
+    await client.close();
   }
 }
 
+//#endregion
+