From: Ondrej Lichtner <olichtne(a)redhat.com>
There's no reason for redefinitions of the property getters the way it
was done, it only makes the code a bit more ugly. Instread I recreated
the properties from scratch where required.
Signed-off-by: Ondrej Lichtner <olichtne(a)redhat.com>
---
lnst/Controller/RecipeResults.py | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/lnst/Controller/RecipeResults.py b/lnst/Controller/RecipeResults.py
index d19d6e8..b4db8ad 100644
--- a/lnst/Controller/RecipeResults.py
+++ b/lnst/Controller/RecipeResults.py
@@ -66,17 +66,17 @@ def __init__(self, job, success):
def job(self):
return self._job
- @BaseResult.level.getter
+ @property
def level(self):
return self.job.level
- @BaseResult.data_level.getter
+ @property
def data_level(self):
return self.job.level+1
class JobStartResult(JobResult):
"""Generated automatically when a Job is succesfully started on a
slave"""
- @BaseResult.description.getter
+ @property
def description(self):
return "Job started: {}".format(str(self.job))
@@ -88,15 +88,15 @@ class JobFinishResult(JobResult):
def __init__(self, job):
super(JobFinishResult, self).__init__(job, None)
- @BaseResult.success.getter
+ @property
def success(self):
return self._job.passed
- @BaseResult.description.getter
+ @property
def description(self):
return "Job finished: {}".format(str(self.job))
- @BaseResult.data.getter
+ @property
def data(self):
return self.job.result
@@ -118,18 +118,18 @@ def __init__(self, success, description="", data=None,
if isinstance(data_level, ResultLevel)
else ResultLevel.IMPORTANT+1)
- @BaseResult.description.getter
+ @property
def description(self):
return self._description
- @BaseResult.data.getter
+ @property
def data(self):
return self._data
- @BaseResult.level.getter
+ @property
def level(self):
return self._level
- @BaseResult.data_level.getter
+ @property
def data_level(self):
return self._data_level
--
2.21.0