Model_type should be None when not existing

Author: bouwew

plugwise_usb/nodes/node.py

@@ -510,10 +510,9 @@ async def update_node_details(
                         self.mac,
                         hardware,
                     )
+                self._node_info.model_type = None
                 if len(model_info) > 1:
                     self._node_info.model_type = " ".join(model_info[1:])
-                else:
-                    self._node_info.model_type = ""
                 if self._node_info.model is not None:
                     self._node_info.name = f"{model_info[0]} {self._node_info.mac[-5:]}"
             self._set_cache(CACHE_HARDWARE, hardware)

tests/test_usb.py

@@ -588,7 +588,7 @@ async def test_stick_node_discovered_subscription(
         )
         assert stick.nodes["5555555555555555"].node_info.version == "080007"
         assert stick.nodes["5555555555555555"].node_info.model == "Scan"
-        assert stick.nodes["5555555555555555"].node_info.model_type == ""
+        assert stick.nodes["5555555555555555"].node_info.model_type == None
         assert stick.nodes["5555555555555555"].available
         assert stick.nodes["5555555555555555"].node_info.is_battery_powered
         assert sorted(stick.nodes["5555555555555555"].features) == sorted(
@@ -2531,7 +2531,7 @@ async def test_node_discovery_and_load(
         )
         assert stick.nodes["5555555555555555"].node_info.version == "080007"
         assert stick.nodes["5555555555555555"].node_info.model == "Scan"
-        assert stick.nodes["5555555555555555"].node_info.model_type == ""
+        assert stick.nodes["5555555555555555"].node_info.model_type == None
         assert stick.nodes["5555555555555555"].available
         assert stick.nodes["5555555555555555"].node_info.is_battery_powered
         assert sorted(stick.nodes["5555555555555555"].features) == sorted(
@@ -2596,7 +2596,7 @@ async def test_node_discovery_and_load(
         )
         assert stick.nodes["8888888888888888"].node_info.version == "070051"
         assert stick.nodes["8888888888888888"].node_info.model == "Switch"
-        assert stick.nodes["8888888888888888"].node_info.model_type == ""
+        assert stick.nodes["8888888888888888"].node_info.model_type == None
         assert stick.nodes["8888888888888888"].available
         assert stick.nodes["8888888888888888"].node_info.is_battery_powered
         assert sorted(stick.nodes["8888888888888888"].features) == sorted(