General optimations and refactorings in preparation for simpleicon role implementation

tests/unit/filter_plugins/test_get_domain.py

@@ -1,17 +1,8 @@
-# tests/unit/test_get_domain_filter.py
+# tests/unit/test_get_domain.py
 import unittest
 import sys
 import os
-
-# Ensure filter_plugins directory is on the path
-sys.path.insert(
-    0,
-    os.path.abspath(
-        os.path.join(os.path.dirname(__file__), '../../../filter_plugins')
-    )
-)
-
-from get_domain_filter import FilterModule
+from filter_plugins.get_domain import FilterModule
 from ansible.errors import AnsibleFilterError
 
 class TestGetDomainFilter(unittest.TestCase):

tests/unit/filter_plugins/test_get_url.py

@@ -0,0 +1,77 @@
+# tests/unit/filter_plugins/test_get_url.py
+import unittest
+import sys
+import os
+
+# Ensure filter_plugins directory is on the path
+sys.path.insert(
+    0,
+    os.path.abspath(
+        os.path.join(os.path.dirname(__file__), '../../../filter_plugins')
+    )
+)
+
+from get_url import FilterModule
+from ansible.errors import AnsibleFilterError
+
+class TestGetUrlFilter(unittest.TestCase):
+    def setUp(self):
+        # Retrieve the get_url filter function
+        self.get_url = FilterModule().filters()['get_url']
+
+    def test_string_domain(self):
+        domains = {'app': 'example.com'}
+        url = self.get_url(domains, 'app', 'https')
+        self.assertEqual(url, 'https://example.com')
+
+    def test_dict_domain(self):
+        domains = {'app': {'primary': 'primary.com', 'secondary': 'secondary.com'}}
+        url = self.get_url(domains, 'app', 'http')
+        self.assertEqual(url, 'http://primary.com')
+
+    def test_list_domain(self):
+        domains = {'app': ['first.com', 'second.com']}
+        url = self.get_url(domains, 'app', 'ftp')
+        self.assertEqual(url, 'ftp://first.com')
+
+    def test_missing_application_id(self):
+        domains = {'app': 'example.com'}
+        with self.assertRaises(AnsibleFilterError):
+            self.get_url(domains, 'missing', 'https')
+
+    def test_domains_not_dict(self):
+        with self.assertRaises(AnsibleFilterError):
+            self.get_url(['not', 'a', 'dict'], 'app', 'https')
+
+    def test_empty_string_domain(self):
+        domains = {'app': ''}
+        with self.assertRaises(AnsibleFilterError):
+            self.get_url(domains, 'app', 'https')
+
+    def test_empty_dict_domain(self):
+        domains = {'app': {}}
+        with self.assertRaises(AnsibleFilterError):
+            self.get_url(domains, 'app', 'https')
+
+    def test_empty_list_domain(self):
+        domains = {'app': []}
+        with self.assertRaises(AnsibleFilterError):
+            self.get_url(domains, 'app', 'https')
+
+    def test_non_string_in_dict_domain(self):
+        domains = {'app': {'key': 123}}
+        with self.assertRaises(AnsibleFilterError):
+            self.get_url(domains, 'app', 'https')
+
+    def test_non_string_in_list_domain(self):
+        domains = {'app': [123]}
+        with self.assertRaises(AnsibleFilterError):
+            self.get_url(domains, 'app', 'https')
+
+    def test_protocol_not_string(self):
+        domains = {'app': 'example.com'}
+        with self.assertRaises(AnsibleFilterError):
+            self.get_url(domains, 'app', 123)
+
+if __name__ == '__main__':
+    unittest.main()