Fix Patch Steps (#886)

Currently the Apply/Revert Patch steps always return OK, even if they failed.

Co-authored-by: Lukas Abelt <[email protected]>

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(