Allow loading of more then 1 defined Apprise URL

homeassistant/components/apprise/notify.py

@@ -52,9 +52,15 @@ def get_service(
             return None
 
     # Ordered list of URLs
-    if config.get(CONF_URL) and not a_obj.add(config[CONF_URL]):
-        _LOGGER.error("Invalid Apprise URL(s) supplied")
-        return None
+    if config.get(CONF_URL):
+        has_error = False
+        for entry in config[CONF_URL]:
+            if not a_obj.add(entry):
+                has_error = True
+
+        if has_error:
+            _LOGGER.error("One or more specified Apprise URL(s) are invalid")
+            return None
 
     return AppriseNotificationService(a_obj)
 

tests/components/apprise/test_notify.py

@@ -118,7 +118,48 @@ async def test_apprise_notification(hass: HomeAssistant) -> None:
         await hass.async_block_till_done()
 
         # Validate calls were made under the hood correctly
-        obj.add.assert_called_once_with([config[BASE_COMPONENT]["url"]])
+        obj.add.assert_called_once_with(config[BASE_COMPONENT]["url"])
+        obj.notify.assert_called_once_with(
+            **{"body": data["message"], "title": data["title"], "tag": None}
+        )
+
+
+async def test_apprise_multiple_notification(hass: HomeAssistant) -> None:
+    """Test apprise notification."""
+
+    config = {
+        BASE_COMPONENT: {
+            "name": "test",
+            "platform": "apprise",
+            "url": [
+                "mailto://user:pass@example.com, mailto://user:pass@gmail.com",
+                "json://user:pass@gmail.com",
+            ],
+        }
+    }
+
+    # Our Message
+    data = {"title": "Test Title", "message": "Test Message"}
+
+    with patch(
+        "homeassistant.components.apprise.notify.apprise.Apprise"
+    ) as mock_apprise:
+        obj = MagicMock()
+        obj.add.return_value = True
+        obj.notify.return_value = True
+        mock_apprise.return_value = obj
+        assert await async_setup_component(hass, BASE_COMPONENT, config)
+        await hass.async_block_till_done()
+
+        # Test the existence of our service
+        assert hass.services.has_service(BASE_COMPONENT, "test")
+
+        # Test the call to our underlining notify() call
+        await hass.services.async_call(BASE_COMPONENT, "test", data)
+        await hass.async_block_till_done()
+
+        # Validate 2 calls were made under the hood
+        assert obj.add.call_count == 2
         obj.notify.assert_called_once_with(
             **{"body": data["message"], "title": data["title"], "tag": None}
         )