Sort more update_node_info() arguments

Author: bouwew

plugwise_usb/nodes/circle.py

@@ -1038,10 +1038,10 @@ async def update_node_details(
         self,
         firmware: datetime | None,
         hardware: str | None,
+        logaddress_pointer: int | None,
         node_type: NodeType | None,
-        timestamp: datetime | None,
         relay_state: bool | None,
-        logaddress_pointer: int | None,
+        timestamp: datetime | None,
     ) -> bool:
         """Process new node info and return true if all fields are updated."""
         if relay_state is not None:
@@ -1055,10 +1055,10 @@ async def update_node_details(
         return await super().update_node_details(
             firmware,
             hardware,
+            logaddress_pointer,
             node_type,
-            timestamp,
             relay_state,
-            logaddress_pointer,
+            timestamp,
         )
 
     async def unload(self) -> None:

tests/test_usb.py

@@ -2162,10 +2162,10 @@ async def load_callback(event: pw_api.NodeEvent, mac: str) -> None:  # type: ign
         await test_scan.update_node_details(
             firmware=dt(2011, 6, 27, 8, 55, 44, tzinfo=UTC),
             hardware="080007",
+            logaddress_pointer=None,
             node_type=None,
-            timestamp=None,
             relay_state=None,
-            logaddress_pointer=None,
+            timestamp=None,
         )
         assert await test_scan.load()
 
@@ -2266,10 +2266,10 @@ async def load_callback(event: pw_api.NodeEvent, mac: str) -> None:  # type: ign
         await test_scan.update_node_details(
             firmware=dt(2011, 6, 27, 8, 55, 44, tzinfo=UTC),
             hardware="080007",
+            logaddress_pointer=None,
             node_type=None,
-            timestamp=None,
             relay_state=None,
-            logaddress_pointer=None,
+            timestamp=None,
         )
         test_scan.cache_enabled = True
         assert await test_scan.load()
@@ -2342,10 +2342,10 @@ async def load_callback(event: pw_api.NodeEvent, mac: str) -> None:  # type: ign
         await test_switch.update_node_details(
             firmware=dt(2011, 6, 27, 9, 4, 10, tzinfo=UTC),
             hardware="070051",
+            logaddress_pointer=None,
             node_type=None,
-            timestamp=None,
             relay_state=None,
-            logaddress_pointer=None,
+            timestamp=None,
         )
         assert await test_switch.load()
 
@@ -2359,10 +2359,10 @@ async def load_callback(event: pw_api.NodeEvent, mac: str) -> None:  # type: ign
         await test_switch.update_node_details(
             firmware=dt(2011, 6, 27, 9, 4, 10, tzinfo=UTC),
             hardware="070051",
+            logaddress_pointer=None,
             node_type=None,
-            timestamp=None,
             relay_state=None,
-            logaddress_pointer=None,
+            timestamp=None,
         )
         test_switch.cache_enabled = True
         assert test_switch.cache_enabled is True