test_host_plugin: Use HostTracker fixtures

The racker object "remembers" expected state across several tests,
so only changes (rather than all expected state) need to be specified
in each test. Also, the tracker fixture will make it easy to use hosts
in other test modules.

This change makes the tests independent; any permutation of any subset
of these tests should now pass.

Reviewed-By: Tomas Babej <tbabej@redhat.com>

1 files changed, 0 insertions, 29 deletions

diff --git a/ipatests/test_xmlrpc/xmlrpc_test.py b/ipatests/test_xmlrpc/xmlrpc_test.py
index acfdf711d..808abae19 100644
--- a/ipatests/test_xmlrpc/xmlrpc_test.py
+++ b/ipatests/test_xmlrpc/xmlrpc_test.py
@@ -367,32 +367,3 @@ def raises_exact(expected_exception):
         assert expected_exception.strerror == got_exception.strerror
     else:
         raise AssertionError('did not raise!')
-
-
-class RPCTest(XMLRPC_test):
-    """Base class for RPC tests"""
-    @classmethod
-    def setup_class(cls):
-        super(RPCTest, cls).setup_class()
-        cls.clean_up()
-
-    @classmethod
-    def teardown_class(cls):
-        cls.clean_up()
-        super(RPCTest, cls).teardown_class()
-
-    @classmethod
-    def clean_up(self):
-        """Cleanup run on both setup and teardown
-
-        To be overridden in subclasses.
-        Usually calls the clean() method.
-        """
-
-    @classmethod
-    def clean(cls, command, *args, **options):
-        """Run a command, ignoring NotFound/EmptyModlist errors"""
-        try:
-            api.Command[command](*args, **options)
-        except (errors.NotFound, errors.EmptyModlist):
-            pass