BugFix: Update to _to_slug (#101)

Author: FragmentedPacket

plugins/module_utils/netbox_utils.py

@@ -7,6 +7,7 @@
 
 # Import necessary packages
 import traceback
+import re
 from itertools import chain
 from ansible.module_utils.compat import ipaddress
 from ansible.module_utils._text import to_text
@@ -580,11 +581,10 @@ def _to_slug(self, value):
             return value
         elif isinstance(value, int):
             return value
-        elif " " in value:
-            slug = value.replace(" ", "-").lower()
         else:
-            slug = value.lower()
-        return slug
+            removed_chars = re.sub(r"[^\-\.\w\s]", "", value)
+            convert_chars = re.sub(r"[\-\.\s]+", "-", removed_chars)
+            return convert_chars.strip().lower()
 
     def _normalize_data(self, data):
         """

tests/unit/module_utils/test_data/build_query_params_child/data.json

@@ -177,4 +177,4 @@
             "tenant_id": 1
         }
     }
-]
+]

tests/unit/module_utils/test_data/build_query_params_no_child/data.json

@@ -316,4 +316,4 @@
             "cluster_id": 1
         }
     }
-]
+]

tests/unit/module_utils/test_data/slug/data.json

@@ -18,5 +18,9 @@
     {
         "non_slug": null,
         "expected": null
+    },
+    {
+        "non_slug": "test1.example.com",
+        "expected": "test1-example-com"
     }
-]
+]