Merge pull request #452 from pehala/auto_committed

Remove need to specify committed property

testsuite/openshift/__init__.py

@@ -16,22 +16,29 @@ class OpenShiftObject(APIObject, LifecycleObject):
 
     def __init__(self, dict_to_model=None, string_to_model=None, context=None):
         super().__init__(dict_to_model, string_to_model, context)
-        self.committed = False
+        self._committed = None
+
+    @property
+    def committed(self):
+        """Returns True, if the objects is already committed to the server"""
+        if self._committed is None:
+            self._committed, _ = self.exists()
+        return self._committed
 
     def commit(self):
         """
         Creates object on the server and returns created entity.
         It will be the same class but attributes might differ, due to server adding/rejecting some of them.
         """
         self.create(["--save-config=true"])
-        self.committed = True
+        self._committed = True
         return self.refresh()
 
     def delete(self, ignore_not_found=True, cmd_args=None):
         """Deletes the resource, by default ignored not found"""
         with timeout(30):
             deleted = super().delete(ignore_not_found, cmd_args)
-            self.committed = False
+            self._committed = False
             return deleted
 
     def wait_until(self, test_function, timelimit=60):

testsuite/openshift/service.py

@@ -64,5 +64,4 @@ def delete(self, ignore_not_found=True, cmd_args=None):
         """Deletes Service, introduces bigger waiting times due to LoadBalancer type"""
         with timeout(10 * 60):
             deleted = super(OpenShiftObject, self).delete(ignore_not_found, cmd_args)
-            self.committed = False
             return deleted

testsuite/tests/conftest.py

@@ -275,7 +275,6 @@ def kuadrant(request, testconfig):
     try:
         with kuadrant_openshift.context:
             kuadrant = selector("kuadrant").object(cls=KuadrantCR)
-            kuadrant.committed = True
     except OpenShiftPythonException:
         pytest.fail("Running Kuadrant tests, but Kuadrant resource was not found")