* Revert "Emulated Hue: changed the reported fallback device-type to fix Alexa compatibility issues (#30013)" This reverts commitddc8d9e25c
. * Revert "Emulated Hue: updated tests (#30013)" This reverts commit90df461e75
. * Emulated Hue + Alexa: changed the fallback device-type again to "Dimmable Light" (#30013) after collective debugging; fixed brightness for on/off-devices and scripts to prevent "device malfunction" response from Alexa (#29899) * Emulated Hue + Alexa: lint (#30013, #29899)
This commit is contained in:
parent
3cbd426c52
commit
2c1b465215
2 changed files with 14 additions and 16 deletions
|
@ -688,27 +688,25 @@ def entity_to_json(config, entity):
|
|||
retval["state"].update(
|
||||
{HUE_API_STATE_COLORMODE: "ct", HUE_API_STATE_CT: state[STATE_COLOR_TEMP]}
|
||||
)
|
||||
elif (
|
||||
entity_features
|
||||
& (
|
||||
SUPPORT_BRIGHTNESS
|
||||
| SUPPORT_SET_POSITION
|
||||
| SUPPORT_SET_SPEED
|
||||
| SUPPORT_VOLUME_SET
|
||||
| SUPPORT_TARGET_TEMPERATURE
|
||||
)
|
||||
) or entity.domain == script.DOMAIN:
|
||||
elif entity_features & (
|
||||
SUPPORT_BRIGHTNESS
|
||||
| SUPPORT_SET_POSITION
|
||||
| SUPPORT_SET_SPEED
|
||||
| SUPPORT_VOLUME_SET
|
||||
| SUPPORT_TARGET_TEMPERATURE
|
||||
):
|
||||
# Dimmable light (Zigbee Device ID: 0x0100)
|
||||
# Supports groups, scenes, on/off and dimming
|
||||
retval["type"] = "Dimmable light"
|
||||
retval["modelid"] = "HASS123"
|
||||
retval["state"].update({HUE_API_STATE_BRI: state[STATE_BRIGHTNESS]})
|
||||
else:
|
||||
# On/off plug-in unit (Zigbee Device ID: 0x0000)
|
||||
# Supports groups and on/off control
|
||||
# Used for compatibility purposes with Alexa instead of "On/off light"
|
||||
retval["type"] = "On/off plug-in unit"
|
||||
retval["modelid"] = "HASS321"
|
||||
# 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})
|
||||
|
||||
return retval
|
||||
|
||||
|
|
|
@ -238,7 +238,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"] == "On/off plug-in unit"
|
||||
assert light_without_brightness_json["type"] == "Dimmable light"
|
||||
|
||||
|
||||
async def test_light_without_brightness_can_be_turned_off(hass_hue, hue_client):
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue