Skip to content

Commit c2ac7e3

Browse files
tetronmr-c
andauthored
Increase the time limits of this test and add a note. (#260)
* Increase the time limits of this test and add a note. * typo fix --------- Co-authored-by: Michael R. Crusoe <[email protected]>
1 parent c9d4ade commit c2ac7e3

File tree

1 file changed

+14
-3
lines changed

1 file changed

+14
-3
lines changed

tests/timelimit2-wf.cwl

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,20 @@
22
class: Workflow
33
cwlVersion: v1.2
44

5+
doc: |
6+
The entire test should take ~24 seconds. Test that the 20 second
7+
time limit applies to each step individually (so 1st step has 20
8+
seconds and the 2nd step has 20 seconds). So this 20 second time
9+
limit should not cause the workflow to fail.
10+
11+
The timing on this test was updated from shorter values to
12+
accommodate the startup time of certain container runners, the
13+
previous timelimit of 5 seconds was too short, which is why it is
14+
now 20 seconds.
15+
516
requirements:
617
ToolTimeLimit:
7-
timelimit: 5
18+
timelimit: 20
819
InlineJavascriptRequirement: {}
920

1021
inputs:
@@ -23,7 +34,7 @@ steps:
2334
out: [o]
2435
run:
2536
class: CommandLineTool
26-
baseCommand: ["sleep", "3"]
37+
baseCommand: ["sleep", "12"]
2738
inputs:
2839
i:
2940
type: string?
@@ -38,7 +49,7 @@ steps:
3849
out: [o]
3950
run:
4051
class: CommandLineTool
41-
baseCommand: ["sleep", "3"]
52+
baseCommand: ["sleep", "12"]
4253
inputs:
4354
i:
4455
type: string?

0 commit comments

Comments
 (0)