openshift_checks: get rid of deprecated module_executor

roles/openshift_health_checker/openshift_checks/__init__.py

@@ -21,13 +21,12 @@ class OpenShiftCheckException(Exception):
 class OpenShiftCheck(object):
     """A base class for defining checks for an OpenShift cluster environment."""
 
-    def __init__(self, execute_module=None, module_executor=None):
-        if execute_module is module_executor is None:
-            raise TypeError(
-                "__init__() takes either execute_module (recommended) "
-                "or module_executor (deprecated), none given")
-        self.execute_module = execute_module or module_executor
-        self.module_executor = self.execute_module
+    def __init__(self, execute_module=None):
+        def placeholder(*_):
+            """Fail tests more helpfully when execute_module not provided."""
+            raise TypeError(self.__class__.__name__ +
+                            " invoked execute_module without providing the method at initialization.")
+        self.execute_module = execute_module or placeholder
 
     @abstractproperty
     def name(self):

roles/openshift_health_checker/openshift_checks/etcd_imagedata_size.py

@@ -46,7 +46,7 @@ class EtcdImageDataSize(OpenShiftCheck):
                 },
             }
 
-            etcdkeysize = self.module_executor("etcdkeysize", args, task_vars)
+            etcdkeysize = self.execute_module("etcdkeysize", args, task_vars)
 
             if etcdkeysize.get("rc", 0) != 0 or etcdkeysize.get("failed"):
                 msg = 'Failed to retrieve stats for etcd host "{host}": {reason}'

roles/openshift_health_checker/openshift_checks/logging/curator.py

@@ -15,7 +15,7 @@ class Curator(LoggingCheck):
     def run(self, tmp, task_vars):
         self.logging_namespace = get_var(task_vars, "openshift_logging_namespace", default="logging")
         curator_pods, error = super(Curator, self).get_pods_for_component(
-            self.module_executor,
+            self.execute_module,
             self.logging_namespace,
             "curator",
             task_vars

roles/openshift_health_checker/test/openshift_check_test.py

@@ -30,32 +30,18 @@ def test_OpenShiftCheck_init():
 
     # initialization requires at least one argument (apart from self)
     with pytest.raises(TypeError) as excinfo:
-        TestCheck()
+        TestCheck().execute_module("foo")
     assert 'execute_module' in str(excinfo.value)
-    assert 'module_executor' in str(excinfo.value)
 
     execute_module = object()
 
     # initialize with positional argument
     check = TestCheck(execute_module)
-    # new recommended name
     assert check.execute_module == execute_module
-    # deprecated attribute name
-    assert check.module_executor == execute_module
 
-    # initialize with keyword argument, recommended name
+    # initialize with keyword argument
     check = TestCheck(execute_module=execute_module)
-    # new recommended name
     assert check.execute_module == execute_module
-    # deprecated attribute name
-    assert check.module_executor == execute_module
-
-    # initialize with keyword argument, deprecated name
-    check = TestCheck(module_executor=execute_module)
-    # new recommended name
-    assert check.execute_module == execute_module
-    # deprecated attribute name
-    assert check.module_executor == execute_module
 
 
 def test_subclasses():