projects
/
arvados.git
/ blobdiff
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
5827: check readable only attrvalue is a string
[arvados.git]
/
sdk
/
python
/
tests
/
test_sdk.py
diff --git
a/sdk/python/tests/test_sdk.py
b/sdk/python/tests/test_sdk.py
index 9b9f3fccef4d1ba642edeefe90afc0f1ca2ef6c4..20f0d3d56cd5b7611f78918df15b2bd29a006a77 100644
(file)
--- a/
sdk/python/tests/test_sdk.py
+++ b/
sdk/python/tests/test_sdk.py
@@
-7,10
+7,11
@@
import arvados.collection
class TestSDK(unittest.TestCase):
class TestSDK(unittest.TestCase):
- @mock.patch('arvados.api')
@mock.patch('arvados.current_task')
@mock.patch('arvados.current_job')
@mock.patch('arvados.current_task')
@mock.patch('arvados.current_job')
- def test_one_task_per_input_file_normalize(self, mock_job, mock_task, mock_api):
+ def test_one_task_per_input_file_normalize(self, mock_job, mock_task):
+ mock_api = mock.MagicMock()
+
# This manifest will be reduced from three lines to one when it is
# normalized.
nonnormalized_manifest = """. 5348b82a029fd9e971a811ce1f71360b+43 0:43:md5sum.txt
# This manifest will be reduced from three lines to one when it is
# normalized.
nonnormalized_manifest = """. 5348b82a029fd9e971a811ce1f71360b+43 0:43:md5sum.txt
@@
-30,7
+31,7
@@
class TestSDK(unittest.TestCase):
'sequence': 0,
}
# mock the API client to return a collection with a nonnormalized manifest.
'sequence': 0,
}
# mock the API client to return a collection with a nonnormalized manifest.
- mock_api
('v1')
.collections().get().execute.return_value = {
+ mock_api.collections().get().execute.return_value = {
'uuid': 'zzzzz-4zz18-mockcollection0',
'portable_data_hash': dummy_hash,
'manifest_text': nonnormalized_manifest,
'uuid': 'zzzzz-4zz18-mockcollection0',
'portable_data_hash': dummy_hash,
'manifest_text': nonnormalized_manifest,
@@
-38,5
+39,5
@@
class TestSDK(unittest.TestCase):
# Because one_task_per_input_file normalizes this collection,
# it should now create only one job task and not three.
# Because one_task_per_input_file normalizes this collection,
# it should now create only one job task and not three.
- arvados.job_setup.one_task_per_input_file(and_end_task=False)
- mock_api
('v1')
.job_tasks().create().execute.assert_called_once_with()
+ arvados.job_setup.one_task_per_input_file(and_end_task=False
, api_client=mock_api
)
+ mock_api.job_tasks().create().execute.assert_called_once_with()