chore: ruff lint + format pass

fgogolli · fgogolli · commit 7b5b9aa3dcfb · 2026-06-15T21:08:31.000+01:00
diff --git a/src/orb/bootstrap/provider_services.py b/src/orb/bootstrap/provider_services.py
@@ -73,9 +73,7 @@ def _register_provider_utility_services(container: DIContainer) -> None:
     for name in _REGISTERED_PROVIDERS:
         mod_path = f"orb.providers.{name}.registration"
         if importlib.util.find_spec(mod_path) is None:
-            logger.debug(
-                "%s provider not available, skipping utility service registration", name
-            )
+            logger.debug("%s provider not available, skipping utility service registration", name)
             continue
         try:
             mod = importlib.import_module(mod_path)
diff --git a/src/orb/domain/base/events/provider_events.py b/src/orb/domain/base/events/provider_events.py
@@ -85,9 +85,7 @@ class ProviderResourceStateChangedEvent(DomainEvent):
     def model_post_init(self, __context: Any) -> None:
         """Initialize aggregate information after model creation."""
         object.__setattr__(self, "aggregate_id", self.resource_id)
-        object.__setattr__(
-            self, "aggregate_type", f"{self.provider_type}_{self.resource_type}"
-        )
+        object.__setattr__(self, "aggregate_type", f"{self.provider_type}_{self.resource_type}")
         super().model_post_init(__context)
         if not self.resource_type:
             raise ValueError("Resource type cannot be empty")
diff --git a/src/orb/providers/registration.py b/src/orb/providers/registration.py
@@ -69,6 +69,7 @@ def register_all_providers(container: "DIContainer | None" = None) -> None:
 # Deprecated aliases – kept for backward compatibility with existing callers.
 # ---------------------------------------------------------------------------
 
+
 def register_all_provider_types() -> None:
     """Register all available provider types.
 
diff --git a/tests/providers/aws/live/conftest.py b/tests/providers/aws/live/conftest.py
@@ -17,6 +17,7 @@
 from boto3 import Session
 from botocore.exceptions import ClientError, NoCredentialsError
 
+
 def _find_repo_root(start: Path) -> Path:
     """Walk up from *start* until we find pyproject.toml. Hard-fail otherwise.
 
diff --git a/tests/providers/aws/live/test_cleanup_e2e_onaws.py b/tests/providers/aws/live/test_cleanup_e2e_onaws.py
@@ -390,9 +390,15 @@ async def _cleanup() -> None:
                     "Fixture teardown: cleanup_launch_templates failed for %s: %s", req_id, exc
                 )
 
-    for key in ("ORB_CONFIG_DIR", "HF_PROVIDER_CONFDIR", "HF_PROVIDER_LOGDIR",
-                "HF_PROVIDER_WORKDIR", "DEFAULT_PROVIDER_WORKDIR", "AWS_PROVIDER_LOG_DIR",
-                "HF_LOGDIR"):
+    for key in (
+        "ORB_CONFIG_DIR",
+        "HF_PROVIDER_CONFDIR",
+        "HF_PROVIDER_LOGDIR",
+        "HF_PROVIDER_WORKDIR",
+        "DEFAULT_PROVIDER_WORKDIR",
+        "AWS_PROVIDER_LOG_DIR",
+        "HF_LOGDIR",
+    ):
         os.environ.pop(key, None)
 
     try:
diff --git a/tests/providers/aws/live/test_mcp_onaws.py b/tests/providers/aws/live/test_mcp_onaws.py
@@ -184,9 +184,15 @@ async def _cleanup() -> None:
                 )
 
     # Teardown: clean up env vars and reset DI container so next test gets a fresh one
-    for key in ("ORB_CONFIG_DIR", "HF_PROVIDER_CONFDIR", "HF_PROVIDER_LOGDIR",
-                "HF_PROVIDER_WORKDIR", "DEFAULT_PROVIDER_WORKDIR", "AWS_PROVIDER_LOG_DIR",
-                "HF_LOGDIR"):
+    for key in (
+        "ORB_CONFIG_DIR",
+        "HF_PROVIDER_CONFDIR",
+        "HF_PROVIDER_LOGDIR",
+        "HF_PROVIDER_WORKDIR",
+        "DEFAULT_PROVIDER_WORKDIR",
+        "AWS_PROVIDER_LOG_DIR",
+        "HF_LOGDIR",
+    ):
         os.environ.pop(key, None)
 
     try:
diff --git a/tests/providers/aws/live/test_multi_asg_termination.py b/tests/providers/aws/live/test_multi_asg_termination.py
@@ -192,8 +192,14 @@ def _tracking_request_machines(template_name: str, machine_count: int):
     except Exception as exc:
         log.warning("Fixture teardown: cleanup_tracked_requests failed: %s", exc)
 
-    for key in ("ORB_CONFIG_DIR", "HF_PROVIDER_CONFDIR", "HF_PROVIDER_LOGDIR",
-                "HF_PROVIDER_WORKDIR", "AWS_PROVIDER_LOG_DIR", "HF_LOGDIR"):
+    for key in (
+        "ORB_CONFIG_DIR",
+        "HF_PROVIDER_CONFDIR",
+        "HF_PROVIDER_LOGDIR",
+        "HF_PROVIDER_WORKDIR",
+        "AWS_PROVIDER_LOG_DIR",
+        "HF_LOGDIR",
+    ):
         os.environ.pop(key, None)
     processor.cleanup_test_templates(test_name)
     log.removeHandler(file_handler)
diff --git a/tests/providers/aws/live/test_multi_ec2_fleet_termination.py b/tests/providers/aws/live/test_multi_ec2_fleet_termination.py
@@ -209,8 +209,14 @@ def _tracking_request_machines(template_name: str, machine_count: int):
     except Exception as exc:
         log.warning("Fixture teardown: cleanup_tracked_requests failed: %s", exc)
 
-    for key in ("ORB_CONFIG_DIR", "HF_PROVIDER_CONFDIR", "HF_PROVIDER_LOGDIR",
-                "HF_PROVIDER_WORKDIR", "AWS_PROVIDER_LOG_DIR", "HF_LOGDIR"):
+    for key in (
+        "ORB_CONFIG_DIR",
+        "HF_PROVIDER_CONFDIR",
+        "HF_PROVIDER_LOGDIR",
+        "HF_PROVIDER_WORKDIR",
+        "AWS_PROVIDER_LOG_DIR",
+        "HF_LOGDIR",
+    ):
         os.environ.pop(key, None)
     processor.cleanup_test_templates(test_name)
     log.removeHandler(file_handler)
diff --git a/tests/providers/aws/live/test_multi_resource_termination.py b/tests/providers/aws/live/test_multi_resource_termination.py
@@ -223,8 +223,14 @@ def _tracking_request_machines(template_name: str, machine_count: int):
     except Exception as exc:
         log.warning("Fixture teardown: cleanup_tracked_requests failed: %s", exc)
 
-    for key in ("ORB_CONFIG_DIR", "HF_PROVIDER_CONFDIR", "HF_PROVIDER_LOGDIR",
-                "HF_PROVIDER_WORKDIR", "AWS_PROVIDER_LOG_DIR", "HF_LOGDIR"):
+    for key in (
+        "ORB_CONFIG_DIR",
+        "HF_PROVIDER_CONFDIR",
+        "HF_PROVIDER_LOGDIR",
+        "HF_PROVIDER_WORKDIR",
+        "AWS_PROVIDER_LOG_DIR",
+        "HF_LOGDIR",
+    ):
         os.environ.pop(key, None)
     processor.cleanup_test_templates(test_name)
     log.removeHandler(file_handler)
diff --git a/tests/providers/aws/live/test_multi_spot_fleet_termination.py b/tests/providers/aws/live/test_multi_spot_fleet_termination.py
@@ -211,8 +211,14 @@ def _tracking_request_machines(template_name: str, machine_count: int):
     except Exception as exc:
         log.warning("Fixture teardown: cleanup_tracked_requests failed: %s", exc)
 
-    for key in ("ORB_CONFIG_DIR", "HF_PROVIDER_CONFDIR", "HF_PROVIDER_LOGDIR",
-                "HF_PROVIDER_WORKDIR", "AWS_PROVIDER_LOG_DIR", "HF_LOGDIR"):
+    for key in (
+        "ORB_CONFIG_DIR",
+        "HF_PROVIDER_CONFDIR",
+        "HF_PROVIDER_LOGDIR",
+        "HF_PROVIDER_WORKDIR",
+        "AWS_PROVIDER_LOG_DIR",
+        "HF_LOGDIR",
+    ):
         os.environ.pop(key, None)
     processor.cleanup_test_templates(test_name)
     log.removeHandler(file_handler)
diff --git a/tests/providers/aws/live/test_rest_api_onaws.py b/tests/providers/aws/live/test_rest_api_onaws.py
@@ -345,12 +345,19 @@ def setup_rest_api_environment(request, test_session_id):
 
     yield test_case
 
-    for key in ("ORB_CONFIG_DIR", "HF_PROVIDER_CONFDIR", "HF_PROVIDER_LOGDIR",
-                "HF_PROVIDER_WORKDIR", "DEFAULT_PROVIDER_WORKDIR", "AWS_PROVIDER_LOG_DIR",
-                "METRICS_DIR"):
+    for key in (
+        "ORB_CONFIG_DIR",
+        "HF_PROVIDER_CONFDIR",
+        "HF_PROVIDER_LOGDIR",
+        "HF_PROVIDER_WORKDIR",
+        "DEFAULT_PROVIDER_WORKDIR",
+        "AWS_PROVIDER_LOG_DIR",
+        "METRICS_DIR",
+    ):
         os.environ.pop(key, None)
     try:
         from orb.infrastructure.di import reset_container
+
         reset_container()
     except Exception:
         pass
