chore(appium): rename appiumCliIgnore to appiumCliIgnored
which, hopefully, is more indicative that we have a boolean
MODIFY
packages/appium/lib/schema/cli-args.js
packages/appium/lib/schema/cli-args.js
@@ -250,7 +250,7 @@ function subSchemaToArgDef (subSchema, argSpec, opts = {}) {
* aliases to `argparse` arguments; empty if no schema found
*/
export function toParserArgs (opts = {}) {
- const flattened = flattenSchema().filter(({schema}) => !schema.appiumCliIgnore);
+ const flattened = flattenSchema().filter(({schema}) => !schema.appiumCliIgnored);
return new Map(
_.map(flattened, ({schema, argSpec}) =>
subSchemaToArgDef(schema, argSpec, opts),

MODIFY
packages/appium/lib/schema/keywords.js
packages/appium/lib/schema/keywords.js
@@ -86,8 +86,8 @@ export const keywords = {
* Flag to tell Appium to _not_ provide this property as a CLI argument.
* @type {KeywordDefinition}
*/
- appiumCliIgnore: {
- keyword: 'appiumCliIgnore',
+ appiumCliIgnored: {
+ keyword: 'appiumCliIgnored',
metaSchema: {
type: 'boolean',
description: 'If `true`, Appium will not provide this property as a CLI argument. This is NOT the same as a "hidden" argument.',
@@ -124,7 +124,7 @@ export const keywords = {
* @property {string} [appiumCliDest]
* @property {string} [appiumCliDescription]
* @property {string[]} [appiumCliAliases]
- * @property {boolean} [appiumCliIgnore]
+ * @property {boolean} [appiumCliIgnored]
* @property {AppiumCliTransformerName} [appiumCliTransformer]
* @property {boolean} [appiumDeprecated]
*/

MODIFY
packages/appium/test/cli/fixtures/test-driver/package.json
packages/appium/test/cli/fixtures/test-driver/package.json
"properties": {
"mcmonkey-mcbean": {
"type": "string",
- "appiumCliIgnore": true
+ "appiumCliIgnored": true
},
"oliver-boliver": {
"type": "number",

MODIFY
packages/appium/test/cli/schema-e2e-specs.js
packages/appium/test/cli/schema-e2e-specs.js
@@ -34,7 +34,7 @@ describe('CLI behavior controlled by schema', function () {
help = await runAppium(appiumHome, ['server', '--help']);
});
- describe('appiumCliIgnore', function () {
+ describe('appiumCliIgnored', function () {
it('should still support arguments without this keyword', function () {
expect(help).to.match(/oliver-boliver/);
});

chore(appium): rename appiumCliIgnore to appiumCliIgnored

which, hopefully, is more indicative that we have a boolean