Skip to content

Commit 7fb1bb1

Browse files
author
Stephen Hoover
authored
BUG container->custom in run_template (#322)
The `list_container` command works when the Custom script is running from a Container script template, but it fails for templates of other types of scripts (e.g. Python or R).
1 parent eded881 commit 7fb1bb1

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

civis/tests/test_jobs.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ def mock_client_single_json_output():
3939
}
4040
)
4141
]
42-
mock_client.scripts.list_containers_runs_outputs.return_value = mock_output
42+
mock_client.scripts.list_custom_runs_outputs.return_value = mock_output
4343
return mock_client
4444

4545

@@ -64,7 +64,7 @@ def mock_client_multiple_json_output():
6464
}
6565
),
6666
]
67-
mock_client.scripts.list_containers_runs_outputs.return_value = mock_output
67+
mock_client.scripts.list_custom_runs_outputs.return_value = mock_output
6868
return mock_client
6969

7070

@@ -81,7 +81,7 @@ def mock_client_no_json_output():
8181
}
8282
)
8383
]
84-
mock_client.scripts.list_containers_runs_outputs.return_value = mock_output
84+
mock_client.scripts.list_custom_runs_outputs.return_value = mock_output
8585
return mock_client
8686

8787

civis/utils/_jobs.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ def run_template(id, arguments, JSONValue=False, client=None):
8383
client.scripts.get_custom_runs, (job.id, run.id), client=client
8484
)
8585
fut.result()
86-
outputs = client.scripts.list_containers_runs_outputs(job.id, run.id)
86+
outputs = client.scripts.list_custom_runs_outputs(job.id, run.id)
8787
if JSONValue:
8888
json_output = [
8989
o.value for o in outputs if o.object_type == "JSONValue"

0 commit comments

Comments
 (0)