Add binary_platform to iaqualink integration, add unique_id

.coveragerc

@@ -289,6 +289,7 @@ omit =
     homeassistant/components/hydrawise/*
     homeassistant/components/hyperion/light.py
     homeassistant/components/ialarm/alarm_control_panel.py
+    homeassistant/components/iaqualink/binary_sensor.py
     homeassistant/components/iaqualink/climate.py
     homeassistant/components/iaqualink/light.py
     homeassistant/components/iaqualink/sensor.py

homeassistant/components/iaqualink/__init__.py

@@ -7,6 +7,7 @@
 import voluptuous as vol
 
 from iaqualink import (
+    AqualinkBinarySensor,
     AqualinkClient,
     AqualinkLight,
     AqualinkLoginException,
@@ -16,6 +17,7 @@
 )
 
 from homeassistant import config_entries
+from homeassistant.components.binary_sensor import DOMAIN as BINARY_SENSOR_DOMAIN
 from homeassistant.components.climate import DOMAIN as CLIMATE_DOMAIN
 from homeassistant.components.light import DOMAIN as LIGHT_DOMAIN
 from homeassistant.components.sensor import DOMAIN as SENSOR_DOMAIN
@@ -28,7 +30,6 @@
     async_dispatcher_connect,
     async_dispatcher_send,
 )
-from homeassistant.helpers.entity import Entity
 from homeassistant.helpers.event import async_track_time_interval
 import homeassistant.helpers.config_validation as cv
 from homeassistant.helpers.typing import ConfigType, HomeAssistantType
@@ -76,6 +77,7 @@ async def async_setup_entry(hass: HomeAssistantType, entry: ConfigEntry) -> None
     password = entry.data[CONF_PASSWORD]
 
     # These will contain the initialized devices
+    binary_sensors = hass.data[DOMAIN][BINARY_SENSOR_DOMAIN] = []
     climates = hass.data[DOMAIN][CLIMATE_DOMAIN] = []
     lights = hass.data[DOMAIN][LIGHT_DOMAIN] = []
     sensors = hass.data[DOMAIN][SENSOR_DOMAIN] = []
@@ -103,12 +105,17 @@ async def async_setup_entry(hass: HomeAssistantType, entry: ConfigEntry) -> None
             climates += [dev]
         elif isinstance(dev, AqualinkLight):
             lights += [dev]
+        elif isinstance(dev, AqualinkBinarySensor):
+            binary_sensors += [dev]
         elif isinstance(dev, AqualinkSensor):
             sensors += [dev]
         elif isinstance(dev, AqualinkToggle):
             switches += [dev]
 
     forward_setup = hass.config_entries.async_forward_entry_setup
+    if binary_sensors:
+        _LOGGER.debug("Got %s binary sensors: %s", len(binary_sensors), binary_sensors)
+        hass.async_create_task(forward_setup(entry, BINARY_SENSOR_DOMAIN))
     if climates:
         _LOGGER.debug("Got %s climates: %s", len(climates), climates)
         hass.async_create_task(forward_setup(entry, CLIMATE_DOMAIN))
@@ -138,6 +145,8 @@ async def async_unload_entry(hass: HomeAssistantType, entry: ConfigEntry) -> boo
 
     tasks = []
 
+    if hass.data[DOMAIN][BINARY_SENSOR_DOMAIN]:
+        tasks += [forward_unload(entry, BINARY_SENSOR_DOMAIN)]
     if hass.data[DOMAIN][CLIMATE_DOMAIN]:
         tasks += [forward_unload(entry, CLIMATE_DOMAIN)]
     if hass.data[DOMAIN][LIGHT_DOMAIN]:
@@ -164,7 +173,7 @@ async def wrapper(self, *args, **kwargs):
     return wrapper
 
 
-class AqualinkEntity(Entity):
+class AqualinkEntityMixin:
     """Abstract class for all Aqualink platforms.
 
     Entity state is updated via the interval timer within the integration.
@@ -190,3 +199,8 @@ def should_poll(self) -> bool:
         updates on a timer.
         """
         return False
+
+    @property
+    def unique_id(self) -> str:
+        """Return a unique identifier for this entity."""
+        return "iaqualink_{0}_{1}".format(self.dev.system.serial, self.dev.name)

homeassistant/components/iaqualink/binary_sensor.py

@@ -0,0 +1,54 @@
+"""Support for Aqualink temperature sensors."""
+import logging
+
+from iaqualink import AqualinkBinarySensor
+
+from homeassistant.components.binary_sensor import (
+    BinarySensorDevice,
+    DEVICE_CLASS_COLD,
+    DOMAIN,
+)
+from homeassistant.config_entries import ConfigEntry
+from homeassistant.helpers.typing import HomeAssistantType
+
+from . import AqualinkEntityMixin
+from .const import DOMAIN as AQUALINK_DOMAIN
+
+_LOGGER = logging.getLogger(__name__)
+
+PARALLEL_UPDATES = 0
+
+
+async def async_setup_entry(
+    hass: HomeAssistantType, config_entry: ConfigEntry, async_add_entities
+) -> None:
+    """Set up discovered binary sensors."""
+    devs = []
+    for dev in hass.data[AQUALINK_DOMAIN][DOMAIN]:
+        devs.append(HassAqualinkBinarySensor(dev))
+    async_add_entities(devs, True)
+
+
+class HassAqualinkBinarySensor(BinarySensorDevice, AqualinkEntityMixin):
+    """Representation of a binary sensor."""
+
+    def __init__(self, dev: AqualinkBinarySensor):
+        """Initialize the binary sensor."""
+        self.dev = dev
+
+    @property
+    def name(self) -> str:
+        """Return the name of the binary sensor."""
+        return self.dev.label
+
+    @property
+    def is_on(self) -> bool:
+        """Return whether the binary sensor is on or not."""
+        return self.dev.is_on
+
+    @property
+    def device_class(self) -> str:
+        """Return the class of the binary sensor."""
+        if self.name == "Freeze Protection":
+            return DEVICE_CLASS_COLD
+        return None

homeassistant/components/iaqualink/climate.py

@@ -27,7 +27,7 @@
 from homeassistant.const import ATTR_TEMPERATURE, TEMP_CELSIUS, TEMP_FAHRENHEIT
 from homeassistant.helpers.typing import HomeAssistantType
 
-from . import AqualinkEntity, refresh_system
+from . import AqualinkEntityMixin, refresh_system
 from .const import DOMAIN as AQUALINK_DOMAIN, CLIMATE_SUPPORTED_MODES
 
 _LOGGER = logging.getLogger(__name__)
@@ -45,7 +45,7 @@ async def async_setup_entry(
     async_add_entities(devs, True)
 
 
-class HassAqualinkThermostat(ClimateDevice, AqualinkEntity):
+class HassAqualinkThermostat(ClimateDevice, AqualinkEntityMixin):
     """Representation of a thermostat."""
 
     def __init__(self, dev: AqualinkThermostat):

homeassistant/components/iaqualink/light.py

@@ -14,7 +14,7 @@
 from homeassistant.config_entries import ConfigEntry
 from homeassistant.helpers.typing import HomeAssistantType
 
-from . import AqualinkEntity, refresh_system
+from . import AqualinkEntityMixin, refresh_system
 from .const import DOMAIN as AQUALINK_DOMAIN
 
 _LOGGER = logging.getLogger(__name__)
@@ -32,7 +32,7 @@ async def async_setup_entry(
     async_add_entities(devs, True)
 
 
-class HassAqualinkLight(Light, AqualinkEntity):
+class HassAqualinkLight(Light, AqualinkEntityMixin):
     """Representation of a light."""
 
     def __init__(self, dev: AqualinkLight):

homeassistant/components/iaqualink/sensor.py

@@ -7,9 +7,10 @@
 from homeassistant.components.sensor import DOMAIN
 from homeassistant.config_entries import ConfigEntry
 from homeassistant.const import DEVICE_CLASS_TEMPERATURE, TEMP_CELSIUS, TEMP_FAHRENHEIT
+from homeassistant.helpers.entity import Entity
 from homeassistant.helpers.typing import HomeAssistantType
 
-from . import AqualinkEntity
+from . import AqualinkEntityMixin
 from .const import DOMAIN as AQUALINK_DOMAIN
 
 _LOGGER = logging.getLogger(__name__)
@@ -27,7 +28,7 @@ async def async_setup_entry(
     async_add_entities(devs, True)
 
 
-class HassAqualinkSensor(AqualinkEntity):
+class HassAqualinkSensor(Entity, AqualinkEntityMixin):
     """Representation of a sensor."""
 
     def __init__(self, dev: AqualinkSensor):

homeassistant/components/iaqualink/switch.py

@@ -7,7 +7,7 @@
 from homeassistant.config_entries import ConfigEntry
 from homeassistant.helpers.typing import HomeAssistantType
 
-from . import AqualinkEntity, refresh_system
+from . import AqualinkEntityMixin, refresh_system
 from .const import DOMAIN as AQUALINK_DOMAIN
 
 _LOGGER = logging.getLogger(__name__)
@@ -25,7 +25,7 @@ async def async_setup_entry(
     async_add_entities(devs, True)
 
 
-class HassAqualinkSwitch(SwitchDevice, AqualinkEntity):
+class HassAqualinkSwitch(SwitchDevice, AqualinkEntityMixin):
     """Representation of a switch."""
 
     def __init__(self, dev: AqualinkToggle):