Wrong test failure assumption

[michael-kotliar]

The following two tests fail not because of the specified in the test case reason, but because workflows cannot be parsed due to outputEval: "time passed" line

cwl-v1.2/conformance_tests.yaml

Line 2850 in 993e407

- job: tests/empty.json

cwl-v1.2/conformance_tests.yaml

Line 2876 in 993e407

- job: tests/empty.json

[michael-kotliar]

@tetron @mr-c

[mr-c]

Good catch @michael-kotliar !

Here is a suggested fix:

diff --git a/tests/timelimit-wf.cwl b/tests/timelimit-wf.cwl
index bd43ccc..168c10f 100644
--- a/tests/timelimit-wf.cwl
+++ b/tests/timelimit-wf.cwl
@@ -30,5 +30,4 @@ steps:
         o:
           type: string?
           outputBinding:
-            outputEval: "time passed"
-      
+            outputEval: $(runtime.outdir)
diff --git a/tests/timelimit4-wf.cwl b/tests/timelimit4-wf.cwl
index f7d1054..a03fa9d 100644
--- a/tests/timelimit4-wf.cwl
+++ b/tests/timelimit4-wf.cwl
@@ -33,4 +33,4 @@ steps:
         o:
           type: string?
           outputBinding:
-            outputEval: "time passed"
+            outputEval: $("time passed")

As per #54 would you like to make a PR against https://github.com/common-workflow-language/cwl-v1.2/tree/1.2.1_proposed ?