linter fix 2

Author: KarolinaPomian

tests/validation/Engine/csv_report.py

@@ -44,6 +44,7 @@
 _test_results = []
 _compliance_results = {}
 
+
 def initialize_csv_report(log_dir: Optional[str] = None) -> None:
     """
     Initialize CSV report files.
@@ -92,7 +93,7 @@ def csv_add_test(
             TEST_RESULT_COLUMN: test_result,
             TEST_TIME_COLUMN: test_time,
             TEST_DURATION_COLUMN: f"{duration:.2f}s",
-            TEST_ERROR_COLUMN: error_message
+            TEST_ERROR_COLUMN: error_message,
         }
     )
 
@@ -110,9 +111,7 @@ def update_compliance_result(requirement_id: str, is_compliant: bool) -> None:
     global _compliance_results
 
     _compliance_results[requirement_id] = is_compliant
-    logger.info(
-        f"Updated compliance for {requirement_id}: {'COMPLIANT' if is_compliant else 'NON-COMPLIANT'}"
-    )
+    logger.info(f"Updated compliance for {requirement_id}: {'COMPLIANT' if is_compliant else 'NON-COMPLIANT'}")
 
 
 def csv_write_report(log_dir: str = ".") -> None:
@@ -128,7 +127,7 @@ def csv_write_report(log_dir: str = ".") -> None:
     # Write test results
     if _test_results:
         try:
-            with open(report_path, "w", newline='') as csvfile:
+            with open(report_path, "w", newline="") as csvfile:
                 writer = csv.DictWriter(csvfile, fieldnames=CSV_HEADERS)
                 writer.writeheader()
                 for result in _test_results:
@@ -140,16 +139,14 @@ def csv_write_report(log_dir: str = ".") -> None:
     # Write compliance results
     if _compliance_results:
         try:
-            with open(compliance_path, "w", newline='') as csvfile:
-                writer = csv.DictWriter(
-                    csvfile, fieldnames=['Requirement ID', 'Status']
-                )
+            with open(compliance_path, "w", newline="") as csvfile:
+                writer = csv.DictWriter(csvfile, fieldnames=["Requirement ID", "Status"])
                 writer.writeheader()
                 for req_id, is_compliant in _compliance_results.items():
                     writer.writerow(
                         {
-                            'Requirement ID': req_id,
-                            'Status': 'COMPLIANT' if is_compliant else 'NON-COMPLIANT',
+                            "Requirement ID": req_id,
+                            "Status": "COMPLIANT" if is_compliant else "NON-COMPLIANT",
                         }
                     )
             logger.info(f"Compliance report written to {compliance_path}")
@@ -172,9 +169,7 @@ def get_test_summary() -> Dict[str, int | str]:
         "total": total_count,
         "passed": pass_count,
         "failed": fail_count,
-        "pass_rate": (
-            f"{(pass_count / total_count * 100):.1f}%" if total_count > 0 else "N/A"
-        ),
+        "pass_rate": (f"{(pass_count / total_count * 100):.1f}%" if total_count > 0 else "N/A"),
     }
 
 
@@ -191,7 +186,7 @@ def get_component_summary() -> Dict[str, Dict[str, int | str]]:
         component = result[COMPONENT_COLUMN]
         if component not in components:
             components[component] = {"total": 0, "passed": 0, "failed": 0}
-        
+
         components[component]["total"] += 1
         if result[TEST_RESULT_COLUMN] == "PASS":
             components[component]["passed"] += 1
@@ -202,9 +197,7 @@ def get_component_summary() -> Dict[str, Dict[str, int | str]]:
     for component in components:
         total = components[component]["total"]
         passed = components[component]["passed"]
-        components[component]["pass_rate"] = (
-            f"{(passed / total * 100):.1f}%" if total > 0 else "N/A"
-        )
+        components[component]["pass_rate"] = f"{(passed / total * 100):.1f}%" if total > 0 else "N/A"
 
     return components
 
@@ -224,9 +217,5 @@ def get_compliance_summary() -> Dict[str, int | str]:
         "total": total_count,
         "compliant": compliant_count,
         "non_compliant": non_compliant_count,
-        "compliance_rate": (
-            f"{(compliant_count / total_count * 100):.1f}%"
-            if total_count > 0
-            else "N/A"
-        ),
+        "compliance_rate": (f"{(compliant_count / total_count * 100):.1f}%" if total_count > 0 else "N/A"),
     }

tests/validation/Engine/html_report.py

@@ -41,15 +41,15 @@ def generate_html_report(log_dir: str = ".") -> None:
 
     # Read CSV data
     test_results = []
-    with open(report_path, "r", newline='') as csvfile:
+    with open(report_path, "r", newline="") as csvfile:
         reader = csv.DictReader(csvfile)
         for row in reader:
             test_results.append(row)
 
     # Read compliance data if available
     compliance_results = []
     if compliance_path.exists():
-        with open(compliance_path, "r", newline='') as csvfile:
+        with open(compliance_path, "r", newline="") as csvfile:
             reader = csv.DictReader(csvfile)
             for row in reader:
                 compliance_results.append(row)
@@ -231,9 +231,10 @@ def generate_html_report(log_dir: str = ".") -> None:
                     <p>Pass Rate: <strong>{summary['pass_rate']}</strong></p>
                 </div>
 """
-        )
+            )
 
-        f.write("""
+        f.write(
+            """
             </div>
         </div>
 """
@@ -255,10 +256,11 @@ def generate_html_report(log_dir: str = ".") -> None:
             </div>
         </div>
 """
-        )
+            )
 
         # Test results filter
-        f.write("""
+        f.write(
+            """
         <h2>Test Results</h2>
         <div class="filter-container">
             <label>
@@ -294,11 +296,11 @@ def generate_html_report(log_dir: str = ".") -> None:
 
         # Test results
         for result in test_results:
-            component = result.get('Component', '')
+            component = result.get("Component", "")
             component_class = component.lower().replace(" ", "-")
-            result_class = 'pass' if result.get('Test Result') == 'PASS' else 'fail'
-            error_msg = result.get('Error Message', '')
-            
+            result_class = "pass" if result.get("Test Result") == "PASS" else "fail"
+            error_msg = result.get("Error Message", "")
+
             f.write(
                 f"""
                 <tr class="{result_class}" data-component="{component}">
@@ -309,34 +311,37 @@ def generate_html_report(log_dir: str = ".") -> None:
                     <td>{result.get('Test Time', '')}</td>
                     <td>
 """
-        )
+            )
             if error_msg:
                 f.write(
-                f"""
+                    f"""
                         <button class="collapsible">Show Error Details</button>
                         <div class="content">
                             <div class="error-detail">{error_msg}</div>
                         </div>
 """
-        )
+                )
             else:
                 f.write("N/A")
-            
-            f.write("""
+
+            f.write(
+                """
                     </td>
                 </tr>
 """
-        )
+            )
 
-        f.write("""
+        f.write(
+            """
             </tbody>
         </table>
 """
         )
 
         # Compliance results if available
         if compliance_results:
-            f.write("""
+            f.write(
+                """
         <h2>Compliance Results</h2>
         <table>
             <thead>
@@ -347,26 +352,28 @@ def generate_html_report(log_dir: str = ".") -> None:
             </thead>
             <tbody>
 """
-        )
+            )
             for result in compliance_results:
-                status = result.get('Status', '')
-                result_class = 'pass' if status == 'COMPLIANT' else 'fail'
+                status = result.get("Status", "")
+                result_class = "pass" if status == "COMPLIANT" else "fail"
                 f.write(
-                f"""
+                    f"""
                 <tr class="{result_class}">
                     <td>{result.get('Requirement ID', '')}</td>
                     <td>{status}</td>
                 </tr>
 """
-        )
-            f.write("""
+                )
+            f.write(
+                """
             </tbody>
         </table>
 """
-        )
+            )
 
         # JavaScript for filtering and collapsible elements
-        f.write("""
+        f.write(
+            """
         <script>
             // Filter functionality
             document.getElementById('showPassed').addEventListener('change', filterResults);

tests/validation/Engine/logging_utils.py

@@ -41,7 +41,7 @@ def log_test_result(
         test_name: Name of the test
         test_description: Description of the test
         error_message: Error message if test failed
-        
+
     Returns:
         Decorated function
     """
@@ -52,7 +52,7 @@ def wrapper(*args, **kwargs) -> Any:
             start_time = time.time()
             result = True
             error = ""
-            
+
             try:
                 func_result = func(*args, **kwargs)
                 return func_result
@@ -69,17 +69,15 @@ def wrapper(*args, **kwargs) -> Any:
                     component=component,
                     result=result,
                     duration=duration,
-                    error_message=error_message or error
+                    error_message=error_message or error,
                 )
-        
+
         return wrapper
 
     return decorator
 
 
-def log_ffmpeg_test(
-    test_name: str, test_description: str = "", error_message: str = ""
-) -> Callable:
+def log_ffmpeg_test(test_name: str, test_description: str = "", error_message: str = "") -> Callable:
     """Decorator to log FFmpeg test results"""
     return log_test_result(
         component=COMPONENT_FFMPEG,
@@ -89,9 +87,7 @@ def log_ffmpeg_test(
     )
 
 
-def log_media_proxy_test(
-    test_name: str, test_description: str = "", error_message: str = ""
-) -> Callable:
+def log_media_proxy_test(test_name: str, test_description: str = "", error_message: str = "") -> Callable:
     """Decorator to log Media Proxy test results"""
     return log_test_result(
         component=COMPONENT_MEDIA_PROXY,
@@ -101,9 +97,7 @@ def log_media_proxy_test(
     )
 
 
-def log_mesh_agent_test(
-    test_name: str, test_description: str = "", error_message: str = ""
-) -> Callable:
+def log_mesh_agent_test(test_name: str, test_description: str = "", error_message: str = "") -> Callable:
     """Decorator to log Mesh Agent test results"""
     return log_test_result(
         component=COMPONENT_MESH_AGENT,
@@ -113,9 +107,7 @@ def log_mesh_agent_test(
     )
 
 
-def log_rxtx_app_test(
-    test_name: str, test_description: str = "", error_message: str = ""
-) -> Callable:
+def log_rxtx_app_test(test_name: str, test_description: str = "", error_message: str = "") -> Callable:
     """Decorator to log RxTx App test results"""
     return log_test_result(
         component=COMPONENT_RXTX_APP,
@@ -135,17 +127,15 @@ def parse_output_for_errors(
         component: Component name (FFmpeg, Media Proxy, Mesh Agent, RxTx App)
         output: Output text to parse
         custom_error_keywords: Custom error keywords to use instead of component defaults
-        
+
     Returns:
         Dictionary with is_pass and errors fields
     """
     if component not in COMPONENT_ERROR_KEYWORDS and not custom_error_keywords:
         logger.warning(f"Unknown component: {component}, using generic error keywords")
         error_keywords = ["error", "failed", "exception", "fatal"]
     else:
-        error_keywords = custom_error_keywords or COMPONENT_ERROR_KEYWORDS.get(
-            component, []
-        )
+        error_keywords = custom_error_keywords or COMPONENT_ERROR_KEYWORDS.get(component, [])
 
     errors = []
     for line in output.splitlines():
@@ -174,7 +164,7 @@ def streamline_component_output_validation(
         test_name: Name of the test
         duration: Test duration in seconds
         custom_error_keywords: Custom error keywords to use
-        
+
     Returns:
         Dictionary with is_pass and errors fields
     """