Skip to content

Commit 3e1972a

Browse files
nicodecleyreplamber
authored andcommitted
Removes 'defines correct option sets'. Closes pnp#4068
1 parent e762d52 commit 3e1972a

File tree

63 files changed

+0
-379
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

63 files changed

+0
-379
lines changed

src/m365/aad/commands/app/app-add.spec.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6503,13 +6503,6 @@ describe(commands.APP_ADD, () => {
65036503
assert.strictEqual(actual, true);
65046504
});
65056505

6506-
it('defines correct option sets', () => {
6507-
const optionSets = command.optionSets;
6508-
assert.deepStrictEqual(optionSets, [
6509-
{ options: ['name', 'manifest'] }
6510-
]);
6511-
});
6512-
65136506
it('creates AAD app reg for a web app from a manifest with redirectUris and options overriding them', async () => {
65146507
sinon.stub(request, 'get').callsFake(opts => {
65156508
if (opts.url === 'https://graph.microsoft.com/v1.0/myorganization/servicePrincipals?$select=appId,appRoles,id,oauth2PermissionScopes,servicePrincipalNames') {

src/m365/aad/commands/o365group/o365group-recyclebinitem-remove.spec.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -112,11 +112,6 @@ describe(commands.O365GROUP_RECYCLEBINITEM_REMOVE, () => {
112112
assert.notStrictEqual(command.description, null);
113113
});
114114

115-
it('defines correct option sets', () => {
116-
const optionSets = command.optionSets;
117-
assert.deepStrictEqual(optionSets, [{ options: ['id', 'displayName', 'mailNickname'] }]);
118-
});
119-
120115
it('fails validation when id is not a valid GUID', async () => {
121116
const actual = await command.validate({
122117
options: {

src/m365/aad/commands/o365group/o365group-recyclebinitem-restore.spec.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -106,11 +106,6 @@ describe(commands.O365GROUP_RECYCLEBINITEM_RESTORE, () => {
106106
assert.notStrictEqual(command.description, null);
107107
});
108108

109-
it('defines correct option sets', () => {
110-
const optionSets = command.optionSets;
111-
assert.deepStrictEqual(optionSets, [{ options: ['id', 'displayName', 'mailNickname'] }]);
112-
});
113-
114109
it('fails validation if the id is not a valid GUID', async () => {
115110
const actual = await command.validate({ options: { id: 'abc' } }, commandInfo);
116111
assert.notStrictEqual(actual, true);

src/m365/adaptivecard/commands/adaptivecard-send.spec.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -820,11 +820,6 @@ describe(commands.SEND, () => {
820820
assert.strictEqual(actual, true);
821821
});
822822

823-
it('defines correct option sets', () => {
824-
const optionSets = command.optionSets;
825-
assert.deepStrictEqual(optionSets, [{ options: ['title', 'card'] }]);
826-
});
827-
828823
it('supports specifying unknown options', () => {
829824
assert.strictEqual(command.allowUnknownOptions(), true);
830825
});

src/m365/booking/commands/business/business-get.spec.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -78,11 +78,6 @@ describe(commands.BUSINESS_GET, () => {
7878
assert.notStrictEqual(command.description, null);
7979
});
8080

81-
it('defines correct option sets', () => {
82-
const optionSets = command.optionSets;
83-
assert.deepStrictEqual(optionSets, [{ options: ['id', 'name'] }]);
84-
});
85-
8681
it('defines correct properties for the text output', () => {
8782
assert.deepStrictEqual(command.defaultProperties(), ['id', 'displayName', 'businessType', 'phone', 'email', 'defaultCurrencyIso']);
8883
});

src/m365/planner/commands/bucket/bucket-get.spec.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -257,11 +257,6 @@ describe(commands.BUCKET_GET, () => {
257257
assert.strictEqual(actual, true);
258258
});
259259

260-
it('defines correct option sets', () => {
261-
const optionSets = command.optionSets;
262-
assert.deepStrictEqual(optionSets, [{ options: ['id', 'name'] }]);
263-
});
264-
265260
it('fails validation when no groups found', async () => {
266261
sinon.stub(request, 'get').callsFake((opts) => {
267262
if (opts.url === `https://graph.microsoft.com/v1.0/groups?$filter=displayName eq '${formatting.encodeQueryParameter(validOwnerGroupName)}'`) {

src/m365/planner/commands/bucket/bucket-remove.spec.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -150,11 +150,6 @@ describe(commands.BUCKET_REMOVE, () => {
150150
assert.notStrictEqual(command.description, null);
151151
});
152152

153-
it('defines correct option sets', () => {
154-
const optionSets = command.optionSets;
155-
assert.deepStrictEqual(optionSets, [{ options: ['id', 'name'] }]);
156-
});
157-
158153
it('fails validation id when id and plan details are specified', async () => {
159154
const actual = await command.validate({
160155
options: {

src/m365/planner/commands/bucket/bucket-set.spec.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -145,11 +145,6 @@ describe(commands.BUCKET_SET, () => {
145145
assert.notStrictEqual(command.description, null);
146146
});
147147

148-
it('defines correct option sets', () => {
149-
const optionSets = command.optionSets;
150-
assert.deepStrictEqual(optionSets, [{ options: ['id', 'name'] }]);
151-
});
152-
153148
it('fails validation id when id and plan details are specified', async () => {
154149
const actual = await command.validate({
155150
options: {

src/m365/planner/commands/plan/plan-get.spec.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -116,11 +116,6 @@ describe(commands.PLAN_GET, () => {
116116
assert.notStrictEqual(actual, true);
117117
});
118118

119-
it('defines correct option sets', () => {
120-
const optionSets = command.optionSets;
121-
assert.deepStrictEqual(optionSets, [{ options: ['id', 'title'] }]);
122-
});
123-
124119
it('fails validation when both ownerGroupId and ownerGroupName are specified when using title', async () => {
125120
const actual = await command.validate({
126121
options: {

src/m365/planner/commands/plan/plan-remove.spec.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -112,11 +112,6 @@ describe(commands.PLAN_REMOVE, () => {
112112
assert.notStrictEqual(command.description, null);
113113
});
114114

115-
it('defines correct option sets', () => {
116-
const optionSets = command.optionSets;
117-
assert.deepStrictEqual(optionSets, [{ options: ['id', 'title'] }]);
118-
});
119-
120115
it('fails validation when id and ownerGroupId is specified', async () => {
121116
const actual = await command.validate({
122117
options: {

0 commit comments

Comments
 (0)