From d74d84590730f040539d3a17de620d05d31db84e Mon Sep 17 00:00:00 2001 From: Chad Smith Date: Fri, 4 Feb 2022 15:03:51 -0700 Subject: [PATCH] tests: do not check instance.pull_file().ok() Test pycloudlib's BaseInstance.pull_file doesn't return a Result object. So we can't call ok() on the response in integration tests. Leave the try/except handling as pull_file will raise an IOError if there is an error connecting via paramiko's sftp.get. --- tests/integration_tests/instances.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/integration_tests/instances.py b/tests/integration_tests/instances.py index f30456fc282..e26ee233106 100644 --- a/tests/integration_tests/instances.py +++ b/tests/integration_tests/instances.py @@ -81,7 +81,7 @@ def pull_file(self, remote_path, local_path): # First copy to a temporary directory because of permissions issues tmp_path = _get_tmp_path() self.instance.execute("cp {} {}".format(str(remote_path), tmp_path)) - assert self.instance.pull_file(tmp_path, str(local_path)).ok + self.instance.pull_file(tmp_path, str(local_path)) def push_file(self, local_path, remote_path): # First push to a temporary directory because of permissions issues