From 8c1935ce0eb8697ba4abdb0dac6121341811c9f7 Mon Sep 17 00:00:00 2001 From: epenet <6771947+epenet@users.noreply.github.com> Date: Wed, 27 Oct 2021 15:13:18 +0200 Subject: [PATCH] Use DeviceInfo in smappee (#58522) Co-authored-by: epenet --- .../components/smappee/binary_sensor.py | 33 ++++++++++--------- homeassistant/components/smappee/sensor.py | 17 +++++----- 2 files changed, 26 insertions(+), 24 deletions(-) diff --git a/homeassistant/components/smappee/binary_sensor.py b/homeassistant/components/smappee/binary_sensor.py index 4cc5cca9a23..7e46a1021e5 100644 --- a/homeassistant/components/smappee/binary_sensor.py +++ b/homeassistant/components/smappee/binary_sensor.py @@ -3,6 +3,7 @@ from homeassistant.components.binary_sensor import ( DEVICE_CLASS_PRESENCE, BinarySensorEntity, ) +from homeassistant.helpers.entity import DeviceInfo from .const import DOMAIN @@ -71,15 +72,15 @@ class SmappeePresence(BinarySensorEntity): ) @property - def device_info(self): + def device_info(self) -> DeviceInfo: """Return the device info for this binary sensor.""" - return { - "identifiers": {(DOMAIN, self._service_location.device_serial_number)}, - "name": self._service_location.service_location_name, - "manufacturer": "Smappee", - "model": self._service_location.device_model, - "sw_version": self._service_location.firmware_version, - } + return DeviceInfo( + identifiers={(DOMAIN, self._service_location.device_serial_number)}, + manufacturer="Smappee", + model=self._service_location.device_model, + name=self._service_location.service_location_name, + sw_version=self._service_location.firmware_version, + ) async def async_update(self): """Get the latest data from Smappee and update the state.""" @@ -154,15 +155,15 @@ class SmappeeAppliance(BinarySensorEntity): ) @property - def device_info(self): + def device_info(self) -> DeviceInfo: """Return the device info for this binary sensor.""" - return { - "identifiers": {(DOMAIN, self._service_location.device_serial_number)}, - "name": self._service_location.service_location_name, - "manufacturer": "Smappee", - "model": self._service_location.device_model, - "sw_version": self._service_location.firmware_version, - } + return DeviceInfo( + identifiers={(DOMAIN, self._service_location.device_serial_number)}, + manufacturer="Smappee", + model=self._service_location.device_model, + name=self._service_location.service_location_name, + sw_version=self._service_location.firmware_version, + ) async def async_update(self): """Get the latest data from Smappee and update the state.""" diff --git a/homeassistant/components/smappee/sensor.py b/homeassistant/components/smappee/sensor.py index af66b788a41..276248fd6ae 100644 --- a/homeassistant/components/smappee/sensor.py +++ b/homeassistant/components/smappee/sensor.py @@ -18,6 +18,7 @@ from homeassistant.const import ( ENERGY_WATT_HOUR, POWER_WATT, ) +from homeassistant.helpers.entity import DeviceInfo from .const import DOMAIN @@ -372,15 +373,15 @@ class SmappeeSensor(SensorEntity): ) @property - def device_info(self): + def device_info(self) -> DeviceInfo: """Return the device info for this sensor.""" - return { - "identifiers": {(DOMAIN, self._service_location.device_serial_number)}, - "name": self._service_location.service_location_name, - "manufacturer": "Smappee", - "model": self._service_location.device_model, - "sw_version": self._service_location.firmware_version, - } + return DeviceInfo( + identifiers={(DOMAIN, self._service_location.device_serial_number)}, + manufacturer="Smappee", + model=self._service_location.device_model, + name=self._service_location.service_location_name, + sw_version=self._service_location.firmware_version, + ) async def async_update(self): """Get the latest data from Smappee and update the state."""