chore: pre-commit changes

Author: mpenning

ciscoconfparse2/ccp_abc.py

@@ -724,9 +724,7 @@ def insert_before(self, insertstr: str = None):
             retval = self.confobj.insert_before(exist_val=self, new_val=insertstr)
 
         elif isinstance(insertstr, BaseCfgLine) is True:
-            retval = self.confobj.insert_before(
-                exist_val=self, new_val=insertstr.text
-            )
+            retval = self.confobj.insert_before(exist_val=self, new_val=insertstr.text)
 
         else:
             raise ValueError(error)
@@ -756,9 +754,7 @@ def insert_after(self, insertstr=None):
 
         elif isinstance(insertstr, BaseCfgLine):
             # Handle insertion of a configuration line obj such as IOSCfgLine()
-            retval = self.confobj.insert_after(
-                exist_val=self, new_val=insertstr.text
-            )
+            retval = self.confobj.insert_after(exist_val=self, new_val=insertstr.text)
 
         else:
             logger.error(error)

ciscoconfparse2/ciscoconfparse2.py

@@ -1253,15 +1253,17 @@ def extend(self, other: list[BaseCfgLine] | tuple[BaseCfgLine, ...]) -> None:
     # This method is on ConfigList()
     @logger.catch(reraise=True)
     def insert_before(
-        self, exist_val: BaseCfgLine | None = None, new_val: str | BaseCfgLine | None = None
+        self,
+        exist_val: BaseCfgLine | None = None,
+        new_val: str | BaseCfgLine | None = None,
     ) -> None:
 
         if not isinstance(new_val, (str, BaseCfgLine)):
             error = f"new_val must be a str or BaseCfgLine(), not {type(new_val)}"
             logger.error(error)
             raise ValueError(error)
 
-        idx  = exist_val.linenum
+        idx = exist_val.linenum
 
         if self.factory is False:
             new_obj = CFGLINE[self.syntax](
@@ -1286,11 +1288,12 @@ def insert_before(
         # Rebuild / renumber items on the modified ConfigList()...
         self.rebuild_after_modification(commit=self.auto_commit)
 
-
     # This method is on ConfigList()
     @logger.catch(reraise=True)
     def insert_after(
-        self, exist_val: BaseCfgLine | None = None, new_val: str | BaseCfgLine | None = None
+        self,
+        exist_val: BaseCfgLine | None = None,
+        new_val: str | BaseCfgLine | None = None,
     ) -> None:
 
         if not isinstance(new_val, (str, BaseCfgLine)):
@@ -1318,16 +1321,14 @@ def insert_after(
             logger.error(error)
             raise ValueError(error)
 
-        self.data.insert(idx+1, new_obj)
+        self.data.insert(idx + 1, new_obj)
 
         # Rebuild / renumber items on the modified ConfigList()...
         self.rebuild_after_modification(commit=self.auto_commit)
 
     # This method is on ConfigList()
     @logger.catch(reraise=True)
-    def insert(
-        self, idx: int = -1, new_val: str | BaseCfgLine | None = None
-    ) -> None:
+    def insert(self, idx: int = -1, new_val: str | BaseCfgLine | None = None) -> None:
 
         if not isinstance(new_val, (str, BaseCfgLine)):
             error = f"new_val must be a str or BaseCfgLine(), not {type(new_val)}"

tests/test_CiscoConfParse.py

@@ -1498,6 +1498,7 @@ def testValues_list_insert_06():
     assert parse.config_objs[4].parent == parse.config_objs[1]
     assert parse.config_objs[5].parent == parse.config_objs[1]
 
+
 def testValues_obj_insert_07():
     """test whether we can insert_before() and find objects after the insert when one of the objects is duplicated"""
 
@@ -1514,7 +1515,7 @@ def testValues_obj_insert_07():
     uut.commit()
 
     actual_result = [ii.text for ii in uut.objs]
-    expected_result = [ "b", "c", "a", "d", "d", "e"]
+    expected_result = ["b", "c", "a", "d", "d", "e"]
 
     assert expected_result == actual_result