✨ add test for valid identifier fct. move to utils

Author: enryH

src/vuegen/streamlit_reportview.py

@@ -12,16 +12,6 @@
 from .utils import create_folder, generate_footer, is_url
 
 
-def make_valid_identifier(name: str) -> str:
-    """Create a valid Python identifier from a given name.
-
-    Used in streamlit report sections to build structure (pages).
-    """
-    ret = re.sub(r"[^a-zA-Z0-9]", "_", name)
-    if not ret[0].isalpha():
-        ret = "_" + ret
-    return ret
-
 
 class StreamlitReportView(r.WebAppReportView):
     """

src/vuegen/utils/variables.py

@@ -0,0 +1,12 @@
+import re
+
+
+def make_valid_identifier(name: str) -> str:
+    """Create a valid Python identifier from a given name.
+
+    Used in streamlit report sections to build structure (pages).
+    """
+    ret = re.sub(r"[^a-zA-Z0-9]", "_", name)
+    if not ret[0].isalpha():
+        ret = "_" + ret
+    return ret

tests/utils/test_variables.py

@@ -0,0 +1,16 @@
+import pytest
+
+from vuegen.utils.variables import make_valid_identifier
+
+test_cases = [
+    ("a.non valid name", "a_non_valid_name"),
+    ("1.non valid name", "_1_non_valid_name"),
+    ("a_valid_name", "a_valid_name"),
+    ("test", "test"),
+]
+
+
+@pytest.mark.parametrize("var_name,expected", test_cases)
+def test_make_valid_identifier(var_name, expected):
+    """Test the make_valid_identifier function."""
+    assert make_valid_identifier(var_name) == expected