resolving TODOs

Author: cmcginley-splunk

contentctl/actions/detection_testing/infrastructures/DetectionTestingInfrastructure.py

@@ -1572,7 +1572,6 @@ def hec_raw_replay(
     def status(self):
         pass
 
-    # TODO (cmcginley): the finish function doesn't actually stop execution
     def finish(self):
         self.pbar.bar_format = (
             f"Finished running tests on instance: [{self.get_name()}]"

contentctl/objects/content_versioning_service.py

@@ -22,9 +22,6 @@
 LOG_LEVEL = logging.DEBUG
 LOG_PATH = "content_versioning_service.log"
 
-# TODO (cmcginley): would it be better for this to only run on one instance? Or to consolidate
-#   error reporting at least?
-
 
 class ContentVersioningService(BaseModel):
     """
@@ -355,7 +352,8 @@ def validate_content_against_cms(self) -> None:
         count = 100
         offset = 0
         remaining_detections = {
-            x.get_action_dot_correlationsearch_dot_label(self.global_config.app): x for x in self.detections
+            x.get_action_dot_correlationsearch_dot_label(self.global_config.app): x
+            for x in self.detections
         }
         matched_detections: dict[str, Detection] = {}