Address @bouwew comments

Address sonarqubecloud comments

Author: dirixmjm

plugwise_usb/network/__init__.py

@@ -434,7 +434,7 @@ async def _discover_stragglers(self) -> None:
         """Repeat Discovery of Nodes with unknown NodeType."""
         while len(self._registry_stragglers) > 0:
             await sleep(NODE_RETRY_DISCOVER_INTERVAL)
-            for mac in list(self._registry_stragglers):
+            for mac in self._registry_stragglers.copy():
                 if await self._discover_node(mac, None):
                     self._registry_stragglers.remove(mac)
             _LOGGER.debug(

plugwise_usb/network/cache.py

@@ -80,11 +80,11 @@ def get_nodetype(self, mac: str) -> NodeType | None:
 
     async def prune_cache(self, registry: list[str]) -> None:
         """Remove items from cache which are not found in registry scan."""
-        _new_nodetypes: dict[str, NodeType] = {}
+        new_nodetypes: dict[str, NodeType] = {}
         for mac in registry:
             if mac == "":
                 continue
             if (node_type := self.get_nodetype(mac)) is not None:
-                _new_nodetypes[mac] = node_type
-        self._nodetypes = _new_nodetypes
+                new_nodetypes[mac] = node_type
+        self._nodetypes = new_nodetypes
         await self.save_cache()

plugwise_usb/network/registry.py

@@ -163,7 +163,7 @@ def network_controller(self) -> tuple[str, NodeType | None]:
         return self.registry[-1]
 
     async def update_network_nodetype(self, mac: str, node_type: NodeType) -> None:
-        """Update NodeType Inside Registry and Cache."""
+        """Update NodeType inside registry and cache."""
         if self._network_cache is None or mac == "":
             return
         await self._network_cache.update_nodetypes(mac, node_type)
@@ -228,7 +228,7 @@ async def load_registrations_from_cache(self) -> None:
         if self._scan_completed_callback is not None:
             await self._scan_completed_callback()
 
-    async def update_node_registration(self, mac: str) -> bool:
+    def update_node_registration(self, mac: str) -> bool:
         """Register (re)joined node to Plugwise network and return network address."""
         return self.update_network_registration(mac)
 

plugwise_usb/nodes/scan.py

@@ -147,9 +147,9 @@ async def _load_defaults(self) -> None:
 
     async def _load_from_cache(self) -> bool:
         """Load states from previous cached information. Returns True if successful."""
-        LoadSuccess = True
+        super_load_success = True
         if not await super()._load_from_cache():
-            LoadSuccess = False
+            super_load_success = False
         self._motion_state = MotionState(
             state=self._motion_from_cache(),
             timestamp=self._motion_timestamp_from_cache(),
@@ -174,7 +174,7 @@ async def _load_from_cache(self) -> bool:
         )
         if dirty:
             await self._scan_configure_update()
-        return LoadSuccess
+        return super_load_success
 
     def _daylight_mode_from_cache(self) -> bool | None:
         """Load awake duration from cache."""

plugwise_usb/nodes/sed.py

@@ -142,9 +142,9 @@ async def _load_defaults(self) -> None:
 
     async def _load_from_cache(self) -> bool:
         """Load states from previous cached information. Returns True if successful."""
-        LoadSuccess = True
+        super_load_success = True
         if not await super()._load_from_cache():
-            LoadSuccess = False
+            super_load_success = False
         dirty = False
         if (awake_duration := self._awake_duration_from_cache()) is None:
             dirty = True
@@ -174,7 +174,7 @@ async def _load_from_cache(self) -> bool:
             await self._sed_configure_update()
         self._awake_timestamp_from_cache()
         self._awake_reason_from_cache()
-        return LoadSuccess
+        return super_load_success
 
     def _awake_duration_from_cache(self) -> int | None:
         """Load awake duration from cache."""

tests/test_usb.py

@@ -1910,7 +1910,7 @@ async def load_callback(event: pw_api.NodeEvent, mac: str) -> None:  # type: ign
     async def test_sed_node(self, monkeypatch: pytest.MonkeyPatch) -> None:  # noqa:  PLR0915
         """Testing properties of SED."""
 
-        def fake_cache(dummy: object, setting: str) -> str | None:  # noqa: PLR0911
+        def fake_cache(dummy: object, setting: str) -> str | bool | None:  # noqa: PLR0911
             """Fake cache retrieval."""
             if setting == pw_node.CACHE_FIRMWARE:
                 return "2011-6-27-8-55-44"
@@ -2106,7 +2106,7 @@ async def load_callback(event: pw_api.NodeEvent, mac: str) -> None:  # type: ign
     async def test_scan_node(self, monkeypatch: pytest.MonkeyPatch) -> None:  # noqa: PLR0915
         """Testing properties of scan."""
 
-        def fake_cache(dummy: object, setting: str) -> str | None:  # noqa: PLR0911 PLR0912
+        def fake_cache(dummy: object, setting: str) -> str | bool | None:  # noqa: PLR0911 PLR0912
             """Fake cache retrieval."""
             if setting == pw_node.CACHE_FIRMWARE:
                 return "2011-6-27-8-55-44"
@@ -2308,7 +2308,7 @@ async def load_callback(event: pw_api.NodeEvent, mac: str) -> None:  # type: ign
     async def test_switch_node(self, monkeypatch: pytest.MonkeyPatch) -> None:
         """Testing properties of switch."""
 
-        def fake_cache(dummy: object, setting: str) -> str | None:  # noqa: PLR0911
+        def fake_cache(dummy: object, setting: str) -> str | bool | None:  # noqa: PLR0911
             """Fake cache retrieval."""
             if setting == pw_node.CACHE_FIRMWARE:
                 return "2011-5-13-7-26-54"
@@ -2411,7 +2411,7 @@ async def test_node_discovery_and_load(  # noqa: PLR0915
     ) -> None:
         """Testing discovery of nodes."""
 
-        def fake_cache(dummy: object, setting: str) -> str | None:  # noqa: PLR0911
+        def fake_cache(dummy: object, setting: str) -> str | bool | None:  # noqa: PLR0911
             """Fake cache retrieval."""
             if setting == pw_node.CACHE_FIRMWARE:
                 return "2011-5-13-7-26-54"