appium 2.0.0-beta.55 → 2.0.0-beta.56

package/lib/appium.js

@@ -10,6 +10,7 @@ import {
   DELETE_SESSION_COMMAND,
   GET_STATUS_COMMAND,
   promoteAppiumOptions,
+  promoteAppiumOptionsForObject,
 } from '@appium/base-driver';
 import AsyncLock from 'async-lock';
 import {parseCapsForInnerDriver, pullSettings} from './utils';
@@ -44,7 +45,7 @@ class AppiumDriver extends DriverCore {
    * It is not recommended to access this property directly from the outside
    * @type {Record<string,ExternalDriver>}
    */
-  sessions = {};
+  sessions;
 
   /**
    * Access to pending drivers list must be guarded with a Semaphore, because
@@ -52,14 +53,14 @@ class AppiumDriver extends DriverCore {
    * It is not recommended to access this property directly from the outside
    * @type {Record<string,ExternalDriver[]>}
    */
-  pendingDrivers = {};
+  pendingDrivers;
 
   /**
    * Note that {@linkcode AppiumDriver} has no `newCommandTimeout` method.
    * `AppiumDriver` does not set and observe its own timeouts; individual
    * sessions (managed drivers) do instead.
    */
-  newCommandTimeoutMs = 0;
+  newCommandTimeoutMs;
 
   /**
    * List of active plugins
@@ -71,13 +72,13 @@ class AppiumDriver extends DriverCore {
    * map of sessions to actual plugin instances per session
    * @type {Record<string,InstanceType<PluginClass>[]>}
    */
-  sessionPlugins = {};
+  sessionPlugins;
 
   /**
    * some commands are sessionless, so we need a set of plugins for them
    * @type {InstanceType<PluginClass>[]}
    */
-  sessionlessPlugins = [];
+  sessionlessPlugins;
 
   /** @type {DriverConfig} */
   driverConfig;
@@ -85,7 +86,11 @@ class AppiumDriver extends DriverCore {
   /** @type {AppiumServer} */
   server;
 
-  desiredCapConstraints = desiredCapabilityConstraints;
+  /**
+   * @type {AppiumDriverConstraints}
+   * @readonly
+   */
+  desiredCapConstraints;
 
   /** @type {DriverOpts} */
   args;
@@ -105,6 +110,13 @@ class AppiumDriver extends DriverCore {
     super(opts);
 
     this.args = {...opts};
+    this.sessions = {};
+    this.pendingDrivers = {};
+    this.newCommandTimeoutMs = 0;
+    this.pluginClasses = new Map();
+    this.sessionPlugins = {};
+    this.sessionlessPlugins = [];
+    this.desiredCapConstraints = desiredCapabilityConstraints;
 
     // allow this to happen in the background, so no `await`
     (async () => {
@@ -243,7 +255,7 @@ class AppiumDriver extends DriverCore {
         jsonwpCaps,
         promoteAppiumOptions(w3cCapabilities),
         this.desiredCapConstraints,
-        defaultCapabilities
+        defaultCapabilities ? promoteAppiumOptionsForObject(defaultCapabilities) : undefined
       );
 
       const {desiredCaps, processedJsonwpCapabilities, processedW3CCapabilities} =

package/lib/cli/extension-command.js

@@ -693,7 +693,7 @@ class ExtensionCommand {
       );
     }
 
-    if (!(scriptName in extScripts)) {
+    if (!(scriptName in /** @type {Record<string,string>} */ (extScripts))) {
       throw this._createFatalError(
         `The ${this.type} named '${installSpec}' does not support the script: '${scriptName}'`
       );

package/lib/extension/extension-config.js

@@ -435,7 +435,7 @@ export class ExtensionConfig {
   /**
    * @param {string} extName
    * @param {ExtManifest<ExtType>} extManifest
-   * @param {ExtensionConfigMutationOpts} [opts]
+   * @param {ExtensionConfigMutationOpts} opts
    * @returns {Promise<void>}
    */
   async addExtension(extName, extManifest, {write = true} = {}) {
@@ -448,7 +448,7 @@ export class ExtensionConfig {
   /**
    * @param {ExtName<ExtType>} extName
    * @param {ExtManifest<ExtType>} extManifest
-   * @param {ExtensionConfigMutationOpts} [opts]
+   * @param {ExtensionConfigMutationOpts} opts
    * @returns {Promise<void>}
    */
   async updateExtension(extName, extManifest, {write = true} = {}) {
@@ -465,7 +465,7 @@ export class ExtensionConfig {
    * Remove an extension from the list of installed extensions, and optionally avoid a write to the manifest file.
    *
    * @param {ExtName<ExtType>} extName
-   * @param {ExtensionConfigMutationOpts} [opts]
+   * @param {ExtensionConfigMutationOpts} opts
    * @returns {Promise<void>}
    */
   async removeExtension(extName, {write = true} = {}) {

package/lib/schema/arg-spec.js

@@ -94,7 +94,7 @@ export class ArgSpec {
    * The _constructor_ is private. Use {@link ArgSpec.create} instead.
    * @private
    * @param {string} name
-   * @param {ArgSpecOptions<D>} [opts]
+   * @param {ArgSpecOptions<D>} opts
    */
   constructor(name, {extType, extName, dest, defaultValue} = {}) {
     // we must normalize the extension name to fit into our convention for CLI

package/lib/schema/schema.js

@@ -41,7 +41,9 @@ export class RoachHotelMap extends Map {
 /**
  * Extensions that an extension schema file can have.
  */
-export const ALLOWED_SCHEMA_EXTENSIONS = new Set(['.json', '.js', '.cjs']);
+export const ALLOWED_SCHEMA_EXTENSIONS = Object.freeze(
+  new Set(/** @type {AllowedSchemaExtension[]} */ (['.json', '.js', '.cjs']))
+);
 
 const SCHEMA_KEY = '$schema';
 
@@ -57,44 +59,39 @@ class AppiumSchema {
    * An "argument" is a CLI argument or a config property.
    *
    * Used to provide easy lookups of argument metadata when converting between different representations of those arguments.
-   * @private
    * @type {RoachHotelMap<string,ArgSpec>}
    */
-  _argSpecs = new RoachHotelMap();
+  #argSpecs = new RoachHotelMap();
 
   /**
    * A map of extension types to extension names to schema objects.
    *
    * This data structure is used to ensure there are no naming conflicts. The schemas
    * are stored here in memory until the instance is _finalized_.
-   * @private
    * @type {Record<ExtensionType,Map<string,SchemaObject>>}
    */
-  _registeredSchemas = {[DRIVER_TYPE]: new Map(), [PLUGIN_TYPE]: new Map()};
+  #registeredSchemas = {[DRIVER_TYPE]: new Map(), [PLUGIN_TYPE]: new Map()};
 
   /**
    * Ajv instance
    *
-   * @private
    * @type {Ajv}
    */
-  _ajv;
+  #ajv;
 
   /**
    * Singleton instance.
-   * @private
    * @type {AppiumSchema}
    */
-  static _instance;
+  static #instance;
 
   /**
    * Lookup of schema IDs to finalized schemas.
    *
    * This does not include references, but rather the root schemas themselves.
-   * @private
    * @type {Record<string,StrictSchemaObject>?}
    */
-  _finalizedSchemas = null;
+  #finalizedSchemas = null;
 
   /**
    * Initializes Ajv, adds standard formats and our custom keywords.
@@ -102,7 +99,7 @@ class AppiumSchema {
    * @private
    */
   constructor() {
-    this._ajv = AppiumSchema._instantiateAjv();
+    this.#ajv = AppiumSchema._instantiateAjv();
   }
 
   /**
@@ -113,9 +110,9 @@ class AppiumSchema {
    * @returns {AppiumSchema}
    */
   static create() {
-    if (!AppiumSchema._instance) {
+    if (!AppiumSchema.#instance) {
       const instance = new AppiumSchema();
-      AppiumSchema._instance = instance;
+      AppiumSchema.#instance = instance;
       _.bindAll(instance, [
         'finalize',
         'flatten',
@@ -133,7 +130,7 @@ class AppiumSchema {
       ]);
     }
 
-    return AppiumSchema._instance;
+    return AppiumSchema.#instance;
   }
 
   /**
@@ -145,7 +142,7 @@ class AppiumSchema {
    * @returns {boolean} If registered
    */
   hasRegisteredSchema(extType, extName) {
-    return this._registeredSchemas[extType].has(extName);
+    return this.#registeredSchemas[extType].has(extName);
   }
 
   /**
@@ -154,11 +151,11 @@ class AppiumSchema {
    * @returns {boolean} If finalized
    */
   isFinalized() {
-    return Boolean(this._finalizedSchemas);
+    return Boolean(this.#finalizedSchemas);
   }
 
   getAllArgSpecs() {
-    return this._argSpecs;
+    return this.#argSpecs;
   }
 
   /**
@@ -182,10 +179,10 @@ class AppiumSchema {
    */
   finalize() {
     if (this.isFinalized()) {
-      return /** @type {NonNullable<typeof this._finalizedSchemas>} */ (this._finalizedSchemas);
+      return /** @type {Record<string,StrictSchemaObject>} */ (this.#finalizedSchemas);
     }
 
-    const ajv = this._ajv;
+    const ajv = this.#ajv;
 
     // Ajv will _mutate_ the schema, so we need to clone it.
     const baseSchema = _.cloneDeep(AppiumConfigJsonSchema);
@@ -205,7 +202,7 @@ class AppiumSchema {
           extName,
         });
         const {arg} = argSpec;
-        this._argSpecs.set(arg, argSpec);
+        this.#argSpecs.set(arg, argSpec);
       }
     };
 
@@ -217,7 +214,7 @@ class AppiumSchema {
     const finalizedSchemas = {};
 
     const finalSchema = _.reduce(
-      this._registeredSchemas,
+      this.#registeredSchemas,
       /**
        * @param {typeof baseSchema} baseSchema
        * @param {Map<string,SchemaObject>} extensionSchemas
@@ -246,7 +243,7 @@ class AppiumSchema {
     finalizedSchemas[APPIUM_CONFIG_SCHEMA_ID] = finalSchema;
     ajv.validateSchema(finalSchema, true);
 
-    this._finalizedSchemas = finalizedSchemas;
+    this.#finalizedSchemas = finalizedSchemas;
     return Object.freeze(finalizedSchemas);
   }
 
@@ -283,18 +280,18 @@ class AppiumSchema {
    * @returns {void}
    */
   reset() {
-    for (const schemaId of Object.keys(this._finalizedSchemas ?? {})) {
-      this._ajv.removeSchema(schemaId);
+    for (const schemaId of Object.keys(this.#finalizedSchemas ?? {})) {
+      this.#ajv.removeSchema(schemaId);
     }
-    this._argSpecs = new RoachHotelMap();
-    this._registeredSchemas = {
+    this.#argSpecs = new RoachHotelMap();
+    this.#registeredSchemas = {
       [DRIVER_TYPE]: new Map(),
       [PLUGIN_TYPE]: new Map(),
     };
-    this._finalizedSchemas = null;
+    this.#finalizedSchemas = null;
 
     // Ajv seems to have an over-eager cache, so we have to dump the object entirely.
-    this._ajv = AppiumSchema._instantiateAjv();
+    this.#ajv = AppiumSchema._instantiateAjv();
   }
 
   /**
@@ -318,14 +315,14 @@ class AppiumSchema {
     }
     const normalizedExtName = _.kebabCase(extName);
     if (this.hasRegisteredSchema(extType, normalizedExtName)) {
-      if (_.isEqual(this._registeredSchemas[extType].get(normalizedExtName), schema)) {
+      if (_.isEqual(this.#registeredSchemas[extType].get(normalizedExtName), schema)) {
         return;
       }
       throw new SchemaNameConflictError(extType, extName);
     }
-    this._ajv.validateSchema(schema, true);
+    this.#ajv.validateSchema(schema, true);
 
-    this._registeredSchemas[extType].set(normalizedExtName, schema);
+    this.#registeredSchemas[extType].set(normalizedExtName, schema);
   }
 
   /**
@@ -336,7 +333,7 @@ class AppiumSchema {
    * @returns {ArgSpec|undefined} ArgSpec or `undefined` if not found
    */
   getArgSpec(name, extType, extName) {
-    return this._argSpecs.get(ArgSpec.toArg(name, extType, extName));
+    return this.#argSpecs.get(ArgSpec.toArg(name, extType, extName));
   }
 
   /**
@@ -347,7 +344,7 @@ class AppiumSchema {
    * @returns {boolean} `true` if such an {@link ArgSpec} exists
    */
   hasArgSpec(name, extType, extName) {
-    return this._argSpecs.has(ArgSpec.toArg(name, extType, extName));
+    return this.#argSpecs.has(ArgSpec.toArg(name, extType, extName));
   }
 
   /**
@@ -391,7 +388,7 @@ class AppiumSchema {
 
     /** @type {DefaultValues<Flattened>} */
     const retval = {};
-    return [...this._argSpecs.values()].reduce(reducer, retval);
+    return [...this.#argSpecs.values()].reduce(reducer, retval);
   }
 
   /**
@@ -405,7 +402,7 @@ class AppiumSchema {
     if (!this.isFinalized()) {
       throw new SchemaFinalizationError();
     }
-    const specs = [...this._argSpecs.values()].filter(
+    const specs = [...this.#argSpecs.values()].filter(
       (spec) => spec.extType === extType && spec.extName === extName
     );
     return specs.reduce((defaults, {defaultValue, rawDest}) => {
@@ -506,7 +503,7 @@ class AppiumSchema {
    * @returns {import('ajv').ValidateFunction}
    */
   _getValidator(id = APPIUM_CONFIG_SCHEMA_ID) {
-    const validator = this._ajv.getSchema(id);
+    const validator = this.#ajv.getSchema(id);
     if (!validator) {
       if (id === APPIUM_CONFIG_SCHEMA_ID) {
         throw new SchemaFinalizationError();
@@ -532,11 +529,13 @@ class AppiumSchema {
 
   /**
    * Returns `true` if `filename`'s file extension is allowed (in {@link ALLOWED_SCHEMA_EXTENSIONS}).
-   * @param {string} filename
+   * @param {import('type-fest').LiteralUnion<AllowedSchemaExtension, string>} filename
    * @returns {boolean}
    */
   static isAllowedSchemaFileExtension(filename) {
-    return ALLOWED_SCHEMA_EXTENSIONS.has(path.extname(filename));
+    return ALLOWED_SCHEMA_EXTENSIONS.has(
+      /** @type {AllowedSchemaExtension} */ (path.extname(filename))
+    );
   }
 
   /**
@@ -720,3 +719,7 @@ export const {isAllowedSchemaFileExtension} = AppiumSchema;
  * @template {boolean|undefined} Flattened
  * @typedef {Record<string,Flattened extends true ? ArgSpecDefaultValue : ArgSpecDefaultValue | NestedArgSpecDefaultValue>} DefaultValues
  */
+
+/**
+ * @typedef {'.json'|'.js'|'.cjs'} AllowedSchemaExtension
+ */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "appium",
-  "version": "2.0.0-beta.55",
+  "version": "2.0.0-beta.56",
   "description": "Automation for Apps.",
   "keywords": [
     "automation",
@@ -59,12 +59,12 @@
     "test:unit": "mocha \"./test/unit/**/*.spec.js\""
   },
   "dependencies": {
-    "@appium/base-driver": "^9.3.1",
-    "@appium/base-plugin": "^2.2.1",
-    "@appium/docutils": "^0.2.1",
-    "@appium/schema": "^0.2.4",
-    "@appium/support": "^3.1.5",
-    "@appium/types": "^0.9.1",
+    "@appium/base-driver": "^9.3.2",
+    "@appium/base-plugin": "^2.2.2",
+    "@appium/docutils": "^0.2.2",
+    "@appium/schema": "^0.2.5",
+    "@appium/support": "^3.1.6",
+    "@appium/types": "^0.10.0",
     "@sidvind/better-ajv-errors": "2.1.0",
     "@types/argparse": "2.0.10",
     "@types/bluebird": "3.5.38",
@@ -77,7 +77,7 @@
     "argparse": "2.0.1",
     "async-lock": "1.4.0",
     "asyncbox": "2.9.4",
-    "axios": "1.2.3",
+    "axios": "1.3.4",
     "bluebird": "3.7.2",
     "cross-env": "7.0.3",
     "find-up": "5.0.0",
@@ -92,7 +92,7 @@
     "semver": "7.3.8",
     "source-map-support": "0.5.21",
     "teen_process": "2.0.2",
-    "type-fest": "3.5.6",
+    "type-fest": "3.6.0",
     "winston": "3.8.2",
     "wrap-ansi": "7.0.0",
     "yaml": "2.2.1"
@@ -105,7 +105,7 @@
     "access": "public",
     "tag": "next"
   },
-  "gitHead": "29a5bdac9aead8eca0329d5672c5e2afc2bdf86c",
+  "gitHead": "7b4935632222123a4fa7422461f6312f1f0dfbe4",
   "typedoc": {
     "entryPoint": "./lib/main.js"
   }