From a0e64b4f21d808ec4f01b630954f614a3c195f2e Mon Sep 17 00:00:00 2001
From: Chris Caron <lead2gold@gmail.com>
Date: Tue, 20 Feb 2024 17:27:44 -0500
Subject: [PATCH] Added test cases to enforce how URL parsing is actioned

---
 test/test_api.py | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 48 insertions(+)

diff --git a/test/test_api.py b/test/test_api.py
index 39aab9680..108088798 100644
--- a/test/test_api.py
+++ b/test/test_api.py
@@ -175,10 +175,58 @@ def apprise_test(do_notify):
 
     # Clear our server listings again
     a.clear()
+    assert len(a) == 0
 
     # No servers to notify
     assert do_notify(a, title="my title", body="my body") is False
 
+    # More Variations of Multiple Adding of URLs
+    a = Apprise()
+    assert a.add(servers)
+    assert len(a) == 2
+    a.clear()
+
+    assert a.add('ntfys://user:pass@host/test, json://localhost')
+    assert len(a) == 2
+    a.clear()
+
+    assert a.add(['ntfys://user:pass@host/test', 'json://localhost'])
+    assert len(a) == 2
+    a.clear()
+
+    assert a.add(('ntfys://user:pass@host/test', 'json://localhost'))
+    assert len(a) == 2
+    a.clear()
+
+    assert a.add(set(['ntfys://user:pass@host/test', 'json://localhost']))
+    assert len(a) == 2
+    a.clear()
+
+    # Pass a list entry containing 1 string with 2 elements in it
+    # Mimic Home-Assistant core-2024.2.1 Issue:
+    #   - https://github.com/home-assistant/core/issues/110242
+    #
+    # In this case, the first one will load, but not the second entry
+    # This is by design; but captured here to illustrate the issue
+    assert a.add(['ntfys://user:pass@host/test, json://localhost'])
+    assert len(a) == 1
+    assert a[0].url().startswith('ntfys://')
+    a.clear()
+
+    # Following thorugh with the problem of providing a list containing
+    # an entry with 2 URLs in it... while the ntfys parsed okay above,
+    # the same can't be said for other combinations.  It's important
+    # to always keep strings separately
+    assert a.add(['mailto://user:pass@example.com, json://localhost']) is False
+    assert len(a) == 0
+
+    # Showing that the URLs were valid on their own:
+    assert a.add(*['mailto://user:pass@example.com, json://localhost'])
+    assert len(a) == 2
+    assert a[0].url().startswith('mailto://')
+    assert a[1].url().startswith('json://')
+    a.clear()
+
     class BadNotification(NotifyBase):
         def __init__(self, **kwargs):
             super().__init__(**kwargs)