* Fix Patch Steps

se-sic · May 16, 2024 · 84ac0a2 · 84ac0a2
1 parent 62cbd3a
commit 84ac0a2
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/varats-core/varats/experiment/steps/patch.py b/varats-core/varats/experiment/steps/patch.py
@@ -21,6 +21,7 @@ def __init__(self, project: VProject, patch: Patch) -> None:
         self.__patch = patch
 
     def __call__(self) -> StepResult:
+        self.status = StepResult.OK
         try:
             print(
                 f"Applying {self.__patch.shortname} to "
@@ -31,9 +32,7 @@ def __call__(self) -> StepResult:
         except ProcessExecutionError:
             self.status = StepResult.ERROR
 
-        self.status = StepResult.OK
-
-        return StepResult.OK
+        return self.status
 
     def __str__(self, indent: int = 0) -> str:
         return textwrap.indent(
@@ -53,6 +52,7 @@ def __init__(self, project: VProject, patch: Patch) -> None:
         self.__patch = patch
 
     def __call__(self) -> StepResult:
+        self.status = StepResult.OK
         try:
             print(
                 f"Reverting {self.__patch.shortname} on "
@@ -65,9 +65,7 @@ def __call__(self) -> StepResult:
         except ProcessExecutionError:
             self.status = StepResult.ERROR
 
-        self.status = StepResult.OK
-
-        return StepResult.OK
+        return self.status
 
     def __str__(self, indent: int = 0) -> str:
         return textwrap.indent(