[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

tests/workbenches/notebook-controller/test_custom_images.py

@@ -60,7 +60,7 @@ class TestCustomImageValidation:
                     ),  # Placeholder - update with sdg_hub image
                 },
                 id="sdg_hub_image",
-                #marks=pytest.mark.skip(reason="Waiting for sdg_hub image URL from workbench image team"),
+                # marks=pytest.mark.skip(reason="Waiting for sdg_hub image URL from workbench image team"),
             ),
             # Test Case: Data Science Notebook (Demonstration of Pattern Reusability)
             # Image: Standard datascience workbench image
@@ -159,7 +159,7 @@ def test_custom_image_package_verification(
         else:
             # Default: basic Python packages
             packages_to_verify = ["sys", "os"]
-        
+
         # Install packages if they're not standard library (not in the default list)
         standard_lib_packages = {"sys", "os"}
         packages_to_install = [pkg for pkg in packages_to_verify if pkg not in standard_lib_packages]
@@ -250,7 +250,7 @@ def _get_pod_failure_details(self, pod: Pod) -> str:
                             f"    Message: {message}"
                         )
                     else:
-                        details.append(f"    Waiting Reason: {reason}\n" f"    Message: {message}")
+                        details.append(f"    Waiting Reason: {reason}\n    Message: {message}")
 
                 # Check terminated state
                 if hasattr(container_status.state, "terminated") and container_status.state.terminated:
@@ -271,9 +271,7 @@ def _get_pod_failure_details(self, pod: Pod) -> str:
 
         return "\n".join(details)
 
-    def _format_package_failure_report(
-        self, failed_packages: list[str], results: dict, pod: Pod
-    ) -> str:
+    def _format_package_failure_report(self, failed_packages: list[str], results: dict, pod: Pod) -> str:
         """
         Format a detailed error report for package import failures.
 
@@ -298,7 +296,7 @@ def _format_package_failure_report(
             report.append(f"     Execution Time: {result.execution_time_seconds:.2f}s")
 
             if result.pod_logs:
-                report.append(f"     Pod Logs (excerpt):")
+                report.append("     Pod Logs (excerpt):")
                 # Show first 500 characters of logs
                 log_excerpt = result.pod_logs[:500]
                 for line in log_excerpt.split("\n"):
@@ -312,4 +310,4 @@ def _format_package_failure_report(
         report.append("  3. Verify package names match import names (pip name vs import name)")
         report.append("  4. Contact the workbench image team for package installation issues")
 
-        return "\n".join(report)
+        return "\n".join(report)

tests/workbenches/utils.py

@@ -12,6 +12,7 @@
 
 LOGGER = get_logger(name=__name__)
 
+
 @dataclass
 class PackageVerificationResult:
     """Represents the outcome of package import verification for a single package."""
@@ -89,9 +90,7 @@ def verify_package_import(
     # Verify container exists
     container_names = [c.name for c in pod.instance.spec.containers]
     if container_name not in container_names:
-        raise RuntimeError(
-            f"Container '{container_name}' not found in pod. Available containers: {container_names}"
-        )
+        raise RuntimeError(f"Container '{container_name}' not found in pod. Available containers: {container_names}")
 
     LOGGER.info(f"Verifying {len(packages)} packages in container '{container_name}' of pod '{pod.name}'")
 
@@ -208,9 +207,7 @@ def install_packages_in_pod(
     # Verify container exists
     container_names = [c.name for c in pod.instance.spec.containers]
     if container_name not in container_names:
-        raise RuntimeError(
-            f"Container '{container_name}' not found in pod. Available containers: {container_names}"
-        )
+        raise RuntimeError(f"Container '{container_name}' not found in pod. Available containers: {container_names}")
 
     LOGGER.info(f"Installing {len(packages)} packages in container '{container_name}' of pod '{pod.name}'")
 
@@ -234,6 +231,7 @@ def install_packages_in_pod(
 
     return results
 
+
 def get_username(dyn_client: DynamicClient) -> str | None:
     """Gets the username for the client (see kubectl -v8 auth whoami)"""
     username: str | None