recursively insert output format

Author: jfennick

cwltool/command_line_tool.py

@@ -1495,8 +1495,19 @@ def collect_output(
                             )
                         primary["format"] = format_eval
                 else:
-                    for primary in aslist(result):
-                        primary["format"] = format_field
+
+                    def recursively_insert(j_dict: Any, key: Any, val: Any) -> Any:
+                        """Recursively inserts a value into any dictionaries"""
+                        if isinstance(j_dict, List):
+                            return [recursively_insert(x, key, val) for x in j_dict]
+                        if isinstance(j_dict, Dict):
+                            if j_dict.get("class") == "File":
+                                j_dict[key] = val
+                            else:
+                                return {x: recursively_insert(y, key, val) for x, y in j_dict.items()}
+                        return j_dict
+
+                    result = recursively_insert(result, "format", format_field)
             # Ensure files point to local references outside of the run environment
             adjustFileObjs(result, revmap)
 

tests/output_2D_file_format.cwl

@@ -0,0 +1,29 @@
+#!/usr/bin/env cwl-runner
+cwlVersion: v1.0
+
+class: CommandLineTool
+
+baseCommand: 'true'
+
+requirements:
+  InlineJavascriptRequirement: {}
+
+inputs: {}
+
+outputs:
+  output_array:
+    type: {"type": "array", "items": {"type": "array", "items": "File"}}
+    outputBinding:
+      outputEval: |
+        ${
+          var out2d = [];
+          for (var i = 0; i < 2; i++) {
+            var out1d = [];
+            for (var j = 0; j < 2; j++) {
+              out1d.push({"class": "File", "location": "../../filename.txt"});
+            }
+            out2d.push(out1d);
+          }
+          return out2d;
+        }
+    format: some_format

tests/test_2D.py

@@ -0,0 +1,17 @@
+import subprocess
+import sys
+
+from .util import get_data
+
+
+def test_output_2D_file_format() -> None:
+    """Test format tag for 2D output arrays."""
+
+    params = [
+        sys.executable,
+        "-m",
+        "cwltool",
+        get_data("tests/output_2D_file_format.cwl"),
+    ]
+
+    assert subprocess.check_call(params) == 0