Skip to content

Commit 0a05c87

Browse files
jkalandaibmjkalandaibm
authored
Merge pull request #84 from IBM/security-fix
fix: Fixing the Packing does not respect root-level ignore files in workspace issue
2 parents 33abd42 + 2ebbb7f commit 0a05c87

20 files changed

+10006
-14744
lines changed

‎package-lock.json‎

Lines changed: 9942 additions & 14617 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎package.json‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,17 +53,17 @@
5353
"eslint-plugin-prettier": "^3.0.1",
5454
"highlight.js": "^10.4.1",
5555
"installed-check": "^2.2.0",
56-
"jest": "^24.9.0",
56+
"jest": "^28.1.0",
5757
"marked": "^4.0.10",
5858
"node-notifier": "^8.0.1",
5959
"prettier": "^1.17.1",
60-
"semantic-release": "^17.4.2",
60+
"semantic-release": "^19.0.2",
6161
"set-value": "^4.0.0",
6262
"tsc-publish": "^0.5.2",
6363
"tslint": "^5.16.0",
6464
"tslint-config-prettier": "^1.18.0",
6565
"tslint-eslint-rules": "^5.4.0",
66-
"typedoc": "^0.15.0",
66+
"typedoc": "^0.22.17",
6767
"typescript": "^3.9.5",
6868
"yargs-parser": "^13.1.2"
6969
},

‎test/unit/cis/custom-pages.v1.test.js‎

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,7 @@ describe('CustomPagesV1', () => {
218218
});
219219

220220
describe('negative tests', () => {
221-
test('should enforce required parameters', async done => {
221+
test('should enforce required parameters', async () => {
222222
let err;
223223
try {
224224
await customPagesService.getInstanceCustomPage({});
@@ -227,7 +227,6 @@ describe('CustomPagesV1', () => {
227227
}
228228

229229
expect(err.message).toMatch(/Missing required parameters/);
230-
done();
231230
});
232231

233232
test('should reject promise when required params are not given', done => {
@@ -293,7 +292,7 @@ describe('CustomPagesV1', () => {
293292
});
294293

295294
describe('negative tests', () => {
296-
test('should enforce required parameters', async done => {
295+
test('should enforce required parameters', async () => {
297296
let err;
298297
try {
299298
await customPagesService.updateInstanceCustomPage({});
@@ -302,7 +301,6 @@ describe('CustomPagesV1', () => {
302301
}
303302

304303
expect(err.message).toMatch(/Missing required parameters/);
305-
done();
306304
});
307305

308306
test('should reject promise when required params are not given', done => {
@@ -413,7 +411,7 @@ describe('CustomPagesV1', () => {
413411
});
414412

415413
describe('negative tests', () => {
416-
test('should enforce required parameters', async done => {
414+
test('should enforce required parameters', async () => {
417415
let err;
418416
try {
419417
await customPagesService.getZoneCustomPage({});
@@ -422,7 +420,6 @@ describe('CustomPagesV1', () => {
422420
}
423421

424422
expect(err.message).toMatch(/Missing required parameters/);
425-
done();
426423
});
427424

428425
test('should reject promise when required params are not given', done => {
@@ -493,7 +490,7 @@ describe('CustomPagesV1', () => {
493490
});
494491

495492
describe('negative tests', () => {
496-
test('should enforce required parameters', async done => {
493+
test('should enforce required parameters', async () => {
497494
let err;
498495
try {
499496
await customPagesService.updateZoneCustomPage({});
@@ -502,7 +499,6 @@ describe('CustomPagesV1', () => {
502499
}
503500

504501
expect(err.message).toMatch(/Missing required parameters/);
505-
done();
506502
});
507503

508504
test('should reject promise when required params are not given', done => {

‎test/unit/cis/dns-records.v1.test.js‎

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -314,7 +314,7 @@ describe('DnsRecordsV1', () => {
314314
});
315315

316316
describe('negative tests', () => {
317-
test('should enforce required parameters', async done => {
317+
test('should enforce required parameters', async () => {
318318
let err;
319319
try {
320320
await dnsRecordsService.deleteDnsRecord({});
@@ -323,7 +323,6 @@ describe('DnsRecordsV1', () => {
323323
}
324324

325325
expect(err.message).toMatch(/Missing required parameters/);
326-
done();
327326
});
328327

329328
test('should reject promise when required params are not given', done => {
@@ -388,7 +387,7 @@ describe('DnsRecordsV1', () => {
388387
});
389388

390389
describe('negative tests', () => {
391-
test('should enforce required parameters', async done => {
390+
test('should enforce required parameters', async () => {
392391
let err;
393392
try {
394393
await dnsRecordsService.getDnsRecord({});
@@ -397,7 +396,6 @@ describe('DnsRecordsV1', () => {
397396
}
398397

399398
expect(err.message).toMatch(/Missing required parameters/);
400-
done();
401399
});
402400

403401
test('should reject promise when required params are not given', done => {
@@ -483,7 +481,7 @@ describe('DnsRecordsV1', () => {
483481
});
484482

485483
describe('negative tests', () => {
486-
test('should enforce required parameters', async done => {
484+
test('should enforce required parameters', async () => {
487485
let err;
488486
try {
489487
await dnsRecordsService.updateDnsRecord({});
@@ -492,7 +490,6 @@ describe('DnsRecordsV1', () => {
492490
}
493491

494492
expect(err.message).toMatch(/Missing required parameters/);
495-
done();
496493
});
497494

498495
test('should reject promise when required params are not given', done => {

‎test/unit/cis/edge-functions-api.v1.test.js‎

Lines changed: 6 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,7 @@ describe('EdgeFunctionsApiV1', () => {
233233
});
234234

235235
describe('negative tests', () => {
236-
test('should enforce required parameters', async done => {
236+
test('should enforce required parameters', async () => {
237237
let err;
238238
try {
239239
await edgeFunctionsApiService.updateEdgeFunctionsAction({});
@@ -242,7 +242,6 @@ describe('EdgeFunctionsApiV1', () => {
242242
}
243243

244244
expect(err.message).toMatch(/Missing required parameters/);
245-
done();
246245
});
247246

248247
test('should reject promise when required params are not given', done => {
@@ -306,16 +305,14 @@ describe('EdgeFunctionsApiV1', () => {
306305
});
307306

308307
describe('negative tests', () => {
309-
test('should enforce required parameters', async done => {
308+
test('should enforce required parameters', async () => {
310309
let err;
311310
try {
312311
await edgeFunctionsApiService.getEdgeFunctionsAction({});
313312
} catch (e) {
314313
err = e;
315314
}
316-
317315
expect(err.message).toMatch(/Missing required parameters/);
318-
done();
319316
});
320317

321318
test('should reject promise when required params are not given', done => {
@@ -380,7 +377,7 @@ describe('EdgeFunctionsApiV1', () => {
380377
});
381378

382379
describe('negative tests', () => {
383-
test('should enforce required parameters', async done => {
380+
test('should enforce required parameters', async () => {
384381
let err;
385382
try {
386383
await edgeFunctionsApiService.deleteEdgeFunctionsAction({});
@@ -389,7 +386,6 @@ describe('EdgeFunctionsApiV1', () => {
389386
}
390387

391388
expect(err.message).toMatch(/Missing required parameters/);
392-
done();
393389
});
394390

395391
test('should reject promise when required params are not given', done => {
@@ -569,7 +565,7 @@ describe('EdgeFunctionsApiV1', () => {
569565
});
570566

571567
describe('negative tests', () => {
572-
test('should enforce required parameters', async done => {
568+
test('should enforce required parameters', async () => {
573569
let err;
574570
try {
575571
await edgeFunctionsApiService.getEdgeFunctionsTrigger({});
@@ -578,7 +574,6 @@ describe('EdgeFunctionsApiV1', () => {
578574
}
579575

580576
expect(err.message).toMatch(/Missing required parameters/);
581-
done();
582577
});
583578

584579
test('should reject promise when required params are not given', done => {
@@ -654,7 +649,7 @@ describe('EdgeFunctionsApiV1', () => {
654649
});
655650

656651
describe('negative tests', () => {
657-
test('should enforce required parameters', async done => {
652+
test('should enforce required parameters', async () => {
658653
let err;
659654
try {
660655
await edgeFunctionsApiService.updateEdgeFunctionsTrigger({});
@@ -663,7 +658,6 @@ describe('EdgeFunctionsApiV1', () => {
663658
}
664659

665660
expect(err.message).toMatch(/Missing required parameters/);
666-
done();
667661
});
668662

669663
test('should reject promise when required params are not given', done => {
@@ -733,7 +727,7 @@ describe('EdgeFunctionsApiV1', () => {
733727
});
734728

735729
describe('negative tests', () => {
736-
test('should enforce required parameters', async done => {
730+
test('should enforce required parameters', async () => {
737731
let err;
738732
try {
739733
await edgeFunctionsApiService.deleteEdgeFunctionsTrigger({});
@@ -742,7 +736,6 @@ describe('EdgeFunctionsApiV1', () => {
742736
}
743737

744738
expect(err.message).toMatch(/Missing required parameters/);
745-
done();
746739
});
747740

748741
test('should reject promise when required params are not given', done => {

‎test/unit/cis/firewall-access-rules.v1.test.js‎

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,7 @@ describe('FirewallAccessRulesV1', () => {
317317
});
318318

319319
describe('negative tests', () => {
320-
test('should enforce required parameters', async done => {
320+
test('should enforce required parameters', async () => {
321321
let err;
322322
try {
323323
await firewallAccessRulesService.deleteAccountAccessRule({});
@@ -326,7 +326,6 @@ describe('FirewallAccessRulesV1', () => {
326326
}
327327

328328
expect(err.message).toMatch(/Missing required parameters/);
329-
done();
330329
});
331330

332331
test('should reject promise when required params are not given', done => {
@@ -390,7 +389,7 @@ describe('FirewallAccessRulesV1', () => {
390389
});
391390

392391
describe('negative tests', () => {
393-
test('should enforce required parameters', async done => {
392+
test('should enforce required parameters', async () => {
394393
let err;
395394
try {
396395
await firewallAccessRulesService.getAccountAccessRule({});
@@ -399,7 +398,6 @@ describe('FirewallAccessRulesV1', () => {
399398
}
400399

401400
expect(err.message).toMatch(/Missing required parameters/);
402-
done();
403401
});
404402

405403
test('should reject promise when required params are not given', done => {
@@ -471,7 +469,7 @@ describe('FirewallAccessRulesV1', () => {
471469
});
472470

473471
describe('negative tests', () => {
474-
test('should enforce required parameters', async done => {
472+
test('should enforce required parameters', async () => {
475473
let err;
476474
try {
477475
await firewallAccessRulesService.updateAccountAccessRule({});
@@ -480,7 +478,6 @@ describe('FirewallAccessRulesV1', () => {
480478
}
481479

482480
expect(err.message).toMatch(/Missing required parameters/);
483-
done();
484481
});
485482

486483
test('should reject promise when required params are not given', done => {

‎test/unit/cis/global-load-balancer-monitor.v1.test.js‎

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -351,7 +351,7 @@ describe('GlobalLoadBalancerMonitorV1', () => {
351351
});
352352

353353
describe('negative tests', () => {
354-
test('should enforce required parameters', async done => {
354+
test('should enforce required parameters', async () => {
355355
let err;
356356
try {
357357
await globalLoadBalancerMonitorService.editLoadBalancerMonitor({});
@@ -360,7 +360,6 @@ describe('GlobalLoadBalancerMonitorV1', () => {
360360
}
361361

362362
expect(err.message).toMatch(/Missing required parameters/);
363-
done();
364363
});
365364

366365
test('should reject promise when required params are not given', done => {
@@ -426,7 +425,7 @@ describe('GlobalLoadBalancerMonitorV1', () => {
426425
});
427426

428427
describe('negative tests', () => {
429-
test('should enforce required parameters', async done => {
428+
test('should enforce required parameters', async () => {
430429
let err;
431430
try {
432431
await globalLoadBalancerMonitorService.deleteLoadBalancerMonitor({});
@@ -435,7 +434,6 @@ describe('GlobalLoadBalancerMonitorV1', () => {
435434
}
436435

437436
expect(err.message).toMatch(/Missing required parameters/);
438-
done();
439437
});
440438

441439
test('should reject promise when required params are not given', done => {
@@ -497,7 +495,7 @@ describe('GlobalLoadBalancerMonitorV1', () => {
497495
});
498496

499497
describe('negative tests', () => {
500-
test('should enforce required parameters', async done => {
498+
test('should enforce required parameters', async () => {
501499
let err;
502500
try {
503501
await globalLoadBalancerMonitorService.getLoadBalancerMonitor({});
@@ -506,7 +504,6 @@ describe('GlobalLoadBalancerMonitorV1', () => {
506504
}
507505

508506
expect(err.message).toMatch(/Missing required parameters/);
509-
done();
510507
});
511508

512509
test('should reject promise when required params are not given', done => {

‎test/unit/cis/global-load-balancer-pools.v0.test.js‎

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -307,7 +307,7 @@ describe('GlobalLoadBalancerPoolsV0', () => {
307307
});
308308

309309
describe('negative tests', () => {
310-
test('should enforce required parameters', async done => {
310+
test('should enforce required parameters', async () => {
311311
let err;
312312
try {
313313
await globalLoadBalancerPoolsService.getLoadBalancerPool({});
@@ -316,7 +316,6 @@ describe('GlobalLoadBalancerPoolsV0', () => {
316316
}
317317

318318
expect(err.message).toMatch(/Missing required parameters/);
319-
done();
320319
});
321320

322321
test('should reject promise when required params are not given', done => {
@@ -378,7 +377,7 @@ describe('GlobalLoadBalancerPoolsV0', () => {
378377
});
379378

380379
describe('negative tests', () => {
381-
test('should enforce required parameters', async done => {
380+
test('should enforce required parameters', async () => {
382381
let err;
383382
try {
384383
await globalLoadBalancerPoolsService.deleteLoadBalancerPool({});
@@ -387,7 +386,6 @@ describe('GlobalLoadBalancerPoolsV0', () => {
387386
}
388387

389388
expect(err.message).toMatch(/Missing required parameters/);
390-
done();
391389
});
392390

393391
test('should reject promise when required params are not given', done => {
@@ -483,7 +481,7 @@ describe('GlobalLoadBalancerPoolsV0', () => {
483481
});
484482

485483
describe('negative tests', () => {
486-
test('should enforce required parameters', async done => {
484+
test('should enforce required parameters', async () => {
487485
let err;
488486
try {
489487
await globalLoadBalancerPoolsService.editLoadBalancerPool({});
@@ -492,7 +490,6 @@ describe('GlobalLoadBalancerPoolsV0', () => {
492490
}
493491

494492
expect(err.message).toMatch(/Missing required parameters/);
495-
done();
496493
});
497494

498495
test('should reject promise when required params are not given', done => {

0 commit comments

Comments
 (0)