Fix printing log message for None profile id (#440)

* Fix printing log message for `None` profile id

* Add test for coverage

* Remove part of test already tested elsewhere

Author: TheJulianJES

tests/test_device.py

@@ -950,3 +950,16 @@ async def test_join_binding_reporting(zha_gateway: Gateway) -> None:
     assert mock_reporting_config.mock_calls == [
         call({"measured_value": (30, 900, 1e-6)})
     ]
+
+
+async def test_endpoint_none_profile(
+    zha_gateway: Gateway,
+    caplog: pytest.LogCaptureFixture,
+) -> None:
+    """Test endpoint with None profile id being skipped."""
+    zigpy_dev = zigpy_device(zha_gateway, with_basic_cluster_handler=True)
+    zigpy_dev.endpoints[3].profile_id = None
+    zha_device = await join_zigpy_device(zha_gateway, zigpy_dev)
+
+    assert zha_device.async_get_std_clusters() == {}
+    assert "Skipping endpoint, profile is None" in caplog.text

zha/zigbee/endpoint.py

@@ -135,10 +135,13 @@ def new(cls, zigpy_endpoint: ZigpyEndpoint, device: Device) -> Endpoint:
     def add_all_cluster_handlers(self) -> None:
         """Create and add cluster handlers for all input clusters."""
         profile_id = self._zigpy_endpoint.profile_id
-        if profile_id not in (ZLL_PROFILE_ID, ZHA_PROFILE_ID):
+        if profile_id is None:
+            _LOGGER.debug("Skipping endpoint, profile is None")
+            return
+        elif profile_id not in (ZLL_PROFILE_ID, ZHA_PROFILE_ID):
             _LOGGER.debug(
                 "Skipping endpoint, profile is not ZLL or ZHA: 0x%04X",
-                self._zigpy_endpoint.profile_id,
+                profile_id,
             )
             return