Removed success from OutputSuccess and added super

checkbox-ng/plainbox/impl/session/system_information.py

@@ -1,5 +1,6 @@
 #!/usr/bin/env python3
 
+import abc
 import json
 from subprocess import run, PIPE, check_output, STDOUT, CalledProcessError
 
@@ -61,25 +62,41 @@ def collect() -> dict:
     return CollectorMeta.collect()
 
 
-class OutputSuccess:
+class OutputABC:
+
+    @abc.abstractmethod
+    def to_dict(self):
+        """
+        Creates a dictionary from the class
+        """
+        raise NotImplementedError
+
+    @classmethod
+    @abc.abstractmethod
+    def from_dict(cls, dct) -> "OutputABC":
+        """
+        Creates this class from a dictionary
+        """
+        raise NotImplementedError
+
+
+class OutputSuccess(OutputABC):
     def __init__(self, json_output: dict, stderr: str):
         self.json_output = json_output
         self.stderr = stderr
-        self.success = True
 
     def to_dict(self):
         return {
             "json_output": self.json_output,
             "stderr": self.stderr,
-            "success": self.success,
         }
 
     @classmethod
     def from_dict(cls, dct):
         return cls(dct["json_output"], dct["stderr"])
 
 
-class OutputFailure:
+class OutputFailure(OutputABC):
     def __init__(self, stdout: str, stderr: str, return_code: int):
         self.stdout = stdout
         self.stderr = stderr
@@ -101,7 +118,7 @@ class CollectionOutput:
     def __init__(
         self,
         tool_version: str,
-        outputs: "OutputSuccess|OutputFailure",
+        outputs: "OutputABC",
     ):
         self.tool_version = tool_version
         self.outputs = outputs

checkbox-ng/plainbox/impl/session/test_system_information.py

@@ -190,7 +190,7 @@ def test_from_dict_success(self):
             collection_output.outputs.json_output, {"key": "value"}
         )
         self.assertEqual(collection_output.outputs.stderr, "")
-        self.assertTrue(collection_output.outputs.success)
+        self.assertTrue(collection_output.success)
 
     def test_from_dict_failure(self):
         input_dict = {