detox 20.12.1 → 20.13.0

package/src/realms/DetoxPrimaryContext.js

@@ -16,15 +16,14 @@ const symbols = require('./symbols');
 const { $logFinalizer, $restoreSessionState, $sessionState, $worker } = DetoxContext.protected;
 
 //#region Private symbols
-const _globalLifecycleHandler = Symbol('globalLifecycleHandler');
 const _ipcServer = Symbol('ipcServer');
-const _resetLockFile = Symbol('resetLockFile');
 const _wss = Symbol('wss');
 const _dirty = Symbol('dirty');
 const _emergencyTeardown = Symbol('emergencyTeardown');
 const _lifecycleLogger = Symbol('lifecycleLogger');
 const _sessionFile = Symbol('sessionFile');
 const _logFinalError = Symbol('logFinalError');
+const _deviceAllocator = Symbol('deviceAllocator');
 //#endregion
 
 class DetoxPrimaryContext extends DetoxContext {
@@ -33,7 +32,8 @@ class DetoxPrimaryContext extends DetoxContext {
 
     this[_dirty] = false;
     this[_wss] = null;
-    this[_globalLifecycleHandler] = null;
+    this[_deviceAllocator] = null;
+
     /** Path to file where the initial session object is serialized */
     this[_sessionFile] = '';
     /**
@@ -51,6 +51,12 @@ class DetoxPrimaryContext extends DetoxContext {
     }
   }
 
+  [symbols.conductEarlyTeardown] = async () => {
+    if (this[_ipcServer]) {
+      await this[_ipcServer].onConductEarlyTeardown();
+    }
+  };
+
   async [symbols.resolveConfig](opts = {}) {
     const session = this[$sessionState];
     if (!session.detoxConfig) {
@@ -79,7 +85,6 @@ class DetoxPrimaryContext extends DetoxContext {
     const detoxConfig = await this[symbols.resolveConfig](opts);
 
     const {
-      behavior: behaviorConfig,
       device: deviceConfig,
       logger: loggerConfig,
       session: sessionConfig
@@ -96,20 +101,23 @@ class DetoxPrimaryContext extends DetoxContext {
     this[_ipcServer] = new IPCServer({
       sessionState: this[$sessionState],
       logger: this[symbols.logger],
+      callbacks: {
+        onAllocateDevice: this[symbols.allocateDevice].bind(this),
+        onDeallocateDevice: this[symbols.deallocateDevice].bind(this),
+      },
     });
 
     await this[_ipcServer].init();
 
     const environmentFactory = require('../environmentFactory');
-    this[_globalLifecycleHandler] = await environmentFactory.createGlobalLifecycleHandler(deviceConfig);
 
-    if (this[_globalLifecycleHandler]) {
-      await this[_globalLifecycleHandler].globalInit();
-    }
+    const { deviceAllocatorFactory } = environmentFactory.createFactories(deviceConfig);
+    this[_deviceAllocator] = deviceAllocatorFactory.createDeviceAllocator({
+      detoxConfig,
+      detoxSession: this[$sessionState],
+    });
 
-    if (!behaviorConfig.init.keepLockFile) {
-      await this[_resetLockFile]();
-    }
+    await this[_deviceAllocator].init();
 
     // TODO: Detox-server creation ought to be delegated to a generator/factory.
     const DetoxServer = require('../server/DetoxServer');
@@ -153,6 +161,29 @@ class DetoxPrimaryContext extends DetoxContext {
     await super[symbols.installWorker]({ ...opts, workerId });
   }
 
+  /** @override */
+  async [symbols.allocateDevice]() {
+    const { device } = this[$sessionState].detoxConfig;
+    const deviceCookie = await this[_deviceAllocator].allocate(device);
+
+    try {
+      return await this[_deviceAllocator].postAllocate(deviceCookie);
+    } catch (e) {
+      try {
+        await this[_deviceAllocator].free(deviceCookie, { shutdown: true });
+      } catch (e2) {
+        this[symbols.logger].error({ cat: 'device', err: e2 }, `Failed to free ${deviceCookie.name || deviceCookie.id} after a failed allocation`);
+      }
+
+      throw e;
+    }
+  }
+
+  /** @override */
+  async [symbols.deallocateDevice](cookie) {
+    await this[_deviceAllocator].free(cookie);
+  }
+
   /** @override */
   async [symbols.cleanup]() {
     try {
@@ -160,9 +191,9 @@ class DetoxPrimaryContext extends DetoxContext {
         await this[symbols.uninstallWorker]();
       }
     } finally {
-      if (this[_globalLifecycleHandler]) {
-        await this[_globalLifecycleHandler].globalCleanup();
-        this[_globalLifecycleHandler] = null;
+      if (this[_deviceAllocator]) {
+        await this[_deviceAllocator].cleanup();
+        this[_deviceAllocator] = null;
       }
 
       if (this[_wss]) {
@@ -196,9 +227,9 @@ class DetoxPrimaryContext extends DetoxContext {
       return;
     }
 
-    if (this[_globalLifecycleHandler]) {
-      this[_globalLifecycleHandler].emergencyCleanup();
-      this[_globalLifecycleHandler] = null;
+    if (this[_deviceAllocator]) {
+      this[_deviceAllocator].emergencyCleanup();
+      this[_deviceAllocator] = null;
     }
 
     if (this[_wss]) {
@@ -241,31 +272,6 @@ class DetoxPrimaryContext extends DetoxContext {
     });
   }
   //#endregion
-
-  //#region Private members
-  async[_resetLockFile]() {
-    const DeviceRegistry = require('../devices/DeviceRegistry');
-
-    const deviceType = this[symbols.config].device.type;
-
-    switch (deviceType) {
-      case 'ios.none':
-      case 'ios.simulator':
-        await DeviceRegistry.forIOS().reset();
-        break;
-      case 'android.attached':
-      case 'android.emulator':
-      case 'android.genycloud':
-        await DeviceRegistry.forAndroid().reset();
-        break;
-    }
-
-    if (deviceType === 'android.genycloud') {
-      const GenyDeviceRegistryFactory = require('../devices/allocation/drivers/android/genycloud/GenyDeviceRegistryFactory');
-      await GenyDeviceRegistryFactory.forGlobalShutdown().reset();
-    }
-  }
-  //#endregion
 }
 
 module.exports = DetoxPrimaryContext;

package/src/realms/DetoxSecondaryContext.js

@@ -33,6 +33,14 @@ class DetoxSecondaryContext extends DetoxContext {
     }
   }
 
+  [symbols.conductEarlyTeardown] = async () => {
+    if (this[_ipcClient]) {
+      await this[_ipcClient].conductEarlyTeardown();
+    } else {
+      throw new DetoxInternalError('Detected an attempt to report early teardown using a non-initialized context.');
+    }
+  };
+
   async [symbols.resolveConfig]() {
     return this[symbols.config];
   }
@@ -54,6 +62,25 @@ class DetoxSecondaryContext extends DetoxContext {
     }
   }
 
+  /** @override */
+  async [symbols.allocateDevice]() {
+    if (this[_ipcClient]) {
+      const deviceCookie = await this[_ipcClient].allocateDevice();
+      return deviceCookie;
+    } else {
+      throw new DetoxInternalError('Detected an attempt to allocate a device using a non-initialized context.');
+    }
+  }
+
+  /** @override */
+  async [symbols.deallocateDevice](deviceCookie) {
+    if (this[_ipcClient]) {
+      await this[_ipcClient].deallocateDevice(deviceCookie);
+    } else {
+      throw new DetoxInternalError('Detected an attempt to allocate a device using a non-initialized context.');
+    }
+  }
+
   /** @override */
   async [symbols.cleanup]() {
     try {

package/src/realms/symbols.js

@@ -5,6 +5,8 @@
  *   readonly getStatus: unique symbol;
  *   readonly init: unique symbol;
  *   readonly installWorker: unique symbol;
+ *   readonly allocateDevice: unique symbol;
+ *   readonly deallocateDevice: unique symbol;
  *   readonly logger: unique symbol;
  *   readonly onHookFailure: unique symbol;
  *   readonly onRunDescribeFinish: unique symbol;
@@ -12,6 +14,7 @@
  *   readonly onTestDone: unique symbol;
  *   readonly onTestFnFailure: unique symbol;
  *   readonly onTestStart: unique symbol;
+ *   readonly conductEarlyTeardown: unique symbol;
  *   readonly reportTestResults: unique symbol;
  *   readonly resolveConfig: unique symbol;
  *   readonly session: unique symbol;
@@ -32,6 +35,9 @@ module.exports = {
 
   //#region IPC
   reportTestResults: Symbol('reportTestResults'),
+  conductEarlyTeardown: Symbol('conductEarlyTeardown'),
+  allocateDevice: Symbol('allocateDevice'),
+  deallocateDevice: Symbol('deallocateDevice'),
   //#endregion
 
   //#region Main

package/src/utils/PIDService.js

@@ -0,0 +1,27 @@
+const { pid } = require('process');
+
+class PIDService {
+  getPid() {
+    return pid;
+  }
+
+  /**
+   * Checks if the other process id is running in the current operating system
+   * @param {number} otherPID
+   * @returns {boolean}
+   */
+  isAlive(otherPID) {
+    try {
+      process.kill(otherPID, 0);
+      return true;
+    } catch (ex) {
+      if (ex.code === 'ESRCH') {
+        return false;
+      }
+      throw ex;
+    }
+  }
+
+}
+
+module.exports = PIDService;

package/src/utils/environment.js

@@ -19,9 +19,8 @@ function which(executable, path) {
 const DETOX_LIBRARY_ROOT_PATH = path.join(appdatapath.appDataPath(), 'Detox');
 const MISSING_SDK_ERROR = `$ANDROID_SDK_ROOT is not defined, set the path to the SDK installation directory into $ANDROID_SDK_ROOT,
 Go to https://developer.android.com/studio/command-line/variables.html for more details`;
-const DEVICE_LOCK_FILE_PATH_IOS = path.join(DETOX_LIBRARY_ROOT_PATH, 'device.registry.state.lock');
-const DEVICE_LOCK_FILE_PATH_ANDROID = path.join(DETOX_LIBRARY_ROOT_PATH, 'android-device.registry.state.lock');
-const GENYCLOUD_GLOBAL_CLEANUP_FILE_PATH = path.join(DETOX_LIBRARY_ROOT_PATH, 'genycloud-cleanup.lock');
+const DETOX_LOCK_FILE_PATH = path.join(DETOX_LIBRARY_ROOT_PATH, 'global-context.json');
+const DEVICE_REGISTRY_PATH = path.join(DETOX_LIBRARY_ROOT_PATH, 'device.registry.json');
 const LAST_FAILED_TESTS_PATH = path.join(DETOX_LIBRARY_ROOT_PATH, 'last-failed.txt');
 
 function getAndroidSDKPath() {
@@ -195,17 +194,12 @@ function getDetoxLibraryRootPath() {
   return DETOX_LIBRARY_ROOT_PATH;
 }
 
-function getDeviceLockFilePathIOS() {
-  return DEVICE_LOCK_FILE_PATH_IOS;
+function getDetoxLockFilePath() {
+  return DETOX_LOCK_FILE_PATH;
 }
 
-// TODO This can probably be merged with IOS' by now
-function getDeviceLockFilePathAndroid() {
-  return DEVICE_LOCK_FILE_PATH_ANDROID;
-}
-
-function getGenyCloudGlobalCleanupFilePath() {
-  return GENYCLOUD_GLOBAL_CLEANUP_FILE_PATH;
+function getDeviceRegistryPath() {
+  return DEVICE_REGISTRY_PATH;
 }
 
 function getLastFailedTestsPath() {
@@ -229,9 +223,8 @@ module.exports = {
   getAndroidSDKPath,
   getAndroidEmulatorPath,
   getDetoxLibraryRootPath,
-  getDeviceLockFilePathIOS,
-  getDeviceLockFilePathAndroid,
-  getGenyCloudGlobalCleanupFilePath,
+  getDetoxLockFilePath,
+  getDeviceRegistryPath,
   getLastFailedTestsPath,
   getHomeDir,
 };

package/src/utils/errorUtils.js

@@ -40,15 +40,15 @@ function asError(error) {
   return isError(error) ? error : new Error(error);
 }
 
-function serializeObjectWithError(obj, errorKey) {
-  if (obj[errorKey]) {
+function serializeObjectWithError(obj, errorKey = 'error') {
+  if (obj[errorKey] instanceof Error) {
     return { ...obj, [errorKey]: serializeError(obj[errorKey]) };
   }
 
   return obj;
 }
 
-function deserializeObjectWithError(obj, errorKey) {
+function deserializeObjectWithError(obj, errorKey = 'error') {
   if (typeof obj[errorKey] === 'object' && !(obj[errorKey] instanceof Error)) {
     return { ...obj, [errorKey]: deserializeError(obj[errorKey]) };
   }

package/tsconfig.json

@@ -1,11 +1,11 @@
 {
   "compilerOptions": {
     "module": "commonjs",
-    "lib": ["es2018"],
-    "target": "ES2018",
+    "lib": ["es2022"],
+    "target": "ES2022",
     "allowJs": true,
     "checkJs": true,
-    "moduleResolution": "node",
+    "moduleResolution": "node16",
     "resolveJsonModule": true,
     "esModuleInterop": true,
     "noEmit": true,
@@ -20,6 +20,8 @@
   ],
   "exclude": [
     "android",
+    "allure-report",
+    "allure-results",
     "coverage",
     "ios",
     "test"

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1-javadoc.jar.md5

@@ -1 +0,0 @@
-0780bd77755cae9d89bc326c195d8eb1

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1-javadoc.jar.sha1

@@ -1 +0,0 @@
-9168cee63c5b35da49de50f34c9c41ad5fc0c449

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1-javadoc.jar.sha256

@@ -1 +0,0 @@
-d3b9a104400c779bb7a1e70546c33abd374cf8eeb55ac5b4098a6c54737d0411

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1-javadoc.jar.sha512

@@ -1 +0,0 @@
-ba86f8d9c3163eefc5d13776f4288708d96e00c83bea5527bbbd1a1c2027c79e6d44d19a8c85959bc48f3b7311c2969c96ea4d26569adb18f8f90fb56f7b558e

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1-sources.jar.md5

@@ -1 +0,0 @@
-70745a3a7e09a2682c6a8902a413bff1

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1-sources.jar.sha1

@@ -1 +0,0 @@
-1da2776757d2a28dc4c3e73a7b469b4d92dee9c8

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1-sources.jar.sha256

@@ -1 +0,0 @@
-931d28ebdf7b9bceb5ad5ec36cd7b3696f91e6955579f505c311c1b5334fb57a

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1-sources.jar.sha512

@@ -1 +0,0 @@
-4750dd9cab79b2cab696c552101c755c609e6a4bfa5ab75d742004cbf4fe5bfdbc87b36096e1145559c5a5f65b6216181b6690efaa73016df660a4865d774782

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1.aar.md5

@@ -1 +0,0 @@
-c90cc85230997f3663da1741d1eabc0e

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1.aar.sha1

@@ -1 +0,0 @@
-0e17ccf18dbb2e0295e72b77cae67275591d956a

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1.aar.sha256

@@ -1 +0,0 @@
-00528b5d136df63bc93f55c2f890a1ad69179f341663310b520227f6b7aad63b

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1.aar.sha512

@@ -1 +0,0 @@
-37984e2491685ede0325d74f460ad2a13dc55bd906124e0932fa09e29e2394a91cbbf691aa854848503113f04dfe148a483e616c736918f6e0077af6aa311a31

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1.pom.md5

@@ -1 +0,0 @@
-1ebd5db42295ac98e2151fc5c24b8903

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1.pom.sha1

@@ -1 +0,0 @@
-e5614e7546e21d6757c4b338efc93e6b6a0a4bf6

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1.pom.sha256

@@ -1 +0,0 @@
-b38cc093a239ee307a854533ad801c3ee7548c01519f7964da7d0285486a09a8

package/Detox-android/com/wix/detox/20.12.1/detox-20.12.1.pom.sha512

@@ -1 +0,0 @@
-5f94777c0cbddf9a24e2f661fc598fec79e71d555107020bcbb366a728307e990ae7205b210346562faa08fc00774ff3a9336e8fd185c80bd1fa9ebb4d2fa8c5

package/src/devices/DeviceRegistry.js

@@ -1,176 +0,0 @@
-// @ts-nocheck
-const fs = require('fs-extra');
-const _ = require('lodash');
-
-const ExclusiveLockfile = require('../utils/ExclusiveLockfile');
-const environment = require('../utils/environment');
-const safeAsync = require('../utils/safeAsync');
-
-const readOptions = {
-  encoding: 'utf8',
-};
-
-const FIELD_NAME_ID = 'id';
-const FIELD_NAME_DATA = 'data';
-
-class DevicesList {
-  constructor(devices) {
-    this._devices = Object.freeze(devices);
-  }
-
-  /**
-   * @returns {{id: string, data: *?}[]}
-   */
-  get rawDevices() {
-    return this._devices;
-  }
-
-  /**
-   * @param {string} deviceId
-   * @returns {boolean}
-   */
-  includes(deviceId) {
-    return DevicesList._includes(deviceId, this._devices);
-  }
-
-  static _includes(deviceId, devices) {
-    return _.some(devices, [FIELD_NAME_ID, deviceId]);
-  }
-}
-
-class DeviceRegistry {
-  constructor({ lockfilePath }) {
-    /***
-     * @private
-     * @type {string}
-     */
-    this._lockfilePath = lockfilePath;
-
-    /***
-     * @protected
-     * @type {ExclusiveLockfile}
-     */
-    this._lockfile = new ExclusiveLockfile(lockfilePath, {
-      getInitialState: this._getInitialLockFileState.bind(this),
-      readOptions,
-    });
-  }
-
-  async reset() {
-    await this._lockfile.exclusively(() => {
-      const empty = this._getInitialLockFileState();
-      this._lockfile.write(empty);
-    });
-  }
-
-  /***
-   * @param {string|Function} getDeviceId
-   * @param {*?} data
-   * @returns {Promise<string>}
-   */
-  async allocateDevice(getDeviceId, data) {
-    return this._lockfile.exclusively(async () => {
-      const deviceId = await safeAsync(getDeviceId);
-      this._registerDevice(deviceId, data);
-      return deviceId;
-    });
-  }
-
-  /***
-   * @param {string|Function} getDeviceId
-   * @returns {Promise<void>}
-   */
-  async disposeDevice(getDeviceId) {
-    await this._lockfile.exclusively(async () => {
-      const deviceId = await safeAsync(getDeviceId);
-      if (deviceId) {
-        this._unregisterDevice(deviceId);
-      }
-    });
-  }
-
-  /***
-   * @param {string} deviceId
-   * @returns {boolean}
-   */
-  includes(deviceId) {
-    const devices = this._lockfile.read();
-    return DevicesList._includes(deviceId, devices);
-  }
-
-  /***
-   * @returns {DevicesList}
-   */
-  getRegisteredDevices() {
-    const devices = this._lockfile.read();
-    return new DevicesList(devices);
-  }
-
-  /***
-   * @returns {DevicesList}
-   */
-  async readRegisteredDevices() {
-    let result = null;
-    await this._lockfile.exclusively(() => {
-      result = this.getRegisteredDevices();
-    });
-    return result;
-  }
-
-  /***
-   * @returns {DevicesList}
-   */
-  readRegisteredDevicesUNSAFE() {
-    const contents = fs.readFileSync(this._lockfilePath, readOptions);
-    const devices = JSON.parse(contents);
-    return new DevicesList(devices);
-  }
-
-  /***
-   * @private
-   */
-  _getInitialLockFileState() {
-    return [];
-  }
-
-  /**
-   * @private
-   */
-  _registerDevice(deviceId, data) {
-    const state = this._lockfile.read();
-    let newState = _.reject(state, [FIELD_NAME_ID, deviceId]);
-
-    const device = {};
-    device[FIELD_NAME_ID] = deviceId;
-
-    if (data) {
-      device[FIELD_NAME_DATA] = data;
-    }
-
-    newState = _.concat(newState, device);
-    this._lockfile.write(newState);
-  }
-
-  /**
-   * @private
-   */
-  _unregisterDevice(deviceId) {
-    const state = this._lockfile.read();
-    const newState = _.reject(state, [FIELD_NAME_ID, deviceId]);
-    this._lockfile.write(newState);
-  }
-
-  static forIOS() {
-    return new DeviceRegistry({
-      lockfilePath: environment.getDeviceLockFilePathIOS(),
-    });
-  }
-
-  static forAndroid() {
-    return new DeviceRegistry({
-      lockfilePath: environment.getDeviceLockFilePathAndroid(),
-    });
-  }
-}
-
-module.exports = DeviceRegistry;

package/src/devices/allocation/drivers/AllocationDriverBase.js

@@ -1,30 +0,0 @@
-/* eslint @typescript-eslint/no-unused-vars: ["error", { "args": "none" }] */
-// @ts-nocheck
-
-/**
- * @typedef DeallocOptions
- * @property shutdown { Boolean }
- */
-
-class AllocationDriverBase {
-  /**
-   * @param deviceConfig { Object }
-   * @return {Promise<DeviceCookie>}
-   */
-  async allocate(deviceConfig) {}
-
-  /**
-   * @param {DeviceCookie} deviceCookie
-   * @return {Promise<void>}
-   */
-  async postAllocate(deviceCookie) {}
-
-  /**
-   * @param cookie { DeviceCookie }
-   * @param options { DeallocOptions }
-   * @return {Promise<void>}
-   */
-  async free(cookie, options) {}
-}
-
-module.exports = AllocationDriverBase;

package/src/devices/allocation/drivers/android/attached/AttachedAndroidLauncher.js

@@ -1,13 +0,0 @@
-const DeviceLauncher = require('../../../../common/drivers/DeviceLauncher');
-
-class AttachedAndroidLauncher extends DeviceLauncher {
-  constructor(eventEmitter) {
-    super(eventEmitter);
-  }
-
-  notifyLaunchCompleted(adbName) {
-    return super._notifyBootEvent(adbName, 'device', false);
-  }
-}
-
-module.exports = AttachedAndroidLauncher;