Skip to content

Commit efaafbe

Browse files
committed
Adjust tests for max-retries
1 parent 5fdf7cd commit efaafbe

File tree

1 file changed

+33
-11
lines changed

1 file changed

+33
-11
lines changed

index.test.js

Lines changed: 33 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,9 @@ describe('Deploy to ECS', () => {
5757
core.getInput = jest
5858
.fn()
5959
.mockReturnValueOnce('task-definition.json') // task-definition
60-
.mockReturnValueOnce('service-456') // service
61-
.mockReturnValueOnce('cluster-789'); // cluster
60+
.mockReturnValueOnce('service-456') // service
61+
.mockReturnValueOnce('cluster-789') // cluster
62+
.mockReturnValueOnce('3'); // max-retries
6263

6364
process.env = Object.assign(process.env, { GITHUB_WORKSPACE: __dirname });
6465

@@ -95,7 +96,7 @@ describe('Deploy to ECS', () => {
9596
mockEcsDescribeServices.mockImplementation(
9697
() => Promise.resolve({
9798
failures: [],
98-
services: [{
99+
services: [{
99100
status: 'ACTIVE'
100101
}]
101102
})
@@ -487,6 +488,7 @@ describe('Deploy to ECS', () => {
487488
.mockReturnValueOnce('task-definition.json') // task-definition
488489
.mockReturnValueOnce('service-456') // service
489490
.mockReturnValueOnce('cluster-789') // cluster
491+
.mockReturnValueOnce('3') // max-retries
490492
.mockReturnValueOnce('TRUE'); // wait-for-service-stability
491493

492494
mockEcsDescribeServices.mockImplementation(
@@ -564,6 +566,7 @@ describe('Deploy to ECS', () => {
564566
.mockReturnValueOnce('task-definition.json') // task-definition
565567
.mockReturnValueOnce('service-456') // service
566568
.mockReturnValueOnce('cluster-789') // cluster
569+
.mockReturnValueOnce('3') // max-retries
567570
.mockReturnValueOnce('TRUE') // wait-for-service-stability
568571
.mockReturnValueOnce('60'); // wait-for-minutes
569572

@@ -640,6 +643,7 @@ describe('Deploy to ECS', () => {
640643
.mockReturnValueOnce('task-definition.json') // task-definition
641644
.mockReturnValueOnce('service-456') // service
642645
.mockReturnValueOnce('cluster-789') // cluster
646+
.mockReturnValueOnce('3') // max-retries
643647
.mockReturnValueOnce('TRUE') // wait-for-service-stability
644648
.mockReturnValueOnce('1000'); // wait-for-minutes
645649

@@ -712,6 +716,7 @@ describe('Deploy to ECS', () => {
712716
.mockReturnValueOnce('task-definition.json') // task-definition
713717
.mockReturnValueOnce('service-456') // service
714718
.mockReturnValueOnce('cluster-789') // cluster
719+
.mockReturnValueOnce('3') // max-retries
715720
.mockReturnValueOnce('false') // wait-for-service-stability
716721
.mockReturnValueOnce('') // wait-for-minutes
717722
.mockReturnValueOnce('') // force-new-deployment
@@ -939,6 +944,7 @@ describe('Deploy to ECS', () => {
939944
.mockReturnValueOnce('task-definition.json') // task-definition
940945
.mockReturnValueOnce('service-456') // service
941946
.mockReturnValueOnce('cluster-789') // cluster
947+
.mockReturnValueOnce('3') // max-retries
942948
.mockReturnValueOnce('TRUE') // wait-for-service-stability
943949
.mockReturnValueOnce(''); // desired count
944950

@@ -980,6 +986,7 @@ describe('Deploy to ECS', () => {
980986
.mockReturnValueOnce('task-definition.json') // task-definition
981987
.mockReturnValueOnce('service-456') // service
982988
.mockReturnValueOnce('cluster-789') // cluster
989+
.mockReturnValueOnce('3') // max-retries
983990
.mockReturnValueOnce('TRUE') // wait-for-service-stability
984991
.mockReturnValueOnce('60') // wait-for-minutes
985992
.mockReturnValueOnce(''); // desired count
@@ -1022,6 +1029,7 @@ describe('Deploy to ECS', () => {
10221029
.mockReturnValueOnce('task-definition.json') // task-definition
10231030
.mockReturnValueOnce('service-456') // service
10241031
.mockReturnValueOnce('cluster-789') // cluster
1032+
.mockReturnValueOnce('3') // max-retries
10251033
.mockReturnValueOnce('TRUE') // wait-for-service-stability
10261034
.mockReturnValueOnce('1000') // wait-for-minutes
10271035
.mockReturnValueOnce('abc'); // desired count is NaN
@@ -1064,6 +1072,7 @@ describe('Deploy to ECS', () => {
10641072
.mockReturnValueOnce('task-definition.json') // task-definition
10651073
.mockReturnValueOnce('service-456') // service
10661074
.mockReturnValueOnce('cluster-789') // cluster
1075+
.mockReturnValueOnce('3') // max-retries
10671076
.mockReturnValueOnce('false') // wait-for-service-stability
10681077
.mockReturnValueOnce('') // wait-for-minutes
10691078
.mockReturnValueOnce('true') // force-new-deployment
@@ -1137,6 +1146,7 @@ describe('Deploy to ECS', () => {
11371146
.mockReturnValueOnce('task-definition.json') // task-definition
11381147
.mockReturnValueOnce('') // service
11391148
.mockReturnValueOnce('') // cluster
1149+
.mockReturnValueOnce('3') // max-retries
11401150
.mockReturnValueOnce('') // wait-for-service-stability
11411151
.mockReturnValueOnce('') // wait-for-minutes
11421152
.mockReturnValueOnce('') // enable-ecs-managed-tags
@@ -1173,6 +1183,7 @@ describe('Deploy to ECS', () => {
11731183
.mockReturnValueOnce('task-definition.json') // task-definition
11741184
.mockReturnValueOnce('') // service
11751185
.mockReturnValueOnce('somecluster') // cluster
1186+
.mockReturnValueOnce('3') // max-retries
11761187
.mockReturnValueOnce('') // wait-for-service-stability
11771188
.mockReturnValueOnce('') // wait-for-minutes
11781189
.mockReturnValueOnce('') // force-new-deployment
@@ -1215,6 +1226,7 @@ describe('Deploy to ECS', () => {
12151226
.mockReturnValueOnce('task-definition.json') // task-definition
12161227
.mockReturnValueOnce('') // service
12171228
.mockReturnValueOnce('somecluster') // cluster
1229+
.mockReturnValueOnce('3') // max-retries
12181230
.mockReturnValueOnce('') // wait-for-service-stability
12191231
.mockReturnValueOnce('') // wait-for-minutes
12201232
.mockReturnValueOnce('') // force-new-deployment
@@ -1258,6 +1270,7 @@ describe('Deploy to ECS', () => {
12581270
.mockReturnValueOnce('task-definition.json') // task-definition
12591271
.mockReturnValueOnce('service-456') // service
12601272
.mockReturnValueOnce('somecluster') // cluster
1273+
.mockReturnValueOnce('3') // max-retries
12611274
.mockReturnValueOnce('true') // wait-for-service-stability
12621275
.mockReturnValueOnce('') // wait-for-minutes
12631276
.mockReturnValueOnce('') // force-new-deployment
@@ -1271,7 +1284,7 @@ describe('Deploy to ECS', () => {
12711284
.mockReturnValueOnce('a,b') // run-task-subnet-ids
12721285
.mockReturnValueOnce('c,d') // run-task-security-group-ids
12731286
.mockReturnValueOnce(JSON.stringify([{ name: 'someapp', command: 'somecmd' }])); // run-task-container-overrides
1274-
1287+
12751288
await run();
12761289
expect(core.setFailed).toHaveBeenCalledTimes(0);
12771290

@@ -1311,6 +1324,7 @@ describe('Deploy to ECS', () => {
13111324
.mockReturnValueOnce('task-definition.json') // task-definition
13121325
.mockReturnValueOnce('') // service
13131326
.mockReturnValueOnce('somecluster') // cluster
1327+
.mockReturnValueOnce('3') // max-retries
13141328
.mockReturnValueOnce('') // wait-for-service-stability
13151329
.mockReturnValueOnce('') // wait-for-minutes
13161330
.mockReturnValueOnce('') // force-new-deployment
@@ -1336,6 +1350,7 @@ describe('Deploy to ECS', () => {
13361350
.mockReturnValueOnce('task-definition.json') // task-definition
13371351
.mockReturnValueOnce('service-456') // service
13381352
.mockReturnValueOnce('somecluster') // cluster
1353+
.mockReturnValueOnce('3') // max-retries
13391354
.mockReturnValueOnce('true') // wait-for-service-stability
13401355
.mockReturnValueOnce('') // wait-for-minutes
13411356
.mockReturnValueOnce('') // enable-ecs-managed-tags
@@ -1365,13 +1380,14 @@ describe('Deploy to ECS', () => {
13651380
volumeConfigurations: []
13661381
});
13671382
});
1368-
1383+
13691384
test('run task with setting true to enableECSManagedTags', async () => {
13701385
core.getInput = jest
13711386
.fn()
13721387
.mockReturnValueOnce('task-definition.json') // task-definition
13731388
.mockReturnValueOnce('') // service
13741389
.mockReturnValueOnce('somecluster') // cluster
1390+
.mockReturnValueOnce('3') // max-retries
13751391
.mockReturnValueOnce('') // wait-for-service-stability
13761392
.mockReturnValueOnce('') // wait-for-minutes
13771393
.mockReturnValueOnce('') // force-new-deployment
@@ -1394,13 +1410,14 @@ describe('Deploy to ECS', () => {
13941410
volumeConfigurations: []
13951411
});
13961412
});
1397-
1413+
13981414
test('run task with setting false to enableECSManagedTags', async () => {
13991415
core.getInput = jest
14001416
.fn()
14011417
.mockReturnValueOnce('task-definition.json') // task-definition
14021418
.mockReturnValueOnce('') // service
14031419
.mockReturnValueOnce('somecluster') // cluster
1420+
.mockReturnValueOnce('3') // max-retries
14041421
.mockReturnValueOnce('') // wait-for-service-stability
14051422
.mockReturnValueOnce('') // wait-for-minutes
14061423
.mockReturnValueOnce('') // force-new-deployment
@@ -1430,6 +1447,7 @@ describe('Deploy to ECS', () => {
14301447
.mockReturnValueOnce('task-definition.json') // task-definition
14311448
.mockReturnValueOnce('') // service
14321449
.mockReturnValueOnce('somecluster') // cluster
1450+
.mockReturnValueOnce('3') // max-retries
14331451
.mockReturnValueOnce('') // wait-for-service-stability
14341452
.mockReturnValueOnce('') // wait-for-minutes
14351453
.mockReturnValueOnce('') // force-new-deployment
@@ -1473,6 +1491,7 @@ describe('Deploy to ECS', () => {
14731491
.mockReturnValueOnce('task-definition.json') // task-definition
14741492
.mockReturnValueOnce('') // service
14751493
.mockReturnValueOnce('somecluster') // cluster
1494+
.mockReturnValueOnce('3') // max-retries
14761495
.mockReturnValueOnce('') // wait-for-service-stability
14771496
.mockReturnValueOnce('') // wait-for-minutes
14781497
.mockReturnValueOnce('') // force-new-deployment
@@ -1481,7 +1500,7 @@ describe('Deploy to ECS', () => {
14811500
.mockReturnValueOnce('') // propagate-tags
14821501
.mockReturnValueOnce('true') // run-task
14831502
.mockReturnValueOnce('false'); // wait-for-task-stopped
1484-
1503+
14851504
mockRunTask.mockImplementation(
14861505
() => Promise.resolve({
14871506
failures: [{
@@ -1596,12 +1615,13 @@ describe('Deploy to ECS', () => {
15961615
.mockReturnValueOnce('task-definition.json') // task-definition
15971616
.mockReturnValueOnce('service-456') // service
15981617
.mockReturnValueOnce('cluster-789') // cluster
1618+
.mockReturnValueOnce('3') // max-retries
15991619
.mockReturnValueOnce('false') // wait-for-service-stability
16001620
.mockReturnValueOnce('') // wait-for-minutes
16011621
.mockReturnValueOnce('') // force-new-deployment
16021622
.mockReturnValueOnce('') // desired-count
16031623
.mockReturnValueOnce('') // enable-ecs-managed-tags
1604-
.mockReturnValueOnce('SERVICE'); // propagate-tags
1624+
.mockReturnValueOnce('SERVICE'); // propagate-tags
16051625

16061626
await run();
16071627
expect(core.setFailed).toHaveBeenCalledTimes(0);
@@ -1622,13 +1642,14 @@ describe('Deploy to ECS', () => {
16221642
volumeConfigurations: []
16231643
});
16241644
});
1625-
1645+
16261646
test('update service with setting true to enableECSManagedTags', async () => {
16271647
core.getInput = jest
16281648
.fn()
16291649
.mockReturnValueOnce('task-definition.json') // task-definition
16301650
.mockReturnValueOnce('service-456') // service
16311651
.mockReturnValueOnce('cluster-789') // cluster
1652+
.mockReturnValueOnce('3') // max-retries
16321653
.mockReturnValueOnce('false') // wait-for-service-stability
16331654
.mockReturnValueOnce('') // wait-for-minutes
16341655
.mockReturnValueOnce('') // force-new-deployment
@@ -1655,13 +1676,14 @@ describe('Deploy to ECS', () => {
16551676
volumeConfigurations: []
16561677
});
16571678
});
1658-
1679+
16591680
test('update service with setting false to enableECSManagedTags', async () => {
16601681
core.getInput = jest
16611682
.fn()
16621683
.mockReturnValueOnce('task-definition.json') // task-definition
16631684
.mockReturnValueOnce('service-456') // service
16641685
.mockReturnValueOnce('cluster-789') // cluster
1686+
.mockReturnValueOnce('3') // max-retries
16651687
.mockReturnValueOnce('false') // wait-for-service-stability
16661688
.mockReturnValueOnce('') // wait-for-minutes
16671689
.mockReturnValueOnce('') // force-new-deployment
@@ -1915,4 +1937,4 @@ describe('Deploy to ECS', () => {
19151937
}]
19161938
});
19171939
});
1918-
});
1940+
});

0 commit comments

Comments
 (0)