Don't advertise switch devices as dimmable lights (#37978)

This issue has been corrected and then reverted multiple times.
It seems that the core issue was a casing one (On/off vs On/Off) ; for better
matching with a real Hue hub, also add the productname.

Tested to work with echo 2 and echo 5.
This commit is contained in:
Jean-Yves Avenard 2020-07-20 18:33:56 +10:00 committed by GitHub
parent 6ea5c8aed9
commit 36ee9ff58f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 7 deletions

View file

@ -744,12 +744,11 @@ def entity_to_json(config, entity):
retval["modelid"] = "HASS123"
retval["state"].update({HUE_API_STATE_BRI: state[STATE_BRIGHTNESS]})
else:
# Dimmable light (Zigbee Device ID: 0x0100)
# Supports groups, scenes, on/off and dimming
# Reports fixed brightness for compatibility with Alexa.
retval["type"] = "Dimmable light"
retval["modelid"] = "HASS123"
retval["state"].update({HUE_API_STATE_BRI: HUE_API_STATE_BRI_MAX})
# On/Off light (ZigBee Device ID: 0x0000)
# Supports groups, scenes and on/off control
retval["type"] = "On/Off light"
retval["productname"] = "On/Off light"
retval["modelid"] = "HASS321"
return retval

View file

@ -252,7 +252,7 @@ async def test_light_without_brightness_supported(hass_hue, hue_client):
)
assert light_without_brightness_json["state"][HUE_API_STATE_ON] is True
assert light_without_brightness_json["type"] == "Dimmable light"
assert light_without_brightness_json["type"] == "On/Off light"
async def test_light_without_brightness_can_be_turned_off(hass_hue, hue_client):