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

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

Signed-off-by: PritamP20 <pripritam7@gmail.com>

Author: pre-commit-ci[bot]

ardupilot_methodic_configurator/data_model_signing_config.py

@@ -103,8 +103,12 @@ def from_dict(cls, data: dict[str, Any]) -> "SigningConfig":
         """
         # Filter to only known fields to avoid unexpected arguments
         known_fields = {
-            "enabled", "sign_outgoing", "allow_unsigned_in",
-            "accept_unsigned_callbacks", "timestamp_offset", "link_id",
+            "enabled",
+            "sign_outgoing",
+            "allow_unsigned_in",
+            "accept_unsigned_callbacks",
+            "timestamp_offset",
+            "link_id",
         }
         filtered_data = {k: v for k, v in data.items() if k in known_fields}
         return cls(**filtered_data)

tests/sitl_signing.py

@@ -32,15 +32,16 @@
 def is_sitl_available(connection: str = SITL_CONNECTION, timeout: float = 5.0) -> bool:
     """Check if SITL is running and accepting connections."""
     try:
-        from pymavlink import mavutil  
+        from pymavlink import mavutil
 
         master = mavutil.mavlink_connection(connection, baud=115200)
         heartbeat = master.wait_heartbeat(timeout=timeout)
         master.close()
         return heartbeat is not None
-    except Exception:  
+    except Exception:
         return False
 
+
 pytestmark = [
     pytest.mark.integration,
     pytest.mark.skipif(
@@ -131,9 +132,7 @@ def test_can_setup_signing_with_generated_key(
         else:
             assert "not supported" in error_msg.lower() or "error" in error_msg.lower()
 
-    def test_can_disable_signing_after_setup(
-        self, connected_fc: FlightController, signing_keystore: SigningKeystore
-    ) -> None:
+    def test_can_disable_signing_after_setup(self, connected_fc: FlightController, signing_keystore: SigningKeystore) -> None:
         """
         User can disable signing after it has been configured.
 
@@ -213,9 +212,7 @@ def test_signing_with_invalid_link_id_raises_error(
         with pytest.raises(ValueError, match="link_id"):
             connected_fc.setup_signing(key, link_id=256)
 
-    def test_can_reconnect_after_signing_enabled(
-        self, signing_keystore: SigningKeystore
-    ) -> None:
+    def test_can_reconnect_after_signing_enabled(self, signing_keystore: SigningKeystore) -> None:
         """
         FlightController can reconnect after signing was enabled.
 
@@ -232,7 +229,7 @@ def test_can_reconnect_after_signing_enabled(
         fc.setup_signing(key)
 
         fc.disconnect()
-        time.sleep(1)  
+        time.sleep(1)
 
         new_result = fc.connect(device=SITL_CONNECTION)
 

tests/test_signing_config.py

@@ -11,8 +11,6 @@
 """
 
 import json
-from pathlib import Path
-from unittest.mock import patch
 
 import pytest
 
@@ -123,14 +121,16 @@ def test_user_can_deserialize_config_from_json(self) -> None:
         THEN: A SigningConfig instance should be created
         AND: All values should match the JSON input
         """
-        json_str = json.dumps({
-            "enabled": True,
-            "sign_outgoing": False,
-            "allow_unsigned_in": True,
-            "accept_unsigned_callbacks": True,
-            "timestamp_offset": 100,
-            "link_id": 7,
-        })
+        json_str = json.dumps(
+            {
+                "enabled": True,
+                "sign_outgoing": False,
+                "allow_unsigned_in": True,
+                "accept_unsigned_callbacks": True,
+                "timestamp_offset": 100,
+                "link_id": 7,
+            }
+        )
 
         config = SigningConfig.from_json(json_str)
 
@@ -206,7 +206,7 @@ def test_invalid_boolean_type_raises_validation_error(self) -> None:
         THEN: A ValueError should be raised
         """
         with pytest.raises(ValueError, match="enabled"):
-            SigningConfig(enabled="true")  
+            SigningConfig(enabled="true")
 
     def test_invalid_timestamp_type_raises_validation_error(self) -> None:
         """
@@ -217,7 +217,7 @@ def test_invalid_timestamp_type_raises_validation_error(self) -> None:
         THEN: A ValueError should be raised
         """
         with pytest.raises(ValueError, match="timestamp_offset"):
-            SigningConfig(timestamp_offset="100")  
+            SigningConfig(timestamp_offset="100")
 
 
 class TestVehicleSigningConfig:

tests/test_signing_keystore.py

@@ -10,17 +10,13 @@
 SPDX-License-Identifier: GPL-3.0-or-later
 """
 
-import json
-import os
-from pathlib import Path
-from unittest.mock import MagicMock, patch
+from unittest.mock import patch
 
 import pytest
 
 from ardupilot_methodic_configurator.backend_signing_keystore import (
     SIGNING_KEY_LENGTH,
     SigningKeystore,
-    StoredKey,
 )