X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/570509ab4d2ef93d870fd2b1f2eab178afb1bad9..670b4ba238d93910ff087794d359da0d5ac469fa:/sdk/cwl/tests/test_submit.py diff --git a/sdk/cwl/tests/test_submit.py b/sdk/cwl/tests/test_submit.py index d3bdf8fedc..7faef6992c 100644 --- a/sdk/cwl/tests/test_submit.py +++ b/sdk/cwl/tests/test_submit.py @@ -1,16 +1,17 @@ -import arvados -import arvados.keep -import arvados.collection -import arvados_cwl import copy import cStringIO import functools import hashlib +import json +import logging import mock import sys import unittest -import json -import logging + +import arvados +import arvados.collection +import arvados_cwl +import arvados.keep from .matcher import JsonDiffMatcher @@ -38,6 +39,7 @@ def stubs(func): stubs.api = mock.MagicMock() + stubs.api._rootDesc = arvados.api('v1')._rootDesc stubs.api.users().current().execute.return_value = { "uuid": stubs.fake_user_uuid, } @@ -89,7 +91,7 @@ def stubs(func): } stubs.expect_job_spec = { 'runtime_constraints': { - 'docker_image': 'arvados/jobs' + 'docker_image': 'arvados/jobs:'+arvados_cwl.__version__ }, 'script_parameters': { 'x': { @@ -115,7 +117,7 @@ def stubs(func): '99999999999999999999999999999991+99/wf/submit_wf.cwl' }, 'repository': 'arvados', - 'script_version': 'master', + 'script_version': arvados_cwl.__version__, 'script': 'cwl-runner' } stubs.pipeline_component = stubs.expect_job_spec.copy() @@ -124,7 +126,7 @@ def stubs(func): 'state': 'RunningOnServer', "components": { "cwl-runner": { - 'runtime_constraints': {'docker_image': 'arvados/jobs'}, + 'runtime_constraints': {'docker_image': 'arvados/jobs:'+arvados_cwl.__version__}, 'script_parameters': { 'y': {"value": {'basename': '99999999999999999999999999999998+99', 'location': 'keep:99999999999999999999999999999998+99', 'class': 'Directory'}}, 'x': {"value": {'basename': 'blorp.txt', 'class': 'File', 'location': 'keep:99999999999999999999999999999994+99/blorp.txt'}}, @@ -135,8 +137,9 @@ def stubs(func): 'cwl:tool': '99999999999999999999999999999991+99/wf/submit_wf.cwl' }, 'repository': 'arvados', - 'script_version': 'master', - 'script': 'cwl-runner' + 'script_version': arvados_cwl.__version__, + 'script': 'cwl-runner', + 'job': {'state': 'Queued', 'uuid': 'zzzzz-8i9sb-zzzzzzzzzzzzzzz'} } } } @@ -175,7 +178,7 @@ def stubs(func): 'owner_uuid': 'zzzzz-tpzed-zzzzzzzzzzzzzzz', 'command': ['arvados-cwl-runner', '--local', '--api=containers', '/var/lib/cwl/workflow/submit_wf.cwl', '/var/lib/cwl/job/cwl.input.json'], 'name': 'submit_wf.cwl', - 'container_image': '99999999999999999999999999999993+99', + 'container_image': 'arvados/jobs:'+arvados_cwl.__version__, 'output_path': '/var/spool/cwl', 'cwd': '/var/spool/cwl', 'runtime_constraints': { @@ -390,7 +393,7 @@ class TestTemplateInputs(unittest.TestCase): "components": { "inputs_test.cwl": { 'runtime_constraints': { - 'docker_image': 'arvados/jobs', + 'docker_image': 'arvados/jobs:'+arvados_cwl.__version__, }, 'script_parameters': { 'cwl:tool': @@ -425,7 +428,7 @@ class TestTemplateInputs(unittest.TestCase): }, }, 'repository': 'arvados', - 'script_version': 'master', + 'script_version': arvados_cwl.__version__, 'script': 'cwl-runner', }, },