check-msdefender 1.1.1__tar.gz → 1.1.3__tar.gz

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/PKG-INFO

@@ -1,14 +1,10 @@
-Metadata-Version: 2.4
+Metadata-Version: 2.1
 Name: check-msdefender
-Version: 1.1.1
+Version: 1.1.3
 Summary: A Nagios plugin for monitoring Microsoft Defender API endpoints
-Author-email: ldvchosal <ldvchosal@github.com>
-License: MIT
-Project-URL: Homepage, https://github.com/lduchosal/check_msdefender
-Project-URL: Bug Reports, https://github.com/lduchosal/check_msdefender/issues
-Project-URL: Source, https://github.com/lduchosal/check_msdefender
-Project-URL: Documentation, https://github.com/lduchosal/check_msdefender/blob/main/README.md
 Keywords: nagios,monitoring,microsoft,graph,api,azure
+Author-Email: ldvchosal <ldvchosal@github.com>
+License: MIT
 Classifier: Development Status :: 5 - Production/Stable
 Classifier: Intended Audience :: System Administrators
 Classifier: License :: OSI Approved :: MIT License
@@ -19,9 +15,11 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Topic :: System :: Monitoring
 Classifier: Topic :: System :: Systems Administration
+Project-URL: Homepage, https://github.com/lduchosal/check_msdefender
+Project-URL: Bug Reports, https://github.com/lduchosal/check_msdefender/issues
+Project-URL: Source, https://github.com/lduchosal/check_msdefender
+Project-URL: Documentation, https://github.com/lduchosal/check_msdefender/blob/main/README.md
 Requires-Python: >=3.9
-Description-Content-Type: text/markdown
-License-File: LICENSE
 Requires-Dist: nagiosplugin>=1.4.0
 Requires-Dist: azure-identity>=1.12.0
 Requires-Dist: click<9.0,>=8.0
@@ -33,7 +31,9 @@ Requires-Dist: black>=21.0; extra == "dev"
 Requires-Dist: flake8>=3.8; extra == "dev"
 Requires-Dist: mypy>=0.800; extra == "dev"
 Requires-Dist: twine>=6.2.0; extra == "dev"
-Dynamic: license-file
+Requires-Dist: pdm>=2.0.0; extra == "dev"
+Requires-Dist: ruff>=0.13.0; extra == "dev"
+Description-Content-Type: text/markdown
 
 # 🛡️ Check MS Defender
 
@@ -295,7 +295,6 @@ source .venv/bin/activate  # Windows: .venv\Scripts\activate
 
 # Install in development mode
 pip install -e .
-pip install -r requirements-dev.txt
 ```
 
 ### Code Quality Tools
@@ -428,4 +427,4 @@ This project is licensed under the **MIT License** - see the [LICENSE](LICENSE)
 
 [🐛 Report Bug](https://github.com/lduchosal/check_msdefender/issues) • [💡 Request Feature](https://github.com/lduchosal/check_msdefender/issues) • [📖 Documentation](https://github.com/lduchosal/check_msdefender/blob/main/README.md)
 
-</div>
+</div>

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/README.md

@@ -258,7 +258,6 @@ source .venv/bin/activate  # Windows: .venv\Scripts\activate
 
 # Install in development mode
 pip install -e .
-pip install -r requirements-dev.txt
 ```
 
 ### Code Quality Tools

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/check_msdefender.py

@@ -1,6 +1,5 @@
 """Main entry point for check_msdefender Nagios plugin."""
 
-import sys
 from check_msdefender.cli import main
 
 if __name__ == "__main__":

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/cli/commands/alerts.py

@@ -1,7 +1,6 @@
 """Alerts commands for CLI."""
 
 import sys
-import click
 from typing import Optional, Any
 
 from check_msdefender.core.auth import get_authenticator

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/cli/commands/detail.py

@@ -16,7 +16,6 @@ def register_detail_commands(main_group: Any) -> None:
     """Register detail commands with the main CLI group."""
 
     @main_group.command("detail")
-    @click.option("-i", "--id", "machine_id_alt", help="Machine ID (GUID)")
     @common_options
     def detail_cmd(
         config: str,
@@ -25,7 +24,6 @@ def register_detail_commands(main_group: Any) -> None:
         dns_name: Optional[str],
         warning: Optional[float],
         critical: Optional[float],
-        machine_id_alt: Optional[str],
     ) -> None:
         """Get detailed machine information from Microsoft Defender."""
         try:
@@ -44,9 +42,6 @@ def register_detail_commands(main_group: Any) -> None:
             # Create custom Nagios plugin for detail output
             plugin = NagiosPlugin(service, "detail")
 
-            # Use -i option if provided, otherwise fallback to -m
-            final_machine_id = machine_id_alt or machine_id
-
             # Set default thresholds for detail command to show proper performance data
             # Based on expected test output patterns
             if warning is not None and critical is None:
@@ -58,7 +53,7 @@ def register_detail_commands(main_group: Any) -> None:
 
             # Execute check
             result = plugin.check(
-                machine_id=final_machine_id,
+                machine_id=machine_id,
                 dns_name=dns_name,
                 warning=warning,
                 critical=critical,

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/cli/commands/lastseen.py

@@ -1,7 +1,6 @@
 """Last seen commands for CLI."""
 
 import sys
-import click
 from typing import Optional, Any
 
 from check_msdefender.core.auth import get_authenticator

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/cli/commands/machines.py

@@ -1,7 +1,6 @@
 """List machines commands for CLI."""
 
 import sys
-import click
 from typing import Optional, Any
 
 from check_msdefender.core.auth import get_authenticator

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/cli/commands/onboarding.py

@@ -1,7 +1,6 @@
 """Onboarding status commands for CLI."""
 
 import sys
-import click
 from typing import Optional, Any
 
 from check_msdefender.core.auth import get_authenticator

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/cli/commands/vulnerabilities.py

@@ -1,7 +1,6 @@
 """Vulnerability commands for CLI."""
 
 import sys
-import click
 from typing import Optional, Any
 
 from check_msdefender.core.auth import get_authenticator

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/cli/decorators.py

@@ -10,7 +10,9 @@ def common_options(func: Callable[..., Any]) -> Callable[..., Any]:
         "-c", "--config", default="check_msdefender.ini", help="Configuration file path"
     )(func)
     func = click.option("-v", "--verbose", count=True, help="Increase verbosity")(func)
-    func = click.option("-m", "--machine-id", help="Machine ID (GUID)")(func)
+    func = click.option("-m", "--machine-id", "-i", "--id", help="Machine ID (GUID)")(
+        func
+    )
     func = click.option("-d", "--dns-name", help="Computer DNS Name (FQDN)")(func)
     func = click.option("-W", "--warning", type=float, help="Warning threshold")(func)
     func = click.option("-C", "--critical", type=float, help="Critical threshold")(func)

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/cli/handlers.py

@@ -1,7 +1,6 @@
 """Error handlers and formatters for click CLI."""
 
 import click
-from typing import Any
 
 
 class ClickErrorHandler:

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/core/auth.py

@@ -20,7 +20,9 @@ def get_authenticator(
     tenant_id = auth_section.get("tenant_id")
 
     if not client_id or not tenant_id:
-        raise ConfigurationError("client_id and tenant_id are required in [auth] section")
+        raise ConfigurationError(
+            "client_id and tenant_id are required in [auth] section"
+        )
 
     # Check for client secret authentication
     client_secret = auth_section.get("client_secret")

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/core/defender.py

@@ -13,7 +13,11 @@ class DefenderClient:
     application_json = "application/json"
 
     def __init__(
-        self, authenticator: Any, timeout: int = 5, region: str = "eu3", verbose_level: int = 0
+        self,
+        authenticator: Any,
+        timeout: int = 5,
+        region: str = "eu3",
+        verbose_level: int = 0,
     ) -> None:
         """Initialize with authenticator and optional region.
 
@@ -56,7 +60,9 @@ class DefenderClient:
         try:
             start_time = time.time()
             self.logger.info(f"Querying machine by DNS name: {dns_name}")
-            response = requests.get(url, headers=headers, params=params, timeout=self.timeout)
+            response = requests.get(
+                url, headers=headers, params=params, timeout=self.timeout
+            )
             elapsed = time.time() - start_time
 
             self.logger.api_call("GET", url, response.status_code, elapsed)
@@ -151,7 +157,9 @@ class DefenderClient:
         try:
             start_time = time.time()
             self.logger.info("Querying all machines")
-            response = requests.get(url, headers=headers, params=params, timeout=self.timeout)
+            response = requests.get(
+                url, headers=headers, params=params, timeout=self.timeout
+            )
             elapsed = time.time() - start_time
 
             self.logger.api_call("GET", url, response.status_code, elapsed)
@@ -189,7 +197,9 @@ class DefenderClient:
         try:
             start_time = time.time()
             self.logger.info("Querying alerts")
-            response = requests.get(url, headers=headers, params=params, timeout=self.timeout)
+            response = requests.get(
+                url, headers=headers, params=params, timeout=self.timeout
+            )
             elapsed = time.time() - start_time
 
             self.logger.api_call("GET", url, response.status_code, elapsed)

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/core/logging_config.py

@@ -36,7 +36,8 @@ class VerboseLogger:
         if self.verbose_level >= 3:
             # Full trace format
             formatter = logging.Formatter(
-                "[%(levelname)s] %(asctime)s %(name)s:%(lineno)d - %(message)s", datefmt="%H:%M:%S"
+                "[%(levelname)s] %(asctime)s %(name)s:%(lineno)d - %(message)s",
+                datefmt="%H:%M:%S",
             )
         elif self.verbose_level >= 2:
             # Debug format
@@ -81,7 +82,9 @@ class VerboseLogger:
         """Log API call details if verbose >= 2."""
         if self.verbose_level >= 2:
             if status_code and response_time:
-                self.logger.debug(f"API {method} {url} -> {status_code} ({response_time:.3f}s)")
+                self.logger.debug(
+                    f"API {method} {url} -> {status_code} ({response_time:.3f}s)"
+                )
             else:
                 self.logger.debug(f"API {method} {url}")
 

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/core/nagios.py

@@ -128,7 +128,9 @@ class NagiosPlugin:
 
             # Create Nagios check with custom summary
             # Use 'found' as context name for detail command, otherwise use command name
-            context_name = "found" if self.command_name == "detail" else self.command_name
+            context_name = (
+                "found" if self.command_name == "detail" else self.command_name
+            )
             check = nagiosplugin.Check(
                 DefenderResource(self.command_name, value),
                 DefenderScalarContext(context_name, warning, critical),

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/services/alerts_service.py

@@ -1,7 +1,7 @@
 """Alerts service implementation."""
 
-from datetime import datetime
-from typing import Dict, Optional, Any, List
+from typing import Dict, Optional, Any
+
 from check_msdefender.core.exceptions import ValidationError
 from check_msdefender.core.logging_config import get_verbose_logger
 
@@ -53,12 +53,18 @@ class AlertsService:
             or alert.get("computerDnsName") == target_dns_name
         ]
 
-        self.logger.info(f"Found {len(machine_alerts)} alerts for machine {target_dns_name}")
+        self.logger.info(
+            f"Found {len(machine_alerts)} alerts for machine {target_dns_name}"
+        )
 
         # Categorize alerts by status and severity
-        unresolved_alerts = [alert for alert in machine_alerts if alert.get("status") != "Resolved"]
+        unresolved_alerts = [
+            alert for alert in machine_alerts if alert.get("status") != "Resolved"
+        ]
         informational_alerts = [
-            alert for alert in unresolved_alerts if alert.get("severity") == "Informational"
+            alert
+            for alert in unresolved_alerts
+            if alert.get("severity") == "Informational"
         ]
         critical_warning_alerts = [
             alert
@@ -82,7 +88,9 @@ class AlertsService:
                 title = alert.get("title", "Unknown alert")
                 status = alert.get("status", "Unknown")
                 severity = alert.get("severity", "Unknown")
-                details.append(f"{creation_time} - {title} ({status} {severity.lower()})")
+                details.append(
+                    f"{creation_time} - {title} ({status} {severity.lower()})"
+                )
 
         # Return the number of unresolved alerts as the value
         # This will be used by Nagios plugin for determining status based on thresholds
@@ -93,6 +101,8 @@ class AlertsService:
             "details": details,
         }
 
-        self.logger.info(f"Alert analysis complete: {len(unresolved_alerts)} unresolved alerts")
+        self.logger.info(
+            f"Alert analysis complete: {len(unresolved_alerts)} unresolved alerts"
+        )
         self.logger.method_exit("get_result", result)
         return result

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/services/detail_service.py

@@ -54,10 +54,16 @@ class DetailService:
             # Create detailed output
             details = []
             details.append(f"Machine ID: {machine_details.get('id', 'Unknown')}")
-            details.append(f"Computer Name: {machine_details.get('computerDnsName', 'Unknown')}")
-            details.append(f"OS Platform: {machine_details.get('osPlatform', 'Unknown')}")
+            details.append(
+                f"Computer Name: {machine_details.get('computerDnsName', 'Unknown')}"
+            )
+            details.append(
+                f"OS Platform: {machine_details.get('osPlatform', 'Unknown')}"
+            )
             details.append(f"OS Version: {machine_details.get('osVersion', 'Unknown')}")
-            details.append(f"Health Status: {machine_details.get('healthStatus', 'Unknown')}")
+            details.append(
+                f"Health Status: {machine_details.get('healthStatus', 'Unknown')}"
+            )
             details.append(f"Risk Score: {machine_details.get('riskScore', 'Unknown')}")
 
             result = {"value": 1, "details": details}

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/services/lastseen_service.py

@@ -62,7 +62,9 @@ class LastSeenService:
 
             result = {"value": days_diff, "details": details}
 
-            self.logger.info(f"Machine last seen {days_diff} days ago ({last_seen_str})")
+            self.logger.info(
+                f"Machine last seen {days_diff} days ago ({last_seen_str})"
+            )
             self.logger.method_exit("get_result", result)
             return result
         except (ValueError, TypeError) as e:

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/services/machines_service.py

@@ -1,7 +1,7 @@
 """Machines service implementation."""
 
 from typing import Dict, List, Any
-from check_msdefender.core.exceptions import ValidationError
+
 from check_msdefender.core.logging_config import get_verbose_logger
 
 
@@ -23,7 +23,10 @@ class MachinesService:
 
         if not machines_data.get("value"):
             self.logger.info("No machines found")
-            result = {"value": 0, "details": ["No machines found in Microsoft Defender"]}
+            result = {
+                "value": 0,
+                "details": ["No machines found in Microsoft Defender"],
+            }
             self.logger.method_exit("get_result", result)
             return result
 
@@ -39,7 +42,11 @@ class MachinesService:
 
         # Sort by priority
         sorted_machines = sorted(
-            machines, key=lambda x: (status_priority[x["onboardingStatus"]], x["computerDnsName"])
+            machines,
+            key=lambda x: (
+                status_priority[x["onboardingStatus"]],
+                x["computerDnsName"],
+            ),
         )
         for machine in sorted_machines:
             onboarded = "✓" if machine["onboardingStatus"] == "Onboarded" else "✗"

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/services/models.py

@@ -2,8 +2,8 @@
 
 from dataclasses import dataclass
 from datetime import datetime
-from typing import Optional, List
 from enum import Enum
+from typing import Optional
 
 
 class OnboardingStatus(Enum):

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/services/onboarding_service.py

@@ -54,6 +54,8 @@ class OnboardingService:
 
         result = {"value": result_value, "details": details}
 
-        self.logger.info(f"Machine onboarding status: {onboarding_state} -> {result_value}")
+        self.logger.info(
+            f"Machine onboarding status: {onboarding_state} -> {result_value}"
+        )
         self.logger.method_exit("get_result", result)
         return result

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/check_msdefender/services/vulnerabilities_service.py

@@ -41,7 +41,9 @@ class VulnerabilitiesService:
 
         # Process and deduplicate vulnerabilities
         vulnerabilities = self._process_vulnerabilities(raw_vulnerabilities)
-        self.logger.info(f"Found {len(vulnerabilities)} unique vulnerabilities after deduplication")
+        self.logger.info(
+            f"Found {len(vulnerabilities)} unique vulnerabilities after deduplication"
+        )
 
         # Calculate vulnerability score
         score = VulnerabilityScore()
@@ -54,7 +56,9 @@ class VulnerabilitiesService:
 
         for vuln in sorted_vulnerabilities:
             severity = vuln.severity.lower()
-            self.logger.debug(f"Processing vulnerability {vuln.id} with severity: {severity}")
+            self.logger.debug(
+                f"Processing vulnerability {vuln.id} with severity: {severity}"
+            )
 
             if severity == "critical":
                 score.critical += 1
@@ -75,7 +79,8 @@ class VulnerabilitiesService:
         self.logger.info(f"Total vulnerability score: {score.total_score}")
 
         details.insert(
-            0, f"Vulnerabilities: {len(raw_vulnerabilities)}, score: {score.total_score}"
+            0,
+            f"Vulnerabilities: {len(raw_vulnerabilities)}, score: {score.total_score}",
         )
 
         result = {"value": score.total_score, "details": details}
@@ -125,7 +130,9 @@ class VulnerabilitiesService:
         # Sort by severity
         sorted_vulnerabilities = self._sort_by_severity(vulnerabilities)
 
-        self.logger.method_exit("get_detailed_vulnerabilities", len(sorted_vulnerabilities))
+        self.logger.method_exit(
+            "get_detailed_vulnerabilities", len(sorted_vulnerabilities)
+        )
         return sorted_vulnerabilities
 
     def _process_vulnerabilities(
@@ -156,8 +163,11 @@ class VulnerabilitiesService:
 
         return unique_vulnerabilities
 
-    def _sort_by_severity(self, vulnerabilities: List[Vulnerability]) -> List[Vulnerability]:
+    def _sort_by_severity(
+        self, vulnerabilities: List[Vulnerability]
+    ) -> List[Vulnerability]:
         """Sort vulnerabilities by severity (Critical > High > Medium > Low)."""
         return sorted(
-            vulnerabilities, key=lambda v: self._severity_order.get(v.severity.lower(), 999)
+            vulnerabilities,
+            key=lambda v: self._severity_order.get(v.severity.lower(), 999),
         )

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/pyproject.toml

@@ -1,16 +1,17 @@
 [build-system]
-requires = ["setuptools>=45", "wheel", "setuptools_scm[toml]>=6.2"]
-build-backend = "setuptools.build_meta"
+requires = [
+    "pdm-backend",
+]
+build-backend = "pdm.backend"
 
 [project]
 name = "check-msdefender"
-version = "1.1.1"
+version = "1.1.3"
 authors = [
-    {name = "ldvchosal", email = "ldvchosal@github.com"},
+    { name = "ldvchosal", email = "ldvchosal@github.com" },
 ]
 description = "A Nagios plugin for monitoring Microsoft Defender API endpoints"
 readme = "README.md"
-license = {text = "MIT"}
 requires-python = ">=3.9"
 classifiers = [
     "Development Status :: 5 - Production/Stable",
@@ -24,13 +25,23 @@ classifiers = [
     "Topic :: System :: Monitoring",
     "Topic :: System :: Systems Administration",
 ]
-keywords = ["nagios", "monitoring", "microsoft", "graph", "api", "azure"]
+keywords = [
+    "nagios",
+    "monitoring",
+    "microsoft",
+    "graph",
+    "api",
+    "azure",
+]
 dependencies = [
     "nagiosplugin>=1.4.0",
     "azure-identity>=1.12.0",
-    "click>=8.0,<9.0"
+    "click>=8.0,<9.0",
 ]
 
+[project.license]
+text = "MIT"
+
 [project.optional-dependencies]
 dev = [
     "pytest>=6.0",
@@ -40,6 +51,8 @@ dev = [
     "flake8>=3.8",
     "mypy>=0.800",
     "twine>=6.2.0",
+    "pdm>=2.0.0",
+    "ruff>=0.13.0",
 ]
 
 [project.urls]
@@ -52,11 +65,15 @@ Documentation = "https://github.com/lduchosal/check_msdefender/blob/main/README.
 check_msdefender = "check_msdefender.cli:main"
 
 [tool.setuptools.package-data]
-"*" = ["*.ini"]
+"*" = [
+    "*.ini",
+]
 
 [tool.black]
 line-length = 100
-target-version = ['py39']
+target-version = [
+    "py39",
+]
 
 [tool.mypy]
 python_version = "3.9"
@@ -65,8 +82,25 @@ warn_unused_configs = true
 disallow_untyped_defs = true
 
 [tool.pytest.ini_options]
-testpaths = ["tests"]
+testpaths = [
+    "tests",
+]
 python_files = "test_*.py"
 python_classes = "Test*"
 python_functions = "test_*"
-addopts = "-v"
+addopts = "-v"
+
+[tool.pdm.scripts]
+format = "ruff format"
+typecheck = "mypy check_msdefender/"
+lint = "flake8 check_msdefender/"
+build = "python -m build"
+publish = "python -m twine upload dist/* --verbose"
+test = "pytest -v tests/"
+
+[tool.pdm.scripts.all]
+composite = [
+    "format",
+    "build",
+    "test",
+]

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/tests/fixtures/mock_defender_client.py

@@ -1,8 +1,8 @@
 """Mock Defender client for fixture tests."""
 
 import json
-import os
 from pathlib import Path
+
 from check_msdefender.core.exceptions import ValidationError
 
 
@@ -38,7 +38,9 @@ class MockDefenderClient:
 
     def get_machine_vulnerabilities(self, machine_id):
         """Get vulnerabilities for machine from fixtures."""
-        return self.vulnerability_data["vulnerabilities_by_machine"].get(machine_id, {"value": []})
+        return self.vulnerability_data["vulnerabilities_by_machine"].get(
+            machine_id, {"value": []}
+        )
 
     def get_alerts(self):
         """Get all alerts from fixtures."""

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/tests/fixtures/test_alerts_service.py

@@ -67,7 +67,9 @@ class TestAlertsServiceFixtures:
 
     def test_get_result_no_parameters(self):
         """Test error when no parameters provided."""
-        with pytest.raises(ValidationError, match="Either machine_id or dns_name must be provided"):
+        with pytest.raises(
+            ValidationError, match="Either machine_id or dns_name must be provided"
+        ):
             self.service.get_result()
 
     def test_get_result_nonexistent_dns_name(self):
@@ -135,7 +137,8 @@ class TestAlertsServiceFixtures:
         # Should include alert titles and severity/status
         assert "suspicious activity detected (new high)" in details_text.lower()
         assert (
-            "automated investigation started manually (new informational)" in details_text.lower()
+            "automated investigation started manually (new informational)"
+            in details_text.lower()
         )
 
     def test_dns_name_matching(self):

{check_msdefender-1.1.1 → check_msdefender-1.1.3}/tests/fixtures/test_detail_service.py

@@ -56,7 +56,9 @@ class TestDetailServiceFixtures:
 
     def test_get_result_no_parameters(self):
         """Test error when no parameters provided."""
-        with pytest.raises(ValidationError, match="Either machine_id or dns_name must be provided"):
+        with pytest.raises(
+            ValidationError, match="Either machine_id or dns_name must be provided"
+        ):
             self.service.get_result()
 
     def test_get_result_nonexistent_dns_name(self):
@@ -70,7 +72,9 @@ class TestDetailServiceFixtures:
 
     def test_get_result_nonexistent_machine_id(self):
         """Test error when machine ID doesn't exist."""
-        with pytest.raises(ValidationError, match="Machine not found: nonexistent-machine"):
+        with pytest.raises(
+            ValidationError, match="Machine not found: nonexistent-machine"
+        ):
             self.service.get_result(machine_id="nonexistent-machine")
 
     def test_machine_details_comprehensive(self):
@@ -96,7 +100,9 @@ class TestDetailServiceFixtures:
         ]
 
         for field in expected_fields:
-            assert field in details_dict, f"Field '{field}' missing from machine details"
+            assert field in details_dict, (
+                f"Field '{field}' missing from machine details"
+            )
 
         # Verify specific values for test-machine-3
         assert details_dict["id"] == "test-machine-3"