@pnp/cli-microsoft365 6.0.0-beta.27e1012 → 6.0.0-beta.3261c97

package/dist/cli/Cli.js

@@ -99,51 +99,6 @@ class Cli {
                 this.printHelp();
                 return Promise.resolve();
             }
-            // if the command doesn't allow unknown options, check if all specified
-            // options match command options
-            if (!this.commandToExecute.command.allowUnknownOptions()) {
-                for (const optionFromArgs in this.optionsFromArgs.options) {
-                    if (optionFromArgs === '_') {
-                        // ignore catch-all option
-                        continue;
-                    }
-                    let matches = false;
-                    for (let i = 0; i < this.commandToExecute.options.length; i++) {
-                        const option = this.commandToExecute.options[i];
-                        if (optionFromArgs === option.long ||
-                            optionFromArgs === option.short) {
-                            matches = true;
-                            break;
-                        }
-                    }
-                    if (!matches) {
-                        return this.closeWithError(`Invalid option: '${optionFromArgs}'${os.EOL}`, this.optionsFromArgs, true);
-                    }
-                }
-            }
-            const shouldPrompt = this.getSettingWithDefaultValue(settingsNames_1.settingsNames.prompt, false);
-            let inquirer;
-            for (let i = 0; i < this.commandToExecute.options.length; i++) {
-                if (this.commandToExecute.options[i].required &&
-                    typeof this.optionsFromArgs.options[this.commandToExecute.options[i].name] === 'undefined') {
-                    if (!shouldPrompt) {
-                        return this.closeWithError(`Required option ${this.commandToExecute.options[i].name} not specified`, this.optionsFromArgs, true);
-                    }
-                    if (i === 0) {
-                        Cli.log('Provide values for the following parameters:');
-                    }
-                    if (!inquirer) {
-                        inquirer = require('inquirer');
-                    }
-                    const missingRequireOptionValue = yield inquirer
-                        .prompt({
-                        name: 'missingRequireOptionValue',
-                        message: `${this.commandToExecute.options[i].name}: `
-                    })
-                        .then(result => result.missingRequireOptionValue);
-                    this.optionsFromArgs.options[this.commandToExecute.options[i].name] = missingRequireOptionValue;
-                }
-            }
             const optionsWithoutShorts = Cli.removeShortOptions(this.optionsFromArgs);
             try {
                 // replace values staring with @ with file contents
@@ -161,23 +116,10 @@ class Cli {
             }
             // if output not specified, set the configured output value (if any)
             if (optionsWithoutShorts.options.output === undefined) {
-                optionsWithoutShorts.options.output = this.getSettingWithDefaultValue(settingsNames_1.settingsNames.output, undefined);
-            }
-            // validate options sets defined by the command
-            const optionsSets = this.commandToExecute.command.optionSets();
-            const argsOptions = Object.keys(this.optionsFromArgs.options);
-            const cmdArgs = this.optionsFromArgs;
-            optionsSets === null || optionsSets === void 0 ? void 0 : optionsSets.forEach((optionSet) => {
-                const commonOptions = argsOptions.filter(opt => optionSet.includes(opt));
-                if (commonOptions.length === 0) {
-                    return this.closeWithError(`Specify one of the following options: ${optionSet.map(opt => opt).join(', ')}.`, cmdArgs, true);
-                }
-                if (commonOptions.length > 1) {
-                    return this.closeWithError(`Specify one of the following options: ${optionSet.map(opt => opt).join(', ')}, but not multiple.`, cmdArgs, true);
-                }
-            });
-            const validationResult = this.commandToExecute.command.validate(optionsWithoutShorts);
-            if (typeof validationResult === 'string') {
+                optionsWithoutShorts.options.output = this.getSettingWithDefaultValue(settingsNames_1.settingsNames.output, 'json');
+            }
+            const validationResult = yield this.commandToExecute.command.validate(optionsWithoutShorts, this.commandToExecute);
+            if (validationResult !== true) {
                 return this.closeWithError(validationResult, optionsWithoutShorts, true);
             }
             return Cli
@@ -186,7 +128,7 @@ class Cli {
         });
     }
     static executeCommand(command, args) {
-        return new Promise((resolve, reject) => {
+        return __awaiter(this, void 0, void 0, function* () {
             const logger = {
                 log: (message) => {
                     const output = Cli.formatOutput(message, args.options);
@@ -203,22 +145,21 @@ class Cli {
             const cli = Cli.getInstance();
             const parentCommandName = cli.currentCommandName;
             cli.currentCommandName = command.getCommandName(cli.currentCommandName);
-            command.action(logger, args, (err) => {
-                // restore the original command name
-                cli.currentCommandName = parentCommandName;
-                if (err) {
-                    return reject(err);
-                }
+            try {
+                yield command.action(logger, args);
                 if (args.options.debug || args.options.verbose) {
                     const chalk = require('chalk');
                     logger.logToStderr(chalk.green('DONE'));
                 }
-                resolve();
-            });
+            }
+            finally {
+                // restore the original command name
+                cli.currentCommandName = parentCommandName;
+            }
         });
     }
     static executeCommandWithOutput(command, args, listener) {
-        return new Promise((resolve, reject) => {
+        return __awaiter(this, void 0, void 0, function* () {
             const log = [];
             const logErr = [];
             const logger = {
@@ -257,22 +198,29 @@ class Cli {
             cli.currentCommandName = command.getCommandName();
             // store the current logger if any
             const currentLogger = request_1.default.logger;
-            command.action(logger, args, (err) => {
+            try {
+                yield command.action(logger, args);
+                // restoring the command and logger is done here instead of in a 'finally' because there were issues with the code coverage tool
                 // restore the original command name
                 cli.currentCommandName = parentCommandName;
                 // restore the original logger
                 request_1.default.logger = currentLogger;
-                if (err) {
-                    return reject({
-                        error: err,
-                        stderr: logErr.join(os.EOL)
-                    });
-                }
-                resolve({
+                return ({
                     stdout: log.join(os.EOL),
                     stderr: logErr.join(os.EOL)
                 });
-            });
+            }
+            catch (err) {
+                // restoring the command and logger is done here instead of in a 'finally' because there were issues with the code coverage tool
+                // restore the original command name
+                cli.currentCommandName = parentCommandName;
+                // restore the original logger
+                request_1.default.logger = currentLogger;
+                throw {
+                    error: err,
+                    stderr: logErr.join(os.EOL)
+                };
+            }
         });
     }
     loadAllCommands() {
@@ -348,18 +296,21 @@ class Cli {
             this.loadAllCommands();
         }
     }
-    loadCommand(command) {
-        this.commands.push({
+    static getCommandInfo(command) {
+        return {
             aliases: command.alias(),
             name: command.name,
             command: command,
             options: this.getCommandOptions(command),
             defaultProperties: command.defaultProperties()
-        });
+        };
+    }
+    loadCommand(command) {
+        this.commands.push(Cli.getCommandInfo(command));
     }
-    getCommandOptions(command) {
+    static getCommandOptions(command) {
         const options = [];
-        command.options().forEach(option => {
+        command.options.forEach(option => {
             const required = option.option.indexOf('<') > -1;
             const optionArgs = option.option.split(/[ ,|]+/);
             let short;
@@ -390,7 +341,7 @@ class Cli {
             alias: {}
         };
         if (commandInfo) {
-            const commandTypes = commandInfo.command.types();
+            const commandTypes = commandInfo.command.types;
             if (commandTypes) {
                 minimistOptions.string = commandTypes.string;
                 minimistOptions.boolean = commandTypes.boolean;
@@ -719,11 +670,11 @@ class Cli {
             console.error(message, ...optionalParams);
         }
     }
-    static prompt(options, cb) {
-        const inquirer = require('inquirer');
-        inquirer
-            .prompt(options)
-            .then(result => cb(result));
+    static prompt(options) {
+        return __awaiter(this, void 0, void 0, function* () {
+            const inquirer = require('inquirer');
+            return yield inquirer.prompt(options);
+        });
     }
     static removeShortOptions(args) {
         const filteredArgs = JSON.parse(JSON.stringify(args));