Add config flow for simplepush

.coveragerc

@@ -1064,6 +1064,7 @@ omit =
     homeassistant/components/shelly/sensor.py
     homeassistant/components/shelly/utils.py
     homeassistant/components/sigfox/sensor.py
+    homeassistant/components/simplepush/__init__.py
     homeassistant/components/simplepush/notify.py
     homeassistant/components/simplisafe/__init__.py
     homeassistant/components/simplisafe/alarm_control_panel.py

CODEOWNERS

@@ -927,6 +927,8 @@ build.json @home-assistant/supervisor
 /tests/components/sighthound/ @robmarkcole
 /homeassistant/components/signal_messenger/ @bbernhard
 /tests/components/signal_messenger/ @bbernhard
+/homeassistant/components/simplepush/ @engrbm87
+/tests/components/simplepush/ @engrbm87
 /homeassistant/components/simplisafe/ @bachya
 /tests/components/simplisafe/ @bachya
 /homeassistant/components/sinch/ @bendikrb

homeassistant/components/simplepush/__init__.py

@@ -1 +1,42 @@
 """The simplepush component."""
+
+from homeassistant.config_entries import ConfigEntry
+from homeassistant.const import Platform
+from homeassistant.core import HomeAssistant
+from homeassistant.helpers import discovery
+import homeassistant.helpers.config_validation as cv
+from homeassistant.helpers.typing import ConfigType
+
+from .const import DATA_HASS_CONFIG, DOMAIN
+
+PLATFORMS = [Platform.NOTIFY]
+
+CONFIG_SCHEMA = cv.removed(DOMAIN, raise_if_present=False)
+
+
+async def async_setup(hass: HomeAssistant, config: ConfigType) -> bool:
+    """Set up the simplepush component."""
+
+    hass.data[DATA_HASS_CONFIG] = config
+    return True
+
+
+async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
+    """Set up simplepush from a config entry."""
+
+    hass.async_create_task(
+        discovery.async_load_platform(
+            hass,
+            Platform.NOTIFY,
+            DOMAIN,
+            dict(entry.data),
+            hass.data[DATA_HASS_CONFIG],
+        )
+    )
+
+    return True
+
+
+async def async_unload_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
+    """Unload a config entry."""
+    return await hass.config_entries.async_unload_platforms(entry, PLATFORMS)

homeassistant/components/simplepush/config_flow.py

@@ -0,0 +1,47 @@
+"""Config flow for simplepush integration."""
+from __future__ import annotations
+
+from typing import Any
+
+import voluptuous as vol
+
+from homeassistant import config_entries
+from homeassistant.const import CONF_NAME, CONF_PASSWORD
+from homeassistant.data_entry_flow import FlowResult
+
+from .const import ATTR_ENCRYPTED, CONF_DEVICE_KEY, CONF_SALT, DEFAULT_NAME, DOMAIN
+
+
+class SimplePushFlowHandler(config_entries.ConfigFlow, domain=DOMAIN):
+    """Handle a config flow for simplepush."""
+
+    async def async_step_user(
+        self, user_input: dict[str, Any] | None = None
+    ) -> FlowResult:
+        """Handle a flow initiated by the user."""
+        if user_input is not None:
+
+            await self.async_set_unique_id(user_input[CONF_DEVICE_KEY])
+            self._abort_if_unique_id_configured()
+
+            self._async_abort_entries_match(
+                {
+                    CONF_NAME: user_input[CONF_NAME],
+                }
+            )
+            return self.async_create_entry(
+                title=user_input[CONF_NAME],
+                data=user_input,
+            )
+
+        return self.async_show_form(
+            step_id="user",
+            data_schema=vol.Schema(
+                {
+                    vol.Required(CONF_DEVICE_KEY): str,
+                    vol.Required(CONF_NAME, default=DEFAULT_NAME): str,
+                    vol.Inclusive(CONF_PASSWORD, ATTR_ENCRYPTED): str,
+                    vol.Inclusive(CONF_SALT, ATTR_ENCRYPTED): str,
+                }
+            ),
+        )

homeassistant/components/simplepush/const.py

@@ -0,0 +1,13 @@
+"""Constants for the simplepush integration."""
+
+from typing import Final
+
+DOMAIN: Final = "simplepush"
+DEFAULT_NAME: Final = "simplepush"
+DATA_HASS_CONFIG: Final = "simpleush_hass_config"
+
+ATTR_ENCRYPTED: Final = "encrypted"
+ATTR_EVENT: Final = "event"
+
+CONF_DEVICE_KEY: Final = "device_key"
+CONF_SALT: Final = "salt"

homeassistant/components/simplepush/manifest.json

@@ -3,7 +3,8 @@
   "name": "Simplepush",
   "documentation": "https://www.home-assistant.io/integrations/simplepush",
   "requirements": ["simplepush==1.1.4"],
-  "codeowners": [],
+  "codeowners": ["@engrbm87"],
+  "config_flow": true,
   "iot_class": "cloud_polling",
   "loggers": ["simplepush"]
 }

homeassistant/components/simplepush/notify.py

@@ -1,4 +1,9 @@
 """Simplepush notification service."""
+from __future__ import annotations
+
+import logging
+from typing import Any
+
 from simplepush import send, send_encrypted
 import voluptuous as vol
 
@@ -8,14 +13,15 @@
     PLATFORM_SCHEMA,
     BaseNotificationService,
 )
+from homeassistant.components.notify.const import ATTR_DATA, DOMAIN as NOTIFY_DOMAIN
 from homeassistant.const import CONF_EVENT, CONF_PASSWORD
+from homeassistant.core import HomeAssistant
 import homeassistant.helpers.config_validation as cv
+from homeassistant.helpers.typing import ConfigType, DiscoveryInfoType
 
-ATTR_ENCRYPTED = "encrypted"
-
-CONF_DEVICE_KEY = "device_key"
-CONF_SALT = "salt"
+from .const import ATTR_ENCRYPTED, ATTR_EVENT, CONF_DEVICE_KEY, CONF_SALT
 
+# Configuring simplepush under the notify platform will be deprecated in 2022.8.0
 PLATFORM_SCHEMA = PLATFORM_SCHEMA.extend(
     {
         vol.Required(CONF_DEVICE_KEY): cv.string,
@@ -25,34 +31,56 @@
     }
 )
 
+_LOGGER = logging.getLogger(__name__)
+
 
-def get_service(hass, config, discovery_info=None):
+async def async_get_service(
+    hass: HomeAssistant,
+    config: ConfigType,
+    discovery_info: DiscoveryInfoType | None = None,
+) -> SimplePushNotificationService | None:
     """Get the Simplepush notification service."""
+    if discovery_info:
+        return SimplePushNotificationService(discovery_info)
+
+    _LOGGER.warning(
+        "Manually configured simplepush integration found under platform key '%s'. "
+        "Please remove and configure through the UI",
+        NOTIFY_DOMAIN,
+    )
     return SimplePushNotificationService(config)
 
 
 class SimplePushNotificationService(BaseNotificationService):
     """Implementation of the notification service for Simplepush."""
 
-    def __init__(self, config):
+    def __init__(self, config: dict[str, Any]) -> None:
         """Initialize the Simplepush notification service."""
-        self._device_key = config.get(CONF_DEVICE_KEY)
-        self._event = config.get(CONF_EVENT)
-        self._password = config.get(CONF_PASSWORD)
-        self._salt = config.get(CONF_SALT)
+        self._device_key: str = config[CONF_DEVICE_KEY]
+        self._event: str | None = config.get(CONF_EVENT)
+        self._password: str | None = config.get(CONF_PASSWORD)
+        self._salt: str | None = config.get(CONF_SALT)
 
-    def send_message(self, message="", **kwargs):
+    def send_message(self, message: str, **kwargs: Any) -> None:
         """Send a message to a Simplepush user."""
         title = kwargs.get(ATTR_TITLE, ATTR_TITLE_DEFAULT)
 
+        # event can now be passwed in the service data
+        event = None
+        if data := kwargs.get(ATTR_DATA):
+            event = data.get(ATTR_EVENT)
+
+        # use event from config until YAML config is removed
+        event = event or self._event
+
         if self._password:
             send_encrypted(
                 self._device_key,
                 self._password,
                 self._salt,
                 title,
                 message,
-                event=self._event,
+                event=event,
             )
         else:
-            send(self._device_key, title, message, event=self._event)
+            send(self._device_key, title, message, event=event)

homeassistant/components/simplepush/strings.json

@@ -0,0 +1,18 @@
+{
+  "config": {
+    "step": {
+      "user": {
+        "data": {
+          "name": "[%key:common::config_flow::data::name%]",
+          "device_key": "The device key of your device",
+          "event": "The event for the events.",
+          "password": "The password of the encryption used by your device",
+          "salt": "The salt used by your device."
+        }
+      }
+    },
+    "abort": {
+      "already_configured": "[%key:common::config_flow::abort::already_configured_device%]"
+    }
+  }
+}

homeassistant/components/simplepush/translations/en.json

@@ -0,0 +1,18 @@
+{
+    "config": {
+        "abort": {
+            "already_configured": "Device is already configured"
+        },
+        "step": {
+            "user": {
+                "data": {
+                    "device_key": "The device key of your device",
+                    "event": "The event for the events.",
+                    "name": "Name",
+                    "password": "The password of the encryption used by your device",
+                    "salt": "The salt used by your device."
+                }
+            }
+        }
+    }
+}

homeassistant/generated/config_flows.py

@@ -311,6 +311,7 @@
         "shelly",
         "shopping_list",
         "sia",
+        "simplepush",
         "simplisafe",
         "skybell",
         "slack",

requirements_test_all.txt

@@ -1430,6 +1430,9 @@ sharkiq==0.0.1
 # homeassistant.components.sighthound
 simplehound==0.3
 
+# homeassistant.components.simplepush
+simplepush==1.1.4
+
 # homeassistant.components.simplisafe
 simplisafe-python==2022.06.0
 

tests/components/simplepush/__init__.py

@@ -0,0 +1 @@
+"""Tests for the simeplush integration."""

tests/components/simplepush/test_config_flow.py

@@ -0,0 +1,76 @@
+"""Test Simplepush config flow."""
+from homeassistant import config_entries, data_entry_flow
+from homeassistant.components.simplepush.const import CONF_DEVICE_KEY, CONF_SALT, DOMAIN
+from homeassistant.const import CONF_NAME, CONF_PASSWORD
+from homeassistant.core import HomeAssistant
+
+from tests.common import MockConfigEntry
+
+MOCK_CONFIG = {
+    CONF_DEVICE_KEY: "abc",
+    CONF_NAME: "simplepush",
+    CONF_PASSWORD: "password",
+    CONF_SALT: "salt",
+}
+
+
+async def test_flow_successful(hass: HomeAssistant) -> None:
+    """Test user initialized flow."""
+    result = await hass.config_entries.flow.async_init(
+        DOMAIN,
+        context={"source": config_entries.SOURCE_USER},
+    )
+    result = await hass.config_entries.flow.async_configure(
+        result["flow_id"],
+        user_input=MOCK_CONFIG,
+    )
+    assert result["type"] == data_entry_flow.RESULT_TYPE_CREATE_ENTRY
+    assert result["title"] == "simplepush"
+    assert result["data"] == MOCK_CONFIG
+
+
+async def test_flow_user_device_key_already_configured(hass):
+    """Test user initialized flow with duplicate device key."""
+    entry = MockConfigEntry(
+        domain=DOMAIN,
+        data=MOCK_CONFIG,
+        unique_id="abc",
+    )
+
+    entry.add_to_hass(hass)
+
+    result = await hass.config_entries.flow.async_init(
+        DOMAIN,
+        context={"source": config_entries.SOURCE_USER},
+    )
+    result = await hass.config_entries.flow.async_configure(
+        result["flow_id"],
+        user_input=MOCK_CONFIG,
+    )
+    assert result["type"] == data_entry_flow.RESULT_TYPE_ABORT
+    assert result["reason"] == "already_configured"
+
+
+async def test_flow_user_name_already_configured(hass):
+    """Test user initialized flow with duplicate name."""
+    entry = MockConfigEntry(
+        domain=DOMAIN,
+        data=MOCK_CONFIG,
+        unique_id="abc",
+    )
+
+    entry.add_to_hass(hass)
+
+    new_entry = MOCK_CONFIG.copy()
+    new_entry[CONF_DEVICE_KEY] = "abc1"
+
+    result = await hass.config_entries.flow.async_init(
+        DOMAIN,
+        context={"source": config_entries.SOURCE_USER},
+    )
+    result = await hass.config_entries.flow.async_configure(
+        result["flow_id"],
+        user_input=MOCK_CONFIG,
+    )
+    assert result["type"] == data_entry_flow.RESULT_TYPE_ABORT
+    assert result["reason"] == "already_configured"