[RM-16979] Change use of check function.

Check function should be testable in same way as run function.

Signed-off-by: Owen Synge <osynge@suse.com>

diff --git a/ceph_deploy/util/pkg_managers.py b/ceph_deploy/util/pkg_managers.py
index 69cabd62e08cdfcfd0e4145fe8150406cf9230af..bc20eb112efd1fb2724da3b12472fc329fcb8097 100644 (file)
--- a/ceph_deploy/util/pkg_managers.py
+++ b/ceph_deploy/util/pkg_managers.py
@@ -24,6 +24,13 @@ class PackageManager(object):
             **kw
         )
 
+    def _check(self, cmd, **kw):
+        return remoto.process.check(
+            self.remote_conn,
+            cmd,
+            **kw
+        )
+
     def install(self, packages, **kw):
         """Install packages on remote node"""
         raise NotImplementedError()
@@ -306,8 +313,7 @@ class Zypper(PackageManager):
                 extra_flags = [extra_flags]
             cmd.extend(extra_flags)
         cmd.extend(packages)
-        stdout, stderr, exitrc = remoto.process.check(
-            self.remote_conn,
+        stdout, stderr, exitrc = self._check(
             cmd,
             **kw
         )