fix: update is_valid_string function and its tests

- Modify is_valid_string to accept allowed_characters param
- Update tests in test_is_valid_string.py to reflect changes
- Add new test cases for additional allowed characters

test/unit/test_is_valid_string.py

@@ -2,9 +2,11 @@
 
 
 def test_alphanumeric():
-    assert is_valid_string("abc123") == True
-    assert is_valid_string("abc_123") == True
-    assert is_valid_string("abc-123") == True
-    assert is_valid_string("abc@123") == False
-    assert is_valid_string("___---") == False
-    assert is_valid_string("_") == False
+    assert is_valid_string("abc123") is True
+    assert is_valid_string("abc_123") is True
+    assert is_valid_string("abc-123") is True
+    assert is_valid_string("abc@123") is False
+    assert is_valid_string("___---") is False
+    assert is_valid_string("_") is False
+    assert is_valid_string("a,b,c") is True
+    assert is_valid_string("a,b, c") is False

tracardi/process_engine/action/v1/interest/utils.py

@@ -1,4 +1,4 @@
-def is_valid_string(variable):
+def is_valid_string(variable, allowed_characters=None):
     """
     Check if the variable is a string composed of alphanumeric characters, underscores (_),
     hyphens (-), commas (,), and colons (:), contains no spaces, and has at least one alphanumeric character.
@@ -17,5 +17,6 @@ def is_valid_string(variable):
         return False
 
     # Checking if all characters are alphanumeric, underscore, hyphen, comma, or colon, and if there are no spaces
-    allowed_characters = ['_', '-', ',']
+    if allowed_characters is None:
+        allowed_characters = ['_', '-', ',']
     return all(char.isalnum() or char in allowed_characters for char in variable) and ' ' not in variable