evlli/hass-core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Allow None on Renault binary sensors (#65997)

Browse source 
* Enable None on renault binary sensors

* Adjust tests

Co-authored-by: epenet <epenet@users.noreply.github.com>
This commit is contained in:
epenet 2022-02-07 14:06:40 +01:00 • committed by GitHub
parent d81139377c
commit 486c068111
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

7
homeassistant/components/renault/binary_sensor.py
View file

@ -64,10 +64,9 @@ class RenaultBinarySensor(
@property
def is_on(self) -> bool | None:
"""Return true if the binary sensor is on."""
return (
self._get_data_attr(self.entity_description.on_key)
== self.entity_description.on_value
)
if (data := self._get_data_attr(self.entity_description.on_key)) is None:
return None
return data == self.entity_description.on_value
BINARY_SENSOR_TYPES: tuple[RenaultBinarySensorEntityDescription, ...] = (

4
tests/components/renault/test_binary_sensor.py
View file

@ -4,7 +4,7 @@ from unittest.mock import patch
import pytest
from homeassistant.config_entries import ConfigEntry
from homeassistant.const import STATE_OFF, Platform
from homeassistant.const import STATE_UNKNOWN, Platform
from homeassistant.core import HomeAssistant
from . import (
@ -63,7 +63,7 @@ async def test_binary_sensor_empty(
expected_entities = mock_vehicle[Platform.BINARY_SENSOR]
assert len(entity_registry.entities) == len(expected_entities)
check_entities_no_data(hass, entity_registry, expected_entities, STATE_OFF)
check_entities_no_data(hass, entity_registry, expected_entities, STATE_UNKNOWN)
@pytest.mark.usefixtures("fixtures_with_invalid_upstream_exception")