appium 2.0.0-beta.9 → 2.0.0-rc.2

package/lib/cli/args.js

@@ -1,430 +1,298 @@
-import { DEFAULT_BASE_PATH } from 'appium-base-driver';
-import { parseSecurityFeatures, parseDefaultCaps, parsePluginNames, parseInstallTypes } from './parser-helpers';
-import { INSTALL_TYPES, DEFAULT_APPIUM_HOME, DRIVER_TYPE, PLUGIN_TYPE } from '../extension-config';
-import { DEFAULT_CAPS_ARG, StoreDeprecatedDefaultCapabilityAction } from './argparse-actions';
-
+import _ from 'lodash';
+import {
+  DRIVER_TYPE,
+  PLUGIN_TYPE,
+  EXT_SUBCOMMAND_INSTALL,
+  EXT_SUBCOMMAND_LIST,
+  EXT_SUBCOMMAND_RUN,
+  EXT_SUBCOMMAND_UNINSTALL,
+  EXT_SUBCOMMAND_UPDATE,
+} from '../constants';
+import {INSTALL_TYPES} from '../extension/extension-config';
+import {toParserArgs} from '../schema/cli-args';
 const DRIVER_EXAMPLE = 'xcuitest';
 const PLUGIN_EXAMPLE = 'find_by_image';
-const USE_ALL_PLUGINS = 'all';
-
-// sharedArgs will be added to every subcommand
-const sharedArgs = [
-  [['-ah', '--home', '--appium-home'], {
-    required: false,
-    default: process.env.APPIUM_HOME || DEFAULT_APPIUM_HOME,
-    help: 'The path to the directory where Appium will keep installed drivers, plugins, and any other metadata necessary for its operation',
-    dest: 'appiumHome',
-  }],
-
-  [['--log-filters'], {
-    dest: 'logFilters',
-    default: null,
-    required: false,
-    action: 'store_true',
-    help: 'Set the full path to a JSON file containing one or more log filtering rules',
-  }],
-];
-
-const serverArgs = [
-  [['--shell'], {
-    required: false,
-    default: null,
-    help: 'Enter REPL mode',
-    action: 'store_true',
-    dest: 'shell',
-  }],
-
-  [['--plugins'], {
-    required: false,
-    default: [],
-    help: `A comma-separated list of installed plugin names that should be active for this ` +
-          `server. To activate all plugins, you can use the single string "${USE_ALL_PLUGINS}" ` +
-          `as the value (e.g. --plugins=${USE_ALL_PLUGINS})`,
-    type: parsePluginNames,
-    dest: 'plugins',
-  }],
-
-  [['--allow-cors'], {
-    required: false,
-    default: false,
-    action: 'store_true',
-    help: 'Whether the Appium server should allow web browser connections from any host',
-    dest: 'allowCors',
-  }],
-
-  [['--reboot'], {
-    default: false,
-    dest: 'reboot',
-    action: StoreDeprecatedDefaultCapabilityAction,
-    required: false,
-    help: '(Android-only) reboot emulator after each session and kill it at the end',
-  }],
-
-  [['-a', '--address'], {
-    default: '0.0.0.0',
-    required: false,
-    help: 'IP Address to listen on',
-    dest: 'address',
-  }],
-
-  [['-p', '--port'], {
-    default: 4723,
-    required: false,
-    type: 'int',
-    help: 'port to listen on',
-    dest: 'port',
-  }],
-
-  [['-pa', '--base-path'], {
-    required: false,
-    default: DEFAULT_BASE_PATH,
-    dest: 'basePath',
-    help: 'Base path to use as the prefix for all webdriver routes running ' +
-          `on this server`
-  }],
-
-  [['-ka', '--keep-alive-timeout'], {
-    required: false,
-    default: null,
-    dest: 'keepAliveTimeout',
-    type: 'int',
-    help: 'Number of seconds the Appium server should apply as both the keep-alive timeout ' +
-          'and the connection timeout for all requests. Defaults to 600 (10 minutes).'
-  }],
-
-  [['-ca', '--callback-address'], {
-    required: false,
-    dest: 'callbackAddress',
-    default: null,
-    help: 'callback IP Address (default: same as --address)',
-  }],
-
-  [['-cp', '--callback-port'], {
-    required: false,
-    dest: 'callbackPort',
-    default: null,
-    type: 'int',
-    help: 'callback port (default: same as port)',
-  }],
-
-  [['--session-override'], {
-    default: false,
-    dest: 'sessionOverride',
-    action: 'store_true',
-    required: false,
-    help: 'Enables session override (clobbering)',
-  }],
-
-  [['-g', '--log'], {
-    default: null,
-    dest: 'logFile',
-    required: false,
-    help: 'Also send log output to this file',
-  }],
-
-  [['--log-level'], {
-    choices: [
-      'info', 'info:debug', 'info:info', 'info:warn', 'info:error',
-      'warn', 'warn:debug', 'warn:info', 'warn:warn', 'warn:error',
-      'error', 'error:debug', 'error:info', 'error:warn', 'error:error',
-      'debug', 'debug:debug', 'debug:info', 'debug:warn', 'debug:error',
-    ],
-    default: 'debug',
-    dest: 'loglevel',
-    required: false,
-    help: 'log level; default (console[:file]): debug[:debug]',
-  }],
-
-  [['--log-timestamp'], {
-    default: false,
-    required: false,
-    help: 'Show timestamps in console output',
-    action: 'store_true',
-    dest: 'logTimestamp',
-  }],
-
-  [['--local-timezone'], {
-    default: false,
-    required: false,
-    help: 'Use local timezone for timestamps',
-    action: 'store_true',
-    dest: 'localTimezone',
-  }],
 
-  [['--log-no-colors'], {
-    default: false,
-    required: false,
-    help: 'Do not use colors in console output',
-    action: 'store_true',
-    dest: 'logNoColors',
-  }],
+/**
+ * This is necessary because we pass the array into `argparse`. `argparse` is bad and mutates things. We don't want that.
+ * Bad `argparse`! Bad!
+ */
+const INSTALL_TYPES_ARRAY = [...INSTALL_TYPES];
 
-  [['-G', '--webhook'], {
-    default: null,
-    required: false,
-    dest: 'webhook',
-    help: 'Also send log output to this HTTP listener, for example localhost:9876',
-  }],
-
-  [['--nodeconfig'], {
-    required: false,
-    default: null,
-    dest: 'nodeconfig',
-    help: 'Configuration JSON file to register appium with selenium grid',
-  }],
-
-  [['--chromedriver-port'], {
-    default: null,
-    dest: 'chromeDriverPort',
-    required: false,
-    action: StoreDeprecatedDefaultCapabilityAction,
-    type: 'int',
-    help: 'Port upon which ChromeDriver will run. If not given, Android driver will pick a random available port.',
-  }],
-
-  [['--chromedriver-executable'], {
-    default: null,
-    dest: 'chromedriverExecutable',
-    action: StoreDeprecatedDefaultCapabilityAction,
-    required: false,
-    help: 'ChromeDriver executable full path',
-  }],
-
-  [['--show-config'], {
-    default: false,
-    dest: 'showConfig',
-    action: 'store_true',
-    required: false,
-    help: 'Show info about the appium server configuration and exit',
-  }],
-
-  [['--no-perms-check'], {
-    default: false,
-    dest: 'noPermsCheck',
-    action: 'store_true',
-    required: false,
-    help: 'Bypass Appium\'s checks to ensure we can read/write necessary files',
-  }],
-
-  [['--strict-caps'], {
-    default: false,
-    dest: 'enforceStrictCaps',
-    action: 'store_true',
-    required: false,
-    help: 'Cause sessions to fail if desired caps are sent in that Appium ' +
-          'does not recognize as valid for the selected device',
-  }],
-
-  [['--tmp'], {
-    default: null,
-    dest: 'tmpDir',
-    required: false,
-    help: 'Absolute path to directory Appium can use to manage temporary ' +
-          'files, like built-in iOS apps it needs to move around. On *nix/Mac ' +
-          'defaults to /tmp, on Windows defaults to C:\\Windows\\Temp',
-  }],
-
-  [['--trace-dir'], {
-    default: null,
-    dest: 'traceDir',
-    required: false,
-    help: 'Absolute path to directory Appium use to save ios instruments ' +
-          'traces, defaults to <tmp dir>/appium-instruments',
-  }],
-
-  [['--debug-log-spacing'], {
-    dest: 'debugLogSpacing',
-    default: false,
-    action: 'store_true',
-    required: false,
-    help: 'Add exaggerated spacing in logs to help with visual inspection',
-  }],
-
-  [['--suppress-adb-kill-server'], {
-    dest: 'suppressKillServer',
-    default: false,
-    action: StoreDeprecatedDefaultCapabilityAction,
-    required: false,
-    help: '(Android-only) If set, prevents Appium from killing the adb server instance',
-  }],
-
-  [['--long-stacktrace'], {
-    dest: 'longStacktrace',
-    default: false,
-    required: false,
-    action: 'store_true',
-    help: 'Add long stack traces to log entries. Recommended for debugging only.',
-  }],
-
-  [['--webkit-debug-proxy-port'], {
-    default: 27753,
-    dest: 'webkitDebugProxyPort',
-    action: StoreDeprecatedDefaultCapabilityAction,
-    required: false,
-    type: 'int',
-    help: '(IOS-only) Local port used for communication with ios-webkit-debug-proxy'
-  }],
-
-  [['--webdriveragent-port'], {
-    default: 8100,
-    dest: 'wdaLocalPort',
-    action: StoreDeprecatedDefaultCapabilityAction,
-    required: false,
-    type: 'int',
-    help: '(IOS-only, XCUITest-only) Local port used for communication with WebDriverAgent'
-  }],
-
-  [['-dc', DEFAULT_CAPS_ARG], {
-    dest: 'defaultCapabilities',
-    default: {},
-    type: parseDefaultCaps,
-    required: false,
-    help: 'Set the default desired capabilities, which will be set on each ' +
-          'session unless overridden by received capabilities. For example: ' +
-          '[ \'{"app": "myapp.app", "deviceName": "iPhone Simulator"}\' ' +
-          '| /path/to/caps.json ]'
-  }],
-
-  [['--relaxed-security'], {
-    default: false,
-    dest: 'relaxedSecurityEnabled',
-    action: 'store_true',
-    required: false,
-    help: 'Disable additional security checks, so it is possible to use some advanced features, provided ' +
-          'by drivers supporting this option. Only enable it if all the ' +
-          'clients are in the trusted network and it\'s not the case if a client could potentially ' +
-          'break out of the session sandbox. Specific features can be overridden by ' +
-          'using the --deny-insecure flag',
-  }],
-
-  [['--allow-insecure'], {
-    dest: 'allowInsecure',
-    default: [],
-    type: parseSecurityFeatures,
-    required: false,
-    help: 'Set which insecure features are allowed to run in this server\'s sessions. ' +
-          'Features are defined on a driver level; see documentation for more details. ' +
-          'This should be either a comma-separated list of feature names, or a path to ' +
-          'a file where each feature name is on a line. Note that features defined via ' +
-          '--deny-insecure will be disabled, even if also listed here.',
-  }],
-
-  [['--deny-insecure'], {
-    dest: 'denyInsecure',
-    default: [],
-    type: parseSecurityFeatures,
-    required: false,
-    help: 'Set which insecure features are not allowed to run in this server\'s sessions. ' +
-          'Features are defined on a driver level; see documentation for more details. ' +
-          'This should be either a comma-separated list of feature names, or a path to ' +
-          'a file where each feature name is on a line. Features listed here will not be ' +
-          'enabled even if also listed in --allow-insecure, and even if --relaxed-security ' +
-          'is turned on. For example: execute_driver_script,adb_shell',
-  }],
-];
+/** @type {Set<ExtensionType>} */
+const EXTENSION_TYPES = new Set([DRIVER_TYPE, PLUGIN_TYPE]);
 
 // this set of args works for both drivers and plugins ('extensions')
-const globalExtensionArgs = [
-  [['--json'], {
-    required: false,
-    default: false,
-    action: 'store_true',
-    help: 'Use JSON for output format',
-    dest: 'json'
-  }]
-];
-
-const extensionArgs = {[DRIVER_TYPE]: {}, [PLUGIN_TYPE]: {}};
-
-function makeListArgs (type) {
-  return [
-    ...globalExtensionArgs,
-    [['--installed'], {
+/** @type {ArgumentDefinitions} */
+const globalExtensionArgs = new Map([
+  [
+    ['--json'],
+    {
       required: false,
       default: false,
       action: 'store_true',
-      help: `List only installed ${type}s`,
-      dest: 'showInstalled'
-    }],
-    [['--updates'], {
-      required: false,
-      default: false,
-      action: 'store_true',
-      help: 'Show information about newer versions',
-      dest: 'showUpdates'
-    }]
-  ];
+      help: 'Use JSON for output format',
+      dest: 'json',
+    },
+  ],
+]);
+
+/**
+ * Builds a Record of extension types to a Record of subcommands to their argument definitions
+ */
+const getExtensionArgs = _.memoize(function getExtensionArgs() {
+  const extensionArgs = {};
+  for (const type of EXTENSION_TYPES) {
+    extensionArgs[type] = {
+      [EXT_SUBCOMMAND_LIST]: makeListArgs(type),
+      [EXT_SUBCOMMAND_INSTALL]: makeInstallArgs(type),
+      [EXT_SUBCOMMAND_UNINSTALL]: makeUninstallArgs(type),
+      [EXT_SUBCOMMAND_UPDATE]: makeUpdateArgs(type),
+      [EXT_SUBCOMMAND_RUN]: makeRunArgs(type),
+    };
+  }
+  return /** @type {Record<ExtensionType, Record<import('appium/types').CliExtensionSubcommand,ArgumentDefinitions>>} */ (
+    extensionArgs
+  );
+});
+
+/**
+ * Makes the opts for the `list` subcommand for each extension type.
+ * @param {ExtensionType} type
+ * @returns {ArgumentDefinitions}
+ */
+function makeListArgs(type) {
+  return new Map([
+    ...globalExtensionArgs,
+    [
+      ['--installed'],
+      {
+        required: false,
+        default: false,
+        action: 'store_true',
+        help: `List only installed ${type}s`,
+        dest: 'showInstalled',
+      },
+    ],
+    [
+      ['--updates'],
+      {
+        required: false,
+        default: false,
+        action: 'store_true',
+        help: 'Show information about newer versions',
+        dest: 'showUpdates',
+      },
+    ],
+    [
+      ['--verbose'],
+      {
+        required: false,
+        default: false,
+        action: 'store_true',
+        help: 'Show more information about each extension',
+        dest: 'verbose',
+      },
+    ],
+  ]);
 }
 
-
-function makeInstallArgs (type) {
-  return [
+/**
+ * Makes the opts for the `install` subcommand for each extension type
+ * @param {ExtensionType} type
+ * @returns {ArgumentDefinitions}
+ */
+function makeInstallArgs(type) {
+  return new Map([
     ...globalExtensionArgs,
-    [[type], {
-      type: 'str',
-      help: `Name of the ${type} to install, for example: ` +
-            type === DRIVER_TYPE ? DRIVER_EXAMPLE : PLUGIN_EXAMPLE,
-    }],
-    [['--source'], {
-      required: false,
-      default: null,
-      type: parseInstallTypes,
-      help: `Where to look for the ${type} if it is not one of Appium's verified ` +
-            `${type}s. Possible values: ${JSON.stringify(INSTALL_TYPES)}`,
-      dest: 'installType'
-    }],
-    [['--package'], {
-      required: false,
-      default: null,
-      type: 'str',
-      help: `If installing from Git or GitHub, the package name, as defined in the plugin's ` +
-            `package.json file in the "name" field, cannot be determined automatically, and ` +
-            `should be reported here, otherwise the install will probably fail.`,
-      dest: 'packageName',
-    }],
-  ];
+    [
+      [type],
+      {
+        type: 'str',
+        help:
+          `Name of the ${type} to install, for example: ` + type === DRIVER_TYPE
+            ? DRIVER_EXAMPLE
+            : PLUGIN_EXAMPLE,
+      },
+    ],
+    [
+      ['--source'],
+      {
+        required: false,
+        default: null,
+        choices: INSTALL_TYPES_ARRAY,
+        help:
+          `Where to look for the ${type} if it is not one of Appium's verified ` +
+          `${type}s. Possible values: ${INSTALL_TYPES_ARRAY.join(', ')}`,
+        dest: 'installType',
+      },
+    ],
+    [
+      ['--package'],
+      {
+        required: false,
+        default: null,
+        type: 'str',
+        help:
+          `If installing from Git or GitHub, the package name, as defined in the plugin's ` +
+          `package.json file in the "name" field, cannot be determined automatically, and ` +
+          `should be reported here, otherwise the install will probably fail.`,
+        dest: 'packageName',
+      },
+    ],
+  ]);
 }
 
-function makeUninstallArgs (type) {
-  return [
+/**
+ * Makes the opts for the `uninstall` subcommand for each extension type
+ * @param {ExtensionType} type
+ * @returns {ArgumentDefinitions}
+ */
+function makeUninstallArgs(type) {
+  return new Map([
     ...globalExtensionArgs,
-    [[type], {
-      type: 'str',
-      help: 'Name of the driver to uninstall, for example: ' +
-            type === DRIVER_TYPE ? DRIVER_EXAMPLE : PLUGIN_EXAMPLE
-    }],
-  ];
+    [
+      [type],
+      {
+        type: 'str',
+        help:
+          'Name of the driver to uninstall, for example: ' + type === DRIVER_TYPE
+            ? DRIVER_EXAMPLE
+            : PLUGIN_EXAMPLE,
+      },
+    ],
+  ]);
 }
 
-function makeUpdateArgs (type) {
-  return [
+/**
+ * Makes the opts for the `update` subcommand for each extension type
+ * @param {ExtensionType} type
+ * @returns {ArgumentDefinitions}
+ */
+function makeUpdateArgs(type) {
+  return new Map([
     ...globalExtensionArgs,
-    [[type], {
-      type: 'str',
-      help: `Name of the ${type} to update, or the word "installed" to update all installed ` +
+    [
+      [type],
+      {
+        type: 'str',
+        help:
+          `Name of the ${type} to update, or the word "installed" to update all installed ` +
             `${type}s. To see available updates, run "appium ${type} list --installed --updates". ` +
-            'For example: ' + type === DRIVER_TYPE ? DRIVER_EXAMPLE : PLUGIN_EXAMPLE,
-    }],
-    [['--unsafe'], {
-      required: false,
-      default: false,
-      action: 'store_true',
-      help: `Include updates that might have a new major revision, and potentially include ` +
-            `breaking changes`,
-    }],
-  ];
+            'For example: ' +
+            type ===
+          DRIVER_TYPE
+            ? DRIVER_EXAMPLE
+            : PLUGIN_EXAMPLE,
+      },
+    ],
+    [
+      ['--unsafe'],
+      {
+        required: false,
+        default: false,
+        action: 'store_true',
+        help:
+          `Include updates that might have a new major revision, and potentially include ` +
+          `breaking changes`,
+      },
+    ],
+  ]);
+}
+
+/**
+ * Makes the opts for the `run` subcommand for each extension type
+ * @param {ExtensionType} type
+ * @returns {ArgumentDefinitions}
+ */
+function makeRunArgs(type) {
+  return new Map([
+    ...globalExtensionArgs,
+    [
+      [type],
+      {
+        type: 'str',
+        help:
+          `Name of the ${type} to run a script from, for example: ` + type === DRIVER_TYPE
+            ? DRIVER_EXAMPLE
+            : PLUGIN_EXAMPLE,
+      },
+    ],
+    [
+      ['scriptName'],
+      {
+        default: null,
+        type: 'str',
+        help:
+          `Name of the script to run from the ${type}. The script name must be a key ` +
+          `inside the "appium.scripts" field inside the ${type}'s "package.json" file`,
+      },
+    ],
+  ]);
 }
 
-for (const type of [DRIVER_TYPE, PLUGIN_TYPE]) {
-  extensionArgs[type].list = makeListArgs(type);
-  extensionArgs[type].install = makeInstallArgs(type);
-  extensionArgs[type].uninstall = makeUninstallArgs(type);
-  extensionArgs[type].update = makeUpdateArgs(type);
+/**
+ * Derives the options for the `server` command from the schema, and adds the arguments
+ * which are disallowed in the config file.
+ * @returns {ArgumentDefinitions}
+ */
+function getServerArgs() {
+  return new Map([...toParserArgs(), ...serverArgsDisallowedInConfig]);
 }
 
-export {
-  sharedArgs,
-  serverArgs,
-  extensionArgs,
-  USE_ALL_PLUGINS,
-};
+/**
+ * These don't make sense in the context of a config file for obvious reasons.
+ * @type {ArgumentDefinitions}
+ */
+const serverArgsDisallowedInConfig = new Map([
+  [
+    ['--shell'],
+    {
+      required: false,
+      help: 'Enter REPL mode',
+      action: 'store_const',
+      const: true,
+      dest: 'shell',
+    },
+  ],
+  [
+    ['--show-build-info'],
+    {
+      dest: 'showBuildInfo',
+      action: 'store_const',
+      const: true,
+      required: false,
+      help: 'Show info about the Appium build and exit',
+    },
+  ],
+  [
+    ['--show-config'],
+    {
+      dest: 'showConfig',
+      action: 'store_const',
+      const: true,
+      required: false,
+      help: 'Show the current Appium configuration and exit',
+    },
+  ],
+  [
+    ['--config'],
+    {
+      dest: 'configFile',
+      type: 'str',
+      required: false,
+      help: 'Explicit path to Appium configuration file',
+    },
+  ],
+]);
+
+export {getServerArgs, getExtensionArgs};
+
+/**
+ * @typedef {import('@appium/types').ExtensionType} ExtensionType
+ */
+
+/**
+ * A tuple of argument aliases and argument options
+ * @typedef {Map<[name: string]|[name: string, alias: string],import('argparse').ArgumentOptions>} ArgumentDefinitions
+ */