[ARVADOS] updated: bb21634934ae9c16d691912f30444976bef0aa28

[Git user]

Summary of changes:
 sdk/cwl/arvados_cwl/arvjob.py      | 5 +++--
 sdk/cwl/arvados_cwl/arvworkflow.py | 2 +-
 sdk/cwl/tests/test_submit.py       | 2 +-
 3 files changed, 5 insertions(+), 4 deletions(-)

       via  bb21634934ae9c16d691912f30444976bef0aa28 (commit)
      from  cc8a6bcb59408c938c570e48bc9af0fa497cb48f (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit bb21634934ae9c16d691912f30444976bef0aa28
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Thu Sep 15 11:29:09 2016 -0400

    Fix arvados-cwl-runner tests, refs #9766

diff --git a/sdk/cwl/arvados_cwl/arvjob.py b/sdk/cwl/arvados_cwl/arvjob.py
index a796b2a..e6d0cba 100644
--- a/sdk/cwl/arvados_cwl/arvjob.py
+++ b/sdk/cwl/arvados_cwl/arvjob.py
@@ -250,6 +250,7 @@ class RunnerTemplate(object):
     type_to_dataclass = {
         'boolean': 'boolean',
         'File': 'File',
+        'Directory': 'Collection',
         'float': 'number',
         'int': 'number',
         'string': 'text',
@@ -316,8 +317,8 @@ class RunnerTemplate(object):
                 pass
             elif not isinstance(value, dict):
                 param['value'] = value
-            elif param.get('dataclass') == 'File' and value.get('location'):
-                param['value'] = value['location']
+            elif param.get('dataclass') in ('File', 'Collection') and value.get('location'):
+                param['value'] = value['location'][5:]
 
             spec['script_parameters'][param_id] = param
         spec['script_parameters']['cwl:tool'] = job_params['cwl:tool']
diff --git a/sdk/cwl/arvados_cwl/arvworkflow.py b/sdk/cwl/arvados_cwl/arvworkflow.py
index 37a0540..8e45890 100644
--- a/sdk/cwl/arvados_cwl/arvworkflow.py
+++ b/sdk/cwl/arvados_cwl/arvworkflow.py
@@ -25,7 +25,7 @@ def make_workflow(arvRunner, tool, job_order, project_uuid, update_uuid):
 
     name = os.path.basename(tool.tool["id"])
     upload_dependencies(arvRunner, name, document_loader,
-                        packed, uri, "keep:", False)
+                        packed, uri, False)
 
     body = {
         "workflow": {
diff --git a/sdk/cwl/tests/test_submit.py b/sdk/cwl/tests/test_submit.py
index b8d2a55..200f954 100644
--- a/sdk/cwl/tests/test_submit.py
+++ b/sdk/cwl/tests/test_submit.py
@@ -85,7 +85,7 @@ def stubs(func):
             'script_parameters': {
                 'x': {
                     'basename': 'blorp.txt',
-                    'location': '99999999999999999999999999999994+99/blorp.txt',
+                    'location': 'keep:99999999999999999999999999999994+99/blorp.txt',
                     'class': 'File'
                 },
                 'cwl:tool':

-----------------------------------------------------------------------


hooks/post-receive
--