Ensure all chars are polling when requesting manual update in homekit_controller (#124582)

related issue #123963

tests/components/homekit_controller/test_connection.py

@@ -12,6 +12,7 @@ from aiohomekit.testing import FakeController
 import pytest
 
 from homeassistant.components.homekit_controller.const import (
+    DEBOUNCE_COOLDOWN,
     DOMAIN,
     IDENTIFIER_ACCESSORY_ID,
     IDENTIFIER_LEGACY_ACCESSORY_ID,
@@ -22,12 +23,14 @@ from homeassistant.const import STATE_OFF, STATE_UNAVAILABLE
 from homeassistant.core import HomeAssistant
 from homeassistant.exceptions import HomeAssistantError
 from homeassistant.helpers import device_registry as dr, entity_registry as er
+from homeassistant.helpers.entity_component import async_update_entity
 
 from .common import (
     setup_accessories_from_file,
     setup_platform,
     setup_test_accessories,
     setup_test_component,
+    time_changed,
 )
 
 from tests.common import MockConfigEntry
@@ -399,3 +402,40 @@ async def test_poll_firmware_version_only_all_watchable_accessory_mode(
         state = await helper.poll_and_get_state()
         assert state.state == STATE_OFF
         assert mock_get_characteristics.call_count == 8
+
+
+async def test_manual_poll_all_chars(
+    hass: HomeAssistant, get_next_aid: Callable[[], int]
+) -> None:
+    """Test that a manual poll will check all chars."""
+
+    def _create_accessory(accessory: Accessory) -> Service:
+        service = accessory.add_service(ServicesTypes.LIGHTBULB, name="TestDevice")
+
+        on_char = service.add_char(CharacteristicsTypes.ON)
+        on_char.value = 0
+
+        brightness = service.add_char(CharacteristicsTypes.BRIGHTNESS)
+        brightness.value = 0
+
+        return service
+
+    helper = await setup_test_component(hass, get_next_aid(), _create_accessory)
+
+    with mock.patch.object(
+        helper.pairing,
+        "get_characteristics",
+        wraps=helper.pairing.get_characteristics,
+    ) as mock_get_characteristics:
+        # Initial state is that the light is off
+        await helper.poll_and_get_state()
+        # Verify only firmware version is polled
+        assert mock_get_characteristics.call_args_list[0][0][0] == {(1, 7)}
+
+        # Now do a manual poll to ensure all chars are polled
+        mock_get_characteristics.reset_mock()
+        await async_update_entity(hass, helper.entity_id)
+        await time_changed(hass, 60)
+        await time_changed(hass, DEBOUNCE_COOLDOWN)
+        await hass.async_block_till_done()
+        assert len(mock_get_characteristics.call_args_list[0][0][0]) > 1