Merge pull request #488 from common-workflow-language/initialFileSizes

add size attribute to Files in the initial job object

Author: mr-c

appveyor.yml

@@ -26,3 +26,6 @@ test_script:
 
   - "%CMD_IN_ENV% python setup.py test"
 
+branches:
+  only:
+    - master

cwltool/main.py

@@ -591,7 +591,19 @@ def pathToLoc(p):
             p["location"] = p["path"]
             del p["path"]
 
+    def addSizes(p):
+        if 'location' in p:
+            try:
+                p["size"] = os.stat(p["location"][7:]).st_size  # strip off file://
+            except OSError:
+                pass
+        elif 'contents' in p:
+                p["size"] = len(p['contents'])
+        else:
+            return  # best effort
+
     visit_class(job_order_object, ("File", "Directory"), pathToLoc)
+    visit_class(job_order_object, ("File"), addSizes)
     adjustDirObjs(job_order_object, trim_listing)
     normalizeFilesDirs(job_order_object)
 

cwltool/pathmapper.py

@@ -37,7 +37,7 @@ def visit_class(rec, cls, op):  # type: (Any, Iterable, Union[Callable[..., Any]
     """Apply a function to with "class" in cls."""
 
     if isinstance(rec, dict):
-        if rec.get("class") in cls:
+        if "class" in rec and rec.get("class") in cls:
             op(rec)
         for d in rec:
             visit_class(rec[d], cls, op)