From a804c108f51c806f1aa91a127433fcc3d8f8a78f Mon Sep 17 00:00:00 2001 From: Chad Smith Date: Mon, 7 Feb 2022 07:46:16 -0700 Subject: tests: do not check instance.pull_file().ok() (#1246) 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 f30456fc..e26ee233 100644 --- a/tests/integration_tests/instances.py +++ b/tests/integration_tests/instances.py @@ -81,7 +81,7 @@ class IntegrationInstance: # 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 -- cgit v1.2.3