Add test case for fail uncconected ExpressionTool

Original PullRequest
common-workflow-language/cwltool#1566

Original Issue
common-workflow-language/cwltool#1330

conformance_tests.yaml

@@ -3150,6 +3150,16 @@
     }
   tags: [ required, command_line_tool ]
 
+- job: tests/empty.json
+  tool: tests/fail-unconnected-expressiontool.cwl
+  label: wf_step_access_undeclared_param_expressiontool
+  id: 258
+  doc: >-
+    Test that parameters that don't appear in the `run` process
+    inputs are not present in the input object used to run the expressiontool.
+  should_fail: true
+  tags: [ required, workflow ]
+
 - $import: tests/string-interpolation/test-index.yaml
 
 - $import: tests/conditionals/test-index.yaml

tests/fail-unconnected-expressiontool.cwl

@@ -0,0 +1,20 @@
+class: Workflow
+cwlVersion: v1.1
+inputs:
+  inp1:
+    type: string
+    default: hello inp1
+  inp2:
+    type: string
+    default: hello inp2
+outputs:
+  out:
+    type: string
+    outputSource: step1/out
+steps:
+  step1:
+    in:
+      in: inp1
+      in2: inp2
+    out: [out]
+    run: fail-unspecified-input-expressiontool.cwl

tests/fail-unspecified-input-expressiontool.cwl

@@ -0,0 +1,18 @@
+cwlVersion: v1.0
+class: ExpressionTool
+
+requirements:
+  InlineJavascriptRequirement: {} 
+
+inputs:
+  in:
+    type: string
+
+outputs:
+  out:
+    type: string
+
+expression: |
+  ${ 
+    return {"out": inputs.in +" "+inputs.in2};
+  }