So we can move install_what_provides there and simplify
the SoftwareManager class.

Signed-off-by: Lucas Meneghel Rodrigues <l...@redhat.com>
---
 client/shared/software_manager.py |   13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/client/shared/software_manager.py 
b/client/shared/software_manager.py
index 2712e68..c27262d 100755
--- a/client/shared/software_manager.py
+++ b/client/shared/software_manager.py
@@ -130,12 +130,13 @@ class SoftwareManager(object):
 
 
     def __getattr__(self, name):
-        if name == "install_what_provides":
-            return self.install_what_provides(file)
-        else:
-            return self.backend.__getattribute__(self, name)
+        return self.backend.__getattribute__(name)
 
 
+class BaseBackend(object):
+    """
+    This class implements all common methods among backends.
+    """
     def install_what_provides(self, file):
         """
         Installs package that provides [file].
@@ -149,7 +150,7 @@ class SoftwareManager(object):
             logging.warning('No package seems to provide %s', file)
 
 
-class RpmBackend(object):
+class RpmBackend(BaseBackend):
     """
     This class implements operations executed with the rpm package manager.
 
@@ -234,7 +235,7 @@ class RpmBackend(object):
             return []
 
 
-class DpkgBackend(object):
+class DpkgBackend(BaseBackend):
     """
     This class implements operations executed with the dpkg package manager.
 
-- 
1.7.10.4

_______________________________________________
Autotest mailing list
Autotest@test.kernel.org
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to