Fix style issues

Author: hwikle-lanl

lib/pavilion/commands/cat.py

@@ -44,7 +44,7 @@ def run(self, pav_cfg, args):
                 output.fprint(self.errfile, "No last test found.", color=output.RED)
                 return 1
         elif TestID.is_valid_id(args.test_id):
-            test_id = TestID(args.test_id) 
+            test_id = TestID(args.test_id)
         else:
             output.fprint(self.errfile, f"{args.test_id} is not a valid test ID.")
             return errno.EEXIST

lib/pavilion/commands/log.py

@@ -149,7 +149,7 @@ def run(self, pav_cfg, args):
         if cmd_name == 'states':
             if args.id is None:
                 args.id = cmd_utils.get_last_test_id(pav_cfg, self.errfile)
-                
+
             if args.id is None:
                 output.fprint(self.errfile, "No last test found.", color=output.RED)
                 return 1

lib/pavilion/commands/set_status.py

@@ -47,7 +47,7 @@ def run(self, pav_cfg, args):
             return 0
 
         if TestID.is_valid_id(args.test):
-            test_id = TestID(args.test) 
+            test_id = TestID(args.test)
         else:
             output.fprint(self.errfile, f"{args.test} is not a valid test ID.")
             return errno.EEXIST

lib/pavilion/test_ids.py

@@ -10,18 +10,12 @@ class ID(ABC):
 
     def __init__(self, id_str: str):
         self.id_str = id_str
+        self.range_type = None
 
     @staticmethod
     @abstractmethod
     def is_valid_id(id_str: str) -> bool:
         """Determine whether the given string constitutes a valid ID."""
-        
-        raise NotImplementedError
-
-    @property
-    @abstractmethod
-    def range_type() -> type:
-        """Return the range ID type associated with the ID."""
 
         raise NotImplementedError
 
@@ -41,6 +35,10 @@ def __hash__(self) -> int:
 class TestID(ID):
     """Represents a single test ID."""
 
+    def __init__(self):
+        super().__init__()
+        self.range_type = TestRange
+
     @classmethod
     def is_valid_id(cls, id_str: str) -> bool:
         """Determine whether the given string constitutes a valid test ID."""
@@ -92,14 +90,14 @@ def test_num(self) -> Optional[int]:
         elif len(self.parts) > 1:
             return int(self.parts[-1])
 
-    @property
-    def range_type() -> type:
-        return TestRange
-
 
 class SeriesID(ID):
     """Represents a single series ID."""
 
+    def __init__(self):
+        super().__init__()
+        self.range_type = SeriesRange
+
     @classmethod
     def is_valid_id(cls, id_str: str) -> bool:
         """Determine whether the given string constitutes a valid series ID."""
@@ -144,14 +142,10 @@ def as_int(self) -> int:
 
         return int(self.id_str[1:])
 
-    @property
-    def range_type() -> type:
-        return SeriesRange
-
 
 class GroupID:
     """Represents a single group ID."""
-    
+
     def __init__(self, id_str: str):
         self.id_str = id_str
 
@@ -303,7 +297,7 @@ def resolve_ids(id_strs: List[str], id_type: ID, auto_last: bool = True) -> List
             return [id_type("last")]
 
         return ids
-    
+
     if "all" in id_strs:
         return [id_type("all")]
 
@@ -339,7 +333,8 @@ def multi_convert(id_str: str) -> Union[List[TestID], List[SeriesID], List[Group
     return [GroupID(id_str)]
 
 
-def resolve_mixed_ids(ids: Iterable[str], auto_last: bool = True) -> List[Union[TestID, SeriesID, GroupID]]:
+def resolve_mixed_ids(ids: Iterable[str],
+                      auto_last: bool = True) -> List[Union[TestID, SeriesID, GroupID]]:
     """Fully resolve all IDs in the given list into either test IDs, series IDs, or group IDs."""
 
     ids = list(ids)