diff options
| author | Daniel Silverstone <daniel.silverstone@codethink.co.uk> | 2018-09-18 09:37:53 +0100 |
|---|---|---|
| committer | Daniel Silverstone <daniel.silverstone@codethink.co.uk> | 2018-09-18 09:41:32 +0100 |
| commit | 345f5f4978f0379a8bc3bfa216fa07e38154b207 (patch) | |
| tree | 2738f3198b635268f69a461d5231f6c44f6624f7 /tests | |
| parent | 7b117e407ed353529c4ec5b890cba44c987e23a4 (diff) | |
| download | buildstream-345f5f4978f0379a8bc3bfa216fa07e38154b207.tar.gz | |
tests/artifactcache/pull.py: Do not double-initialize remotes
The initialization of remotes is done by ArtifactCache.setup_remotes()
and as such it was wrong for these tests to be calling
CASCache.initialize_remotes() a second time.
Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/artifactcache/pull.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/tests/artifactcache/pull.py b/tests/artifactcache/pull.py index 6336e7ab1..45560aabf 100644 --- a/tests/artifactcache/pull.py +++ b/tests/artifactcache/pull.py @@ -137,7 +137,6 @@ def _test_pull(user_config_file, project_dir, artifact_dir, # Manually setup the CAS remote cas.setup_remotes(use_config=True) - cas.initialize_remotes() if cas.has_push_remotes(element=element): # Push the element's artifact @@ -274,7 +273,6 @@ def _test_push_tree(user_config_file, project_dir, artifact_dir, artifact_digest # Manually setup the CAS remote cas.setup_remotes(use_config=True) - cas.initialize_remotes() if cas.has_push_remotes(): directory = remote_execution_pb2.Directory() @@ -310,7 +308,6 @@ def _test_pull_tree(user_config_file, project_dir, artifact_dir, artifact_digest # Manually setup the CAS remote cas.setup_remotes(use_config=True) - cas.initialize_remotes() if cas.has_push_remotes(): # Pull the artifact using the Tree object |