@pnp/cli-microsoft365 6.0.0-beta.27e1012 → 6.0.0-beta.2ab70db

package/dist/m365/aad/commands/approleassignment/approleassignment-list.js

@@ -1,70 +1,86 @@
 "use strict";
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (receiver, state, kind, f) {
+    if (kind === "a" && !f) throw new TypeError("Private accessor was defined without a getter");
+    if (typeof state === "function" ? receiver !== state || !f : !state.has(receiver)) throw new TypeError("Cannot read private member from an object whose class did not declare it");
+    return kind === "m" ? f : kind === "a" ? f.call(receiver) : f ? f.value : state.get(receiver);
+};
+var _AadAppRoleAssignmentListCommand_instances, _AadAppRoleAssignmentListCommand_initTelemetry, _AadAppRoleAssignmentListCommand_initOptions, _AadAppRoleAssignmentListCommand_initValidators, _AadAppRoleAssignmentListCommand_initOptionSets;
 Object.defineProperty(exports, "__esModule", { value: true });
 const request_1 = require("../../../../request");
 const utils_1 = require("../../../../utils");
 const GraphCommand_1 = require("../../../base/GraphCommand");
 const commands_1 = require("../../commands");
 class AadAppRoleAssignmentListCommand extends GraphCommand_1.default {
+    constructor() {
+        super();
+        _AadAppRoleAssignmentListCommand_instances.add(this);
+        __classPrivateFieldGet(this, _AadAppRoleAssignmentListCommand_instances, "m", _AadAppRoleAssignmentListCommand_initTelemetry).call(this);
+        __classPrivateFieldGet(this, _AadAppRoleAssignmentListCommand_instances, "m", _AadAppRoleAssignmentListCommand_initOptions).call(this);
+        __classPrivateFieldGet(this, _AadAppRoleAssignmentListCommand_instances, "m", _AadAppRoleAssignmentListCommand_initValidators).call(this);
+        __classPrivateFieldGet(this, _AadAppRoleAssignmentListCommand_instances, "m", _AadAppRoleAssignmentListCommand_initOptionSets).call(this);
+    }
     get name() {
         return commands_1.default.APPROLEASSIGNMENT_LIST;
     }
     get description() {
         return 'Lists app role assignments for the specified application registration';
     }
-    getTelemetryProperties(args) {
-        const telemetryProps = super.getTelemetryProperties(args);
-        telemetryProps.appId = typeof args.options.appId !== 'undefined';
-        telemetryProps.displayName = typeof args.options.displayName !== 'undefined';
-        telemetryProps.objectId = typeof args.options.objectId !== 'undefined';
-        return telemetryProps;
-    }
     defaultProperties() {
         return ['resourceDisplayName', 'roleName'];
     }
-    commandAction(logger, args, cb) {
-        let spAppRoleAssignments;
-        this.getAppRoleAssignments(args.options)
-            .then((appRoleAssignments) => {
-            spAppRoleAssignments = appRoleAssignments;
-            // the role assignment has an appRoleId but no name. To get the name,
-            // we need to get all the roles from the resource. the resource is
-            // a service principal. Multiple roles may have same resource id.
-            const resourceIds = appRoleAssignments.map((item) => item.resourceId);
-            const tasks = [];
-            for (let i = 0; i < resourceIds.length; i++) {
-                tasks.push(this.getServicePrincipal(resourceIds[i]));
-            }
-            return Promise.all(tasks);
-        })
-            .then((resources) => {
-            // loop through all appRoleAssignments for the servicePrincipal
-            // and lookup the appRole.Id in the resources[resourceId].appRoles array...
-            const results = [];
-            spAppRoleAssignments.map((appRoleAssignment) => {
-                const resource = resources.find((r) => r.id === appRoleAssignment.resourceId);
-                if (resource) {
-                    const appRole = resource.appRoles.find((r) => r.id === appRoleAssignment.appRoleId);
-                    if (appRole) {
-                        results.push({
-                            appRoleId: appRoleAssignment.appRoleId,
-                            resourceDisplayName: appRoleAssignment.resourceDisplayName,
-                            resourceId: appRoleAssignment.resourceId,
-                            roleId: appRole.id,
-                            roleName: appRole.value,
-                            created: appRoleAssignment.createdDateTime,
-                            deleted: appRoleAssignment.deletedDateTime
-                        });
-                    }
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            try {
+                const spAppRoleAssignments = yield this.getAppRoleAssignments(args.options);
+                // the role assignment has an appRoleId but no name. To get the name,
+                // we need to get all the roles from the resource. the resource is
+                // a service principal. Multiple roles may have same resource id.
+                const resourceIds = spAppRoleAssignments.map((item) => item.resourceId);
+                const tasks = [];
+                for (let i = 0; i < resourceIds.length; i++) {
+                    tasks.push(this.getServicePrincipal(resourceIds[i]));
                 }
-            });
-            logger.log(results);
-            cb();
-        }, (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
+                const resources = yield Promise.all(tasks);
+                // loop through all appRoleAssignments for the servicePrincipal
+                // and lookup the appRole.Id in the resources[resourceId].appRoles array...
+                const results = [];
+                spAppRoleAssignments.map((appRoleAssignment) => {
+                    const resource = resources.find((r) => r.id === appRoleAssignment.resourceId);
+                    if (resource) {
+                        const appRole = resource.appRoles.find((r) => r.id === appRoleAssignment.appRoleId);
+                        if (appRole) {
+                            results.push({
+                                appRoleId: appRoleAssignment.appRoleId,
+                                resourceDisplayName: appRoleAssignment.resourceDisplayName,
+                                resourceId: appRoleAssignment.resourceId,
+                                roleId: appRole.id,
+                                roleName: appRole.value,
+                                created: appRoleAssignment.createdDateTime,
+                                deleted: appRoleAssignment.deletedDateTime
+                            });
+                        }
+                    }
+                });
+                logger.log(results);
+            }
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
+            }
+        });
     }
     getAppRoleAssignments(argOptions) {
         return new Promise((resolve, reject) => {
-            if (argOptions.objectId) {
-                this.getSPAppRoleAssignments(argOptions.objectId)
+            if (argOptions.appObjectId) {
+                this.getSPAppRoleAssignments(argOptions.appObjectId)
                     .then((spAppRoleAssignments) => {
                     if (!spAppRoleAssignments.value.length) {
                         reject('no app role assignments found');
@@ -82,7 +98,7 @@ class AadAppRoleAssignmentListCommand extends GraphCommand_1.default {
                     spMatchQuery = `appId eq '${encodeURIComponent(argOptions.appId)}'`;
                 }
                 else {
-                    spMatchQuery = `displayName eq '${encodeURIComponent(argOptions.displayName)}'`;
+                    spMatchQuery = `displayName eq '${encodeURIComponent(argOptions.appDisplayName)}'`;
                 }
                 this.getServicePrincipalForApp(spMatchQuery)
                     .then((resp) => {
@@ -127,40 +143,35 @@ class AadAppRoleAssignmentListCommand extends GraphCommand_1.default {
         };
         return request_1.default.get(spRequestOptions);
     }
-    options() {
-        const options = [
-            {
-                option: '-i, --appId [appId]'
-            },
-            {
-                option: '-n, --displayName [displayName]'
-            },
-            {
-                option: '--objectId [objectId]'
-            }
-        ];
-        const parentOptions = super.options();
-        return options.concat(parentOptions);
-    }
-    validate(args) {
-        if (!args.options.appId && !args.options.displayName && !args.options.objectId) {
-            return 'Specify either appId, objectId or displayName';
-        }
+}
+_AadAppRoleAssignmentListCommand_instances = new WeakSet(), _AadAppRoleAssignmentListCommand_initTelemetry = function _AadAppRoleAssignmentListCommand_initTelemetry() {
+    this.telemetry.push((args) => {
+        Object.assign(this.telemetryProperties, {
+            appId: typeof args.options.appId !== 'undefined',
+            appDisplayName: typeof args.options.appDisplayName !== 'undefined',
+            appObjectId: typeof args.options.appObjectId !== 'undefined'
+        });
+    });
+}, _AadAppRoleAssignmentListCommand_initOptions = function _AadAppRoleAssignmentListCommand_initOptions() {
+    this.options.unshift({
+        option: '-i, --appId [appId]'
+    }, {
+        option: '-n, --appDisplayName [appDisplayName]'
+    }, {
+        option: '--appObjectId [appObjectId]'
+    });
+}, _AadAppRoleAssignmentListCommand_initValidators = function _AadAppRoleAssignmentListCommand_initValidators() {
+    this.validators.push((args) => __awaiter(this, void 0, void 0, function* () {
         if (args.options.appId && !utils_1.validation.isValidGuid(args.options.appId)) {
             return `${args.options.appId} is not a valid GUID`;
         }
-        if (args.options.objectId && !utils_1.validation.isValidGuid(args.options.objectId)) {
-            return `${args.options.objectId} is not a valid GUID`;
-        }
-        let optionsSpecified = 0;
-        optionsSpecified += args.options.appId ? 1 : 0;
-        optionsSpecified += args.options.displayName ? 1 : 0;
-        optionsSpecified += args.options.objectId ? 1 : 0;
-        if (optionsSpecified > 1) {
-            return 'Specify either appId, objectId or displayName';
+        if (args.options.appObjectId && !utils_1.validation.isValidGuid(args.options.appObjectId)) {
+            return `${args.options.appObjectId} is not a valid GUID`;
         }
         return true;
-    }
-}
+    }));
+}, _AadAppRoleAssignmentListCommand_initOptionSets = function _AadAppRoleAssignmentListCommand_initOptionSets() {
+    this.optionSets.push(['appId', 'appObjectId', 'appDisplayName']);
+};
 module.exports = new AadAppRoleAssignmentListCommand();
 //# sourceMappingURL=approleassignment-list.js.map

package/dist/m365/aad/commands/approleassignment/approleassignment-remove.js

@@ -1,4 +1,19 @@
 "use strict";
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (receiver, state, kind, f) {
+    if (kind === "a" && !f) throw new TypeError("Private accessor was defined without a getter");
+    if (typeof state === "function" ? receiver !== state || !f : !state.has(receiver)) throw new TypeError("Cannot read private member from an object whose class did not declare it");
+    return kind === "m" ? f : kind === "a" ? f.call(receiver) : f ? f.value : state.get(receiver);
+};
+var _AadAppRoleAssignmentRemoveCommand_instances, _AadAppRoleAssignmentRemoveCommand_initTelemetry, _AadAppRoleAssignmentRemoveCommand_initOptions, _AadAppRoleAssignmentRemoveCommand_initValidators, _AadAppRoleAssignmentRemoveCommand_initOptionSets;
 Object.defineProperty(exports, "__esModule", { value: true });
 const os = require("os");
 const cli_1 = require("../../../../cli");
@@ -7,119 +22,117 @@ const utils_1 = require("../../../../utils");
 const GraphCommand_1 = require("../../../base/GraphCommand");
 const commands_1 = require("../../commands");
 class AadAppRoleAssignmentRemoveCommand extends GraphCommand_1.default {
+    constructor() {
+        super();
+        _AadAppRoleAssignmentRemoveCommand_instances.add(this);
+        __classPrivateFieldGet(this, _AadAppRoleAssignmentRemoveCommand_instances, "m", _AadAppRoleAssignmentRemoveCommand_initTelemetry).call(this);
+        __classPrivateFieldGet(this, _AadAppRoleAssignmentRemoveCommand_instances, "m", _AadAppRoleAssignmentRemoveCommand_initOptions).call(this);
+        __classPrivateFieldGet(this, _AadAppRoleAssignmentRemoveCommand_instances, "m", _AadAppRoleAssignmentRemoveCommand_initValidators).call(this);
+        __classPrivateFieldGet(this, _AadAppRoleAssignmentRemoveCommand_instances, "m", _AadAppRoleAssignmentRemoveCommand_initOptionSets).call(this);
+    }
     get name() {
         return commands_1.default.APPROLEASSIGNMENT_REMOVE;
     }
     get description() {
         return 'Deletes an app role assignment for the specified Azure AD Application Registration';
     }
-    getTelemetryProperties(args) {
-        const telemetryProps = super.getTelemetryProperties(args);
-        telemetryProps.appId = typeof args.options.appId !== 'undefined';
-        telemetryProps.displayName = typeof args.options.displayName !== 'undefined';
-        telemetryProps.objectId = typeof args.options.objectId !== 'undefined';
-        telemetryProps.confirm = (!!args.options.confirm).toString();
-        return telemetryProps;
-    }
-    commandAction(logger, args, cb) {
-        const removeAppRoleAssignment = () => {
-            let sp;
-            // get the service principal associated with the appId
-            let spMatchQuery = '';
-            if (args.options.appId) {
-                spMatchQuery = `appId eq '${encodeURIComponent(args.options.appId)}'`;
-            }
-            else if (args.options.objectId) {
-                spMatchQuery = `id eq '${encodeURIComponent(args.options.objectId)}'`;
-            }
-            else {
-                spMatchQuery = `displayName eq '${encodeURIComponent(args.options.displayName)}'`;
-            }
-            this
-                .getServicePrincipalForApp(spMatchQuery)
-                .then((resp) => {
-                if (!resp.value.length) {
-                    return Promise.reject('app registration not found');
-                }
-                sp = resp.value[0];
-                let resource = encodeURIComponent(args.options.resource);
-                // try resolve aliases that the user might enter since these are seen in the Azure portal
-                switch (args.options.resource.toLocaleLowerCase()) {
-                    case 'sharepoint':
-                        resource = 'Office 365 SharePoint Online';
-                        break;
-                    case 'intune':
-                        resource = 'Microsoft Intune API';
-                        break;
-                    case 'exchange':
-                        resource = 'Office 365 Exchange Online';
-                        break;
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            const removeAppRoleAssignment = () => __awaiter(this, void 0, void 0, function* () {
+                let sp;
+                // get the service principal associated with the appId
+                let spMatchQuery = '';
+                if (args.options.appId) {
+                    spMatchQuery = `appId eq '${encodeURIComponent(args.options.appId)}'`;
                 }
-                // will perform resource name, appId or objectId search
-                let filter = `$filter=(displayName eq '${resource}' or startswith(displayName,'${resource}'))`;
-                if (utils_1.validation.isValidGuid(resource)) {
-                    filter += ` or appId eq '${resource}' or id eq '${resource}'`;
+                else if (args.options.appObjectId) {
+                    spMatchQuery = `id eq '${encodeURIComponent(args.options.appObjectId)}'`;
                 }
-                const requestOptions = {
-                    url: `${this.resource}/v1.0/servicePrincipals?${filter}`,
-                    headers: {
-                        'accept': 'application/json'
-                    },
-                    responseType: 'json'
-                };
-                return request_1.default.get(requestOptions);
-            })
-                .then((resp) => {
-                if (!resp.value.length) {
-                    return Promise.reject('Resource not found');
-                }
-                const appRolesToBeDeleted = [];
-                const appRolesFound = resp.value[0].appRoles;
-                if (!appRolesFound.length) {
-                    return Promise.reject(`The resource '${args.options.resource}' does not have any application permissions available.`);
+                else {
+                    spMatchQuery = `displayName eq '${encodeURIComponent(args.options.appDisplayName)}'`;
                 }
-                for (const scope of args.options.scope.split(',')) {
-                    const existingRoles = appRolesFound.filter((role) => {
-                        return role.value.toLocaleLowerCase() === scope.toLocaleLowerCase().trim();
-                    });
-                    if (!existingRoles.length) {
-                        // the role specified in the scope option does not belong to the found service principles
-                        // throw an error and show list with available roles (scopes)
-                        let availableRoles = '';
-                        appRolesFound.map((r) => availableRoles += `${os.EOL}${r.value}`);
-                        return Promise.reject(`The scope value '${scope}' you have specified does not exist for ${args.options.resource}. ${os.EOL}Available scopes (application permissions) are: ${availableRoles}`);
+                try {
+                    let resp = yield this.getServicePrincipalForApp(spMatchQuery);
+                    if (!resp.value.length) {
+                        throw 'app registration not found';
                     }
-                    appRolesToBeDeleted.push(existingRoles[0]);
-                }
-                const tasks = [];
-                for (const appRole of appRolesToBeDeleted) {
-                    const appRoleAssignment = sp.appRoleAssignments.filter((role) => role.appRoleId === appRole.id);
-                    if (!appRoleAssignment.length) {
-                        return Promise.reject('App role assignment not found');
+                    sp = resp.value[0];
+                    let resource = encodeURIComponent(args.options.resource);
+                    // try resolve aliases that the user might enter since these are seen in the Azure portal
+                    switch (args.options.resource.toLocaleLowerCase()) {
+                        case 'sharepoint':
+                            resource = 'Office 365 SharePoint Online';
+                            break;
+                        case 'intune':
+                            resource = 'Microsoft Intune API';
+                            break;
+                        case 'exchange':
+                            resource = 'Office 365 Exchange Online';
+                            break;
+                    }
+                    // will perform resource name, appId or objectId search
+                    let filter = `$filter=(displayName eq '${resource}' or startswith(displayName,'${resource}'))`;
+                    if (utils_1.validation.isValidGuid(resource)) {
+                        filter += ` or appId eq '${resource}' or id eq '${resource}'`;
+                    }
+                    const requestOptions = {
+                        url: `${this.resource}/v1.0/servicePrincipals?${filter}`,
+                        headers: {
+                            'accept': 'application/json'
+                        },
+                        responseType: 'json'
+                    };
+                    resp = yield request_1.default.get(requestOptions);
+                    if (!resp.value.length) {
+                        throw 'Resource not found';
+                    }
+                    const appRolesToBeDeleted = [];
+                    const appRolesFound = resp.value[0].appRoles;
+                    if (!appRolesFound.length) {
+                        throw `The resource '${args.options.resource}' does not have any application permissions available.`;
+                    }
+                    for (const scope of args.options.scope.split(',')) {
+                        const existingRoles = appRolesFound.filter((role) => {
+                            return role.value.toLocaleLowerCase() === scope.toLocaleLowerCase().trim();
+                        });
+                        if (!existingRoles.length) {
+                            // the role specified in the scope option does not belong to the found service principles
+                            // throw an error and show list with available roles (scopes)
+                            let availableRoles = '';
+                            appRolesFound.map((r) => availableRoles += `${os.EOL}${r.value}`);
+                            throw `The scope value '${scope}' you have specified does not exist for ${args.options.resource}. ${os.EOL}Available scopes (application permissions) are: ${availableRoles}`;
+                        }
+                        appRolesToBeDeleted.push(existingRoles[0]);
                     }
-                    tasks.push(this.removeAppRoleAssignmentForServicePrincipal(sp.id, appRoleAssignment[0].id));
+                    const tasks = [];
+                    for (const appRole of appRolesToBeDeleted) {
+                        const appRoleAssignment = sp.appRoleAssignments.filter((role) => role.appRoleId === appRole.id);
+                        if (!appRoleAssignment.length) {
+                            throw 'App role assignment not found';
+                        }
+                        tasks.push(this.removeAppRoleAssignmentForServicePrincipal(sp.id, appRoleAssignment[0].id));
+                    }
+                    yield Promise.all(tasks);
                 }
-                return Promise.all(tasks);
-            }).then(_ => cb(), (res) => this.handleRejectedODataJsonPromise(res, logger, cb));
-        };
-        if (args.options.confirm) {
-            removeAppRoleAssignment();
-        }
-        else {
-            cli_1.Cli.prompt({
-                type: 'confirm',
-                name: 'continue',
-                default: false,
-                message: `Are you sure you want to remove the appRoleAssignment with scope ${args.options.scope} for resource ${args.options.resource}?`
-            }, (result) => {
-                if (!result.continue) {
-                    cb();
-                }
-                else {
-                    removeAppRoleAssignment();
+                catch (err) {
+                    this.handleRejectedODataJsonPromise(err);
                 }
             });
-        }
+            if (args.options.confirm) {
+                yield removeAppRoleAssignment();
+            }
+            else {
+                const result = yield cli_1.Cli.prompt({
+                    type: 'confirm',
+                    name: 'continue',
+                    default: false,
+                    message: `Are you sure you want to remove the appRoleAssignment with scope ${args.options.scope} for resource ${args.options.resource}?`
+                });
+                if (result.continue) {
+                    yield removeAppRoleAssignment();
+                }
+            }
+        });
     }
     getServicePrincipalForApp(filterParam) {
         const spRequestOptions = {
@@ -141,47 +154,43 @@ class AadAppRoleAssignmentRemoveCommand extends GraphCommand_1.default {
         };
         return request_1.default.delete(spRequestOptions);
     }
-    options() {
-        const options = [
-            {
-                option: '--appId [appId]'
-            },
-            {
-                option: '--objectId [objectId]'
-            },
-            {
-                option: '--displayName [displayName]'
-            },
-            {
-                option: '-r, --resource <resource>',
-                autocomplete: ['Microsoft Graph', 'SharePoint', 'OneNote', 'Exchange', 'Microsoft Forms', 'Azure Active Directory Graph', 'Skype for Business']
-            },
-            {
-                option: '-s, --scope <scope>'
-            },
-            {
-                option: '--confirm'
-            }
-        ];
-        const parentOptions = super.options();
-        return options.concat(parentOptions);
-    }
-    validate(args) {
-        let optionsSpecified = 0;
-        optionsSpecified += args.options.appId ? 1 : 0;
-        optionsSpecified += args.options.displayName ? 1 : 0;
-        optionsSpecified += args.options.objectId ? 1 : 0;
-        if (optionsSpecified !== 1) {
-            return 'Specify either appId, objectId or displayName';
-        }
+}
+_AadAppRoleAssignmentRemoveCommand_instances = new WeakSet(), _AadAppRoleAssignmentRemoveCommand_initTelemetry = function _AadAppRoleAssignmentRemoveCommand_initTelemetry() {
+    this.telemetry.push((args) => {
+        Object.assign(this.telemetryProperties, {
+            appId: typeof args.options.appId !== 'undefined',
+            appDisplayName: typeof args.options.appDisplayName !== 'undefined',
+            appObjectId: typeof args.options.appObjectId !== 'undefined',
+            confirm: (!!args.options.confirm).toString()
+        });
+    });
+}, _AadAppRoleAssignmentRemoveCommand_initOptions = function _AadAppRoleAssignmentRemoveCommand_initOptions() {
+    this.options.unshift({
+        option: '--appId [appId]'
+    }, {
+        option: '--appObjectId [appObjectId]'
+    }, {
+        option: '--appDisplayName [appDisplayName]'
+    }, {
+        option: '-r, --resource <resource>',
+        autocomplete: ['Microsoft Graph', 'SharePoint', 'OneNote', 'Exchange', 'Microsoft Forms', 'Azure Active Directory Graph', 'Skype for Business']
+    }, {
+        option: '-s, --scope <scope>'
+    }, {
+        option: '--confirm'
+    });
+}, _AadAppRoleAssignmentRemoveCommand_initValidators = function _AadAppRoleAssignmentRemoveCommand_initValidators() {
+    this.validators.push((args) => __awaiter(this, void 0, void 0, function* () {
         if (args.options.appId && !utils_1.validation.isValidGuid(args.options.appId)) {
             return `${args.options.appId} is not a valid GUID`;
         }
-        if (args.options.objectId && !utils_1.validation.isValidGuid(args.options.objectId)) {
-            return `${args.options.objectId} is not a valid GUID`;
+        if (args.options.appObjectId && !utils_1.validation.isValidGuid(args.options.appObjectId)) {
+            return `${args.options.appObjectId} is not a valid GUID`;
         }
         return true;
-    }
-}
+    }));
+}, _AadAppRoleAssignmentRemoveCommand_initOptionSets = function _AadAppRoleAssignmentRemoveCommand_initOptionSets() {
+    this.optionSets.push(['appId', 'appObjectId', 'appDisplayName']);
+};
 module.exports = new AadAppRoleAssignmentRemoveCommand();
 //# sourceMappingURL=approleassignment-remove.js.map

package/dist/m365/aad/commands/group/group-list.js

@@ -1,55 +1,76 @@
 "use strict";
+var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
+    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
+    return new (P || (P = Promise))(function (resolve, reject) {
+        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
+        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
+        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
+        step((generator = generator.apply(thisArg, _arguments || [])).next());
+    });
+};
+var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (receiver, state, kind, f) {
+    if (kind === "a" && !f) throw new TypeError("Private accessor was defined without a getter");
+    if (typeof state === "function" ? receiver !== state || !f : !state.has(receiver)) throw new TypeError("Cannot read private member from an object whose class did not declare it");
+    return kind === "m" ? f : kind === "a" ? f.call(receiver) : f ? f.value : state.get(receiver);
+};
+var _AadGroupListCommand_instances, _AadGroupListCommand_initTelemetry, _AadGroupListCommand_initOptions;
 Object.defineProperty(exports, "__esModule", { value: true });
 const utils_1 = require("../../../../utils");
 const GraphCommand_1 = require("../../../base/GraphCommand");
 const commands_1 = require("../../commands");
 class AadGroupListCommand extends GraphCommand_1.default {
+    constructor() {
+        super();
+        _AadGroupListCommand_instances.add(this);
+        __classPrivateFieldGet(this, _AadGroupListCommand_instances, "m", _AadGroupListCommand_initTelemetry).call(this);
+        __classPrivateFieldGet(this, _AadGroupListCommand_instances, "m", _AadGroupListCommand_initOptions).call(this);
+    }
     get name() {
         return commands_1.default.GROUP_LIST;
     }
     get description() {
         return 'Lists all groups defined in Azure Active Directory.';
     }
-    getTelemetryProperties(args) {
-        const telemetryProps = super.getTelemetryProperties(args);
-        telemetryProps.deleted = args.options.deleted;
-        return telemetryProps;
-    }
     defaultProperties() {
         return ['id', 'displayName', 'groupType'];
     }
-    commandAction(logger, args, cb) {
-        const endpoint = args.options.deleted ? 'directory/deletedItems/microsoft.graph.group' : 'groups';
-        utils_1.odata
-            .getAllItems(`${this.resource}/v1.0/${endpoint}`)
-            .then((groups) => {
-            if (args.options.output === 'text') {
-                groups.forEach((group) => {
-                    if (group.groupTypes && group.groupTypes.length > 0 && group.groupTypes[0] === 'Unified') {
-                        group.groupType = 'Microsoft 365';
-                    }
-                    else if (group.mailEnabled && group.securityEnabled) {
-                        group.groupType = 'Mail enabled security';
-                    }
-                    else if (group.securityEnabled) {
-                        group.groupType = 'Security';
-                    }
-                    else if (group.mailEnabled) {
-                        group.groupType = 'Distribution';
-                    }
-                });
+    commandAction(logger, args) {
+        return __awaiter(this, void 0, void 0, function* () {
+            const endpoint = args.options.deleted ? 'directory/deletedItems/microsoft.graph.group' : 'groups';
+            try {
+                const groups = yield utils_1.odata.getAllItems(`${this.resource}/v1.0/${endpoint}`);
+                if (args.options.output === 'text') {
+                    groups.forEach((group) => {
+                        if (group.groupTypes && group.groupTypes.length > 0 && group.groupTypes[0] === 'Unified') {
+                            group.groupType = 'Microsoft 365';
+                        }
+                        else if (group.mailEnabled && group.securityEnabled) {
+                            group.groupType = 'Mail enabled security';
+                        }
+                        else if (group.securityEnabled) {
+                            group.groupType = 'Security';
+                        }
+                        else if (group.mailEnabled) {
+                            group.groupType = 'Distribution';
+                        }
+                    });
+                }
+                logger.log(groups);
             }
-            logger.log(groups);
-            cb();
-        }, (err) => this.handleRejectedODataJsonPromise(err, logger, cb));
-    }
-    options() {
-        const options = [
-            { option: '-d, --deleted' }
-        ];
-        const parentOptions = super.options();
-        return options.concat(parentOptions);
+            catch (err) {
+                this.handleRejectedODataJsonPromise(err);
+            }
+        });
     }
 }
+_AadGroupListCommand_instances = new WeakSet(), _AadGroupListCommand_initTelemetry = function _AadGroupListCommand_initTelemetry() {
+    this.telemetry.push((args) => {
+        Object.assign(this.telemetryProperties, {
+            deleted: args.options.deleted
+        });
+    });
+}, _AadGroupListCommand_initOptions = function _AadGroupListCommand_initOptions() {
+    this.options.unshift({ option: '-d, --deleted' });
+};
 module.exports = new AadGroupListCommand();
 //# sourceMappingURL=group-list.js.map