appium 2.0.0-beta.3 → 2.0.0-beta.34

package/lib/extension/index.js

@@ -0,0 +1,102 @@
+
+import _ from 'lodash';
+import { USE_ALL_PLUGINS } from '../constants';
+import log from '../logger';
+import { DriverConfig } from './driver-config';
+import { Manifest } from './manifest';
+import { PluginConfig } from './plugin-config';
+
+/**
+ * Loads extensions and creates `ExtensionConfig` instances.
+ *
+ * - Reads the manifest file, creating if necessary
+ * - Using the parsed extension data, creates/gets the `ExtensionConfig` subclass instances
+ * - Returns these instances
+ *
+ * If `appiumHome` is needed, use `resolveAppiumHome` from the `env` module in `@appium/support`.
+ * @param {string} appiumHome
+ * @returns {Promise<ExtensionConfigs>}
+ */
+export async function loadExtensions (appiumHome) {
+  const manifest = Manifest.getInstance(appiumHome);
+  const {drivers, plugins} = await manifest.read();
+  const driverConfig =
+    DriverConfig.getInstance(manifest) ??
+    DriverConfig.create(manifest, {extData: drivers});
+  const pluginConfig =
+    PluginConfig.getInstance(manifest) ??
+    PluginConfig.create(manifest, {extData: plugins});
+  return {driverConfig, pluginConfig};
+}
+
+/**
+ * Find any plugin name which has been installed, and which has been requested for activation by
+ * using the --use-plugins flag, and turn each one into its class, so we can send them as objects
+ * to the server init. We also want to send/assign them to the umbrella driver so it can use them
+ * to wrap command execution
+ *
+ * @param {import('./plugin-config').PluginConfig} pluginConfig - a plugin extension config
+ * @param {string[]} usePlugins
+ * @returns {import('../../types').PluginClass[]}
+ */
+export function getActivePlugins (pluginConfig, usePlugins = []) {
+  return _.compact(
+    Object.keys(pluginConfig.installedExtensions)
+      .filter(
+        (pluginName) =>
+          _.includes(usePlugins, pluginName) ||
+          (usePlugins.length === 1 && usePlugins[0] === USE_ALL_PLUGINS),
+      )
+      .map((pluginName) => {
+        try {
+          log.info(`Attempting to load plugin ${pluginName}...`);
+          const PluginClass = pluginConfig.require(pluginName);
+
+          PluginClass.pluginName = pluginName; // store the plugin name on the class so it can be used later
+          return PluginClass;
+        } catch (err) {
+          log.error(
+            `Could not load plugin '${pluginName}', so it will not be available. Error ` +
+              `in loading the plugin was: ${err.message}`,
+          );
+          log.debug(err.stack);
+        }
+      }),
+  );
+}
+
+/**
+ * Find any driver name which has been installed, and turn each one into its class, so we can send
+ * them as objects to the server init in case they need to add methods/routes or update the server.
+ * If the --drivers flag was given, this method only loads the given drivers.
+ *
+ * @param {import('./driver-config').DriverConfig} driverConfig - a driver extension config
+ * @param {string[]} [useDrivers] - optional list of drivers to load
+ */
+export function getActiveDrivers (driverConfig, useDrivers = []) {
+  return _.compact(
+    Object.keys(driverConfig.installedExtensions)
+      .filter(
+        (driverName) =>
+          _.includes(useDrivers, driverName) || useDrivers.length === 0,
+      )
+      .map((driverName) => {
+        try {
+          log.info(`Attempting to load driver ${driverName}...`);
+          return driverConfig.require(driverName);
+        } catch (err) {
+          log.error(
+            `Could not load driver '${driverName}', so it will not be available. Error ` +
+              `in loading the driver was: ${err.message}`,
+          );
+          log.debug(err.stack);
+        }
+      }),
+  );
+}
+
+/**
+ * @typedef ExtensionConfigs
+ * @property {DriverConfig} driverConfig
+ * @property {PluginConfig} pluginConfig
+ */

package/lib/extension/manifest.js

@@ -0,0 +1,484 @@
+/**
+ * Module containing {@link Manifest} which handles reading & writing of extension config files.
+ */
+
+import { env, fs } from '@appium/support';
+import _ from 'lodash';
+import path from 'path';
+import YAML from 'yaml';
+import { DRIVER_TYPE, PLUGIN_TYPE } from '../constants';
+import log from '../logger';
+import { INSTALL_TYPE_NPM } from './extension-config';
+import { packageDidChange } from './package-changed';
+
+/**
+ * Default depth to search in directory tree for whatever it is we're looking for.
+ *
+ * It's 4 because smaller numbers didn't work.
+ */
+const DEFAULT_SEARCH_DEPTH = 4;
+
+/**
+ * Default options for {@link findExtensions}.
+ * @type {Readonly<import('klaw').Options>}
+ */
+const DEFAULT_FIND_EXTENSIONS_OPTS = Object.freeze({
+  depthLimit: DEFAULT_SEARCH_DEPTH,
+  /* istanbul ignore next */
+  filter: (filepath) => !path.basename(filepath).startsWith('.'),
+});
+
+/**
+ * Current configuration schema revision!
+ */
+const CONFIG_SCHEMA_REV = 2;
+
+/**
+ * The name of the prop (`drivers`) used in `extensions.yaml` for drivers.
+ * @type {`${typeof DRIVER_TYPE}s`}
+ */
+const CONFIG_DATA_DRIVER_KEY = `${DRIVER_TYPE}s`;
+
+/**
+ * The name of the prop (`plugins`) used in `extensions.yaml` for plugins.
+ * @type {`${typeof PLUGIN_TYPE}s`}
+ */
+const CONFIG_DATA_PLUGIN_KEY = `${PLUGIN_TYPE}s`;
+
+/**
+ * @type {Readonly<ManifestData>}
+ */
+const INITIAL_MANIFEST_DATA = Object.freeze({
+  [CONFIG_DATA_DRIVER_KEY]: Object.freeze({}),
+  [CONFIG_DATA_PLUGIN_KEY]: Object.freeze({}),
+  schemaRev: CONFIG_SCHEMA_REV,
+});
+
+/**
+ * Given a `package.json` return `true` if it represents an Appium Extension (either a driver or plugin).
+ *
+ * The `package.json` must have an `appium` property which is an object.
+ * @param {any} value
+ * @returns {value is ExtPackageJson<ExtensionType>}
+ */
+function isExtension (value) {
+  return (
+    _.isPlainObject(value) &&
+    _.isPlainObject(value.appium) &&
+    _.isString(value.name) &&
+    _.isString(value.version)
+  );
+}
+/**
+ * Given a `package.json`, return `true` if it represents an Appium Driver.
+ *
+ * To be considered a driver, a `package.json` must have a fields
+ * `appium.driverName`, `appium.automationName` and `appium.platformNames`.
+ * @param {any} value - Value to test
+ * @returns {value is ExtPackageJson<DriverType>}
+ */
+function isDriver (value) {
+  return (
+    isExtension(value) &&
+    _.isString(_.get(value, 'appium.driverName')) &&
+    _.isString(_.get(value, 'appium.automationName')) &&
+    _.isArray(_.get(value, 'appium.platformNames'))
+  );
+}
+
+/**
+ * Given a `package.json`, return `true` if it represents an Appium Plugin.
+ *
+ * To be considered a plugin, a `package.json` must have an `appium.pluginName` field.
+ * @param {any} value - Value to test
+ * @returns {value is ExtPackageJson<PluginType>}
+ */
+function isPlugin (value) {
+  return isExtension(value) && _.isString(_.get(value, 'appium.pluginName'));
+}
+
+/**
+ * Handles reading & writing of extension config files.
+ *
+ * Only one instance of this class exists per value of `APPIUM_HOME`.
+ */
+export class Manifest {
+  /**
+   * The entire contents of a parsed YAML extension config file.
+   *
+   * Contains proxies for automatic persistence on disk
+   * @type {ManifestData}
+   * @private
+   */
+  _data;
+
+  /**
+   * Path to `APPIUM_HOME`.
+   * @private
+   * @type {Readonly<string>}
+   */
+  _appiumHome;
+
+  /**
+   * Path to `extensions.yaml`
+   * @type {string}
+   * Not set until {@link Manifest.read} is called.
+   */
+  _manifestPath;
+
+  /**
+   * Helps avoid writing multiple times.
+   *
+   * If this is `undefined`, calling {@link Manifest.write} will cause it to be
+   * set to a `Promise`. When the call to `write()` is complete, the `Promise`
+   * will resolve and then this value will be set to `undefined`.  Concurrent calls
+   * made while this value is a `Promise` will return the `Promise` itself.
+   * @private
+   * @type {Promise<boolean>|undefined}
+   */
+  _writing;
+
+  /**
+   * Helps avoid reading multiple times.
+   *
+   * If this is `undefined`, calling {@link Manifest.read} will cause it to be
+   * set to a `Promise`. When the call to `read()` is complete, the `Promise`
+   * will resolve and then this value will be set to `undefined`.  Concurrent calls
+   * made while this value is a `Promise` will return the `Promise` itself.
+   * @private
+   * @type {Promise<void>|undefined}
+   */
+  _reading;
+
+  /**
+   * Sets internal data to a fresh clone of {@link INITIAL_MANIFEST_DATA}
+   *
+   * Use {@link Manifest.getInstance} instead.
+   * @param {string} appiumHome
+   * @private
+   */
+  constructor (appiumHome) {
+    this._appiumHome = appiumHome;
+    this._data = _.cloneDeep(INITIAL_MANIFEST_DATA);
+  }
+
+  /**
+   * Returns a new or existing {@link Manifest} instance, based on the value of `appiumHome`.
+   *
+   * Maintains one instance per value of `appiumHome`.
+   * @param {string} appiumHome - Path to `APPIUM_HOME`
+   * @returns {Manifest}
+   */
+  static getInstance = _.memoize(function _getInstance (
+    appiumHome,
+  ) {
+    return new Manifest(appiumHome);
+  });
+
+  /**
+   * Searches `APPIUM_HOME` for installed extensions and adds them to the manifest.
+   * @param {SyncWithInstalledExtensionsOpts} opts
+   * @returns {Promise<boolean>} `true` if any extensions were added, `false` otherwise.
+   */
+  async syncWithInstalledExtensions ({depthLimit = DEFAULT_SEARCH_DEPTH} = {}) {
+    const walkOpts = _.defaults({depthLimit}, DEFAULT_FIND_EXTENSIONS_OPTS);
+    // this could be parallelized, but we can't use fs.walk as an async iterator
+    let didChange = false;
+    for await (const {stats, path: filepath} of fs.walk(
+      this._appiumHome,
+      walkOpts,
+    )) {
+      if (filepath !== this._appiumHome && stats.isDirectory()) {
+        try {
+          const pkg = await env.readPackageInDir(filepath);
+          if (pkg && isExtension(pkg)) {
+            // it's possible that this extension already exists in the manifest,
+            // so only update `didChange` if it's new.
+            const added = this.addExtensionFromPackage(
+              pkg,
+              path.join(filepath, 'package.json'),
+            );
+            didChange = didChange || added;
+          }
+        } catch {}
+      }
+    }
+    return didChange;
+  }
+
+  /**
+   * Returns `true` if driver with name `name` is registered.
+   * @param {string} name - Driver name
+   * @returns {boolean}
+   */
+  hasDriver (name) {
+    return Boolean(this._data.drivers[name]);
+  }
+
+  /**
+   * Returns `true` if plugin with name `name` is registered.
+   * @param {string} name - Plugin name
+   * @returns {boolean}
+   */
+  hasPlugin (name) {
+    return Boolean(this._data.plugins[name]);
+  }
+
+  /**
+   * Given a path to a `package.json`, add it as either a driver or plugin to the manifest.
+   *
+   * Will _not_ overwrite existing entries.
+   * @template {ExtensionType} ExtType
+   * @param {ExtPackageJson<ExtType>} pkgJson
+   * @param {string} pkgPath
+   * @returns {boolean} - `true` upon success, `false` if the extension is already registered.
+   */
+  addExtensionFromPackage (pkgJson, pkgPath) {
+    /**
+     * @type {InternalMetadata}
+     */
+    const internal = {
+      pkgName: pkgJson.name,
+      version: pkgJson.version,
+      installType: INSTALL_TYPE_NPM,
+      installSpec: `${pkgJson.name}@${pkgJson.version}`,
+    };
+
+    if (isDriver(pkgJson)) {
+      if (!this.hasDriver(pkgJson.appium.driverName)) {
+        this.addExtension(DRIVER_TYPE, pkgJson.appium.driverName, {
+          ..._.omit(pkgJson.appium, 'driverName'),
+          ...internal
+        });
+        return true;
+      }
+      return false;
+    } else if (isPlugin(pkgJson)) {
+      if (!this.hasPlugin(pkgJson.appium.pluginName)) {
+        this.addExtension(PLUGIN_TYPE, pkgJson.appium.pluginName, {
+          ..._.omit(pkgJson.appium, 'pluginName'),
+          ...internal,
+        });
+        return true;
+      }
+      return false;
+    } else {
+      throw new TypeError(
+        `The extension in ${path.dirname(
+          pkgPath,
+        )} is neither a valid driver nor a valid plugin.`,
+      );
+    }
+  }
+
+  /**
+   * Adds an extension to the manifest as was installed by the `appium` CLI.  The
+   * `extData`, `extType`, and `extName` have already been determined.
+   *
+   * See {@link Manifest.addExtensionFromPackage} for adding an extension from an on-disk package.
+   * @template {ExtensionType} ExtType
+   * @param {ExtType} extType - `driver` or `plugin`
+   * @param {string} extName - Name of extension
+   * @param {ExtManifest<ExtType>} extData - Extension metadata
+   * @returns {void}
+   */
+  addExtension (extType, extName, extData) {
+    this._data[`${extType}s`][extName] = extData;
+  }
+
+  /**
+   * Returns the APPIUM_HOME path
+   */
+  get appiumHome () {
+    return this._appiumHome;
+  }
+
+  /**
+   * Returns the path to the manifest file
+   */
+  get manifestPath () {
+    return this._manifestPath;
+  }
+
+  /**
+   * Returns extension data for a particular type.
+   *
+   * @template {ExtensionType} ExtType
+   * @param {ExtType} extType
+   * @returns {ExtRecord<ExtType>}
+   */
+  getExtensionData (extType) {
+    return this._data[/** @type {string} */ (`${extType}s`)];
+  }
+
+  /**
+   * Reads manifest from disk and _overwrites_ the internal data.
+   *
+   * If the manifest does not exist on disk, an {@link INITIAL_MANIFEST_DATA "empty"} manifest file will be created.
+   *
+   * If `APPIUM_HOME` contains a `package.json` with an `appium` dependency, then a hash of the `package.json` will be taken. If this hash differs from the last hash, the contents of `APPIUM_HOME/node_modules` will be scanned for extensions that may have been installed outside of the `appium` CLI.  Any found extensions will be added to the manifest file, and if so, the manifest file will be written to disk.
+   *
+   * Only one read operation should happen at a time.  This is controlled via the {@link Manifest._reading} property.
+   * @returns {Promise<ManifestData>} The data
+   */
+  async read () {
+    if (this._reading) {
+      await this._reading;
+      return this._data;
+    }
+
+    this._reading = (async () => {
+      /** @type {ManifestData} */
+      let data;
+      let isNewFile = false;
+      await this._setManifestPath();
+      try {
+        log.debug(`Reading ${this._manifestPath}...`);
+        const yaml = await fs.readFile(this._manifestPath, 'utf8');
+        data = YAML.parse(yaml);
+      } catch (err) {
+        if (err.code === 'ENOENT') {
+          data = _.cloneDeep(INITIAL_MANIFEST_DATA);
+          isNewFile = true;
+        } else {
+          if (this._manifestPath) {
+            throw new Error(
+              `Appium had trouble loading the extension installation ` +
+                `cache file (${this._manifestPath}). It may be invalid YAML. Specific error: ${err.message}`,
+            );
+          } else {
+            throw new Error(
+              `Appium encountered an unknown problem. Specific error: ${err.message}`,
+            );
+          }
+        }
+      }
+
+      this._data = data;
+      let installedExtensionsChanged = false;
+      if (
+        await env.hasAppiumDependency(this.appiumHome) &&
+        (await packageDidChange(this.appiumHome))
+      ) {
+        installedExtensionsChanged = await this.syncWithInstalledExtensions();
+      }
+
+      if (isNewFile || installedExtensionsChanged) {
+        await this.write();
+      }
+    })();
+    try {
+      await this._reading;
+      return this._data;
+    } finally {
+      this._reading = undefined;
+    }
+  }
+
+  /**
+   * Ensures {@link Manifest._manifestPath} is set.
+   *
+   * Creates the directory if necessary.
+   * @private
+   * @returns {Promise<string>}
+   */
+  async _setManifestPath () {
+    if (!this._manifestPath) {
+      this._manifestPath = await env.resolveManifestPath(this._appiumHome);
+
+      /* istanbul ignore if */
+      if (path.relative(this._appiumHome, this._manifestPath).startsWith('.')) {
+        throw new Error(
+          `Mismatch between location of APPIUM_HOME and manifest file. APPIUM_HOME: ${this.appiumHome}, manifest file: ${this._manifestPath}`,
+        );
+      }
+    }
+
+    return this._manifestPath;
+  }
+
+  /**
+   * Writes the data if it need s writing.
+   *
+   * If the `schemaRev` prop needs updating, the file will be written.
+   *
+   * @todo If this becomes too much of a bottleneck, throttle it.
+   * @returns {Promise<boolean>} Whether the data was written
+   */
+  async write () {
+    if (this._writing) {
+      return this._writing;
+    }
+    this._writing = (async () => {
+      await this._setManifestPath();
+      try {
+        await fs.mkdirp(path.dirname(this._manifestPath));
+      } catch (err) {
+        throw new Error(
+          `Appium could not create the directory for the manifest file: ${path.dirname(
+            this._manifestPath,
+          )}. Original error: ${err.message}`,
+        );
+      }
+      try {
+        await fs.writeFile(
+          this._manifestPath,
+          YAML.stringify(this._data),
+          'utf8',
+        );
+        return true;
+      } catch (err) {
+        throw new Error(
+          `Appium could not write to manifest at ${this._manifestPath} using APPIUM_HOME ${this._appiumHome}. ` +
+            `Please ensure it is writable. Original error: ${err.message}`,
+        );
+      }
+    })();
+    try {
+      return await this._writing;
+    } finally {
+      this._writing = undefined;
+    }
+  }
+}
+
+/**
+ * Type of the string referring to a driver (typically as a key or type string)
+ * @typedef {import('../../types').DriverType} DriverType
+ */
+
+/**
+ * Type of the string referring to a plugin (typically as a key or type string)
+ * @typedef {import('../../types').PluginType} PluginType
+ */
+
+/**
+ * @typedef SyncWithInstalledExtensionsOpts
+ * @property {number} [depthLimit] - Maximum depth to recurse into subdirectories
+ */
+
+/**
+ * @typedef {import('../../types/appium-manifest').ManifestData} ManifestData
+ * @typedef {import('../../types/appium-manifest').InternalMetadata} InternalMetadata
+ */
+
+/**
+ * @template T
+ * @typedef {import('../../types/external-manifest').ExtPackageJson<T>} ExtPackageJson
+ */
+
+/**
+ * @template T
+ * @typedef {import('../../types/appium-manifest').ExtManifest<T>} ExtManifest
+ */
+
+/**
+ * @template T
+ * @typedef {import('../../types/appium-manifest').ExtRecord<T>} ExtRecord
+ */
+
+/**
+ * Either `driver` or `plugin` rn
+ * @typedef {import('../../types').ExtensionType} ExtensionType
+ */
+

package/lib/extension/package-changed.js

@@ -0,0 +1,63 @@
+
+import { fs } from '@appium/support';
+import { isPackageChanged } from 'package-changed';
+import path from 'path';
+import { PKG_HASHFILE_RELATIVE_PATH } from '../constants';
+import log from '../logger';
+
+/**
+ * Determines if extensions have changed, and updates a hash the `package.json` in `appiumHome` if so.
+ *
+ * If they have, we need to sync them with the `extensions.yaml` manifest.
+ *
+ * _Warning: this makes a blocking call to `writeFileSync`._
+ * @param {string} appiumHome
+ * @returns {Promise<boolean>} `true` if `package.json` `appiumHome` changed
+ */
+export async function packageDidChange (appiumHome) {
+  const hashFilename = path.join(appiumHome, PKG_HASHFILE_RELATIVE_PATH);
+
+  // XXX: the types in `package-changed` seem to be wrong.
+
+  /** @type {boolean} */
+  let isChanged;
+  /** @type {() => void} */
+  let writeHash;
+  /** @type {string} */
+  let hash;
+  /** @type {string|undefined} */
+  let oldHash;
+
+  // first mkdirp the target dir.
+  const hashFilenameDir = path.dirname(hashFilename);
+  log.debug(`Creating hash file directory: ${hashFilenameDir}`);
+  try {
+    await fs.mkdirp(hashFilenameDir);
+  } catch (err) {
+    throw new Error(
+      `Appium could not create the directory for hash file: ${hashFilenameDir}. Original error: ${err.message}`,
+    );
+  }
+
+  try {
+    ({isChanged, writeHash, oldHash, hash} = await isPackageChanged({
+      cwd: appiumHome,
+      hashFilename: PKG_HASHFILE_RELATIVE_PATH,
+    }));
+  } catch {
+    return true;
+  }
+
+  if (isChanged) {
+    try {
+      writeHash();
+      log.debug(`Updated hash of ${appiumHome}/package.json from: ${oldHash ?? '(none)'} to: ${hash}`);
+    } catch (err) {
+      throw new Error(
+        `Appium could not write hash file: ${hashFilenameDir}. Original error: ${err.message}`,
+      );
+    }
+  }
+
+  return isChanged;
+}