raise TypeError if input is not list of str for convert_list_to_comma_separated_string converter (#2154)

* raise typeerror if input is not list of str

* allow tuple iterable along with list

Author: vnktshr21

build/templates/_converters.py.mako

@@ -339,14 +339,16 @@ def convert_comma_separated_string_to_list(comma_separated_string):
 
 
 def convert_list_to_comma_separated_string(list_of_strings):
-    '''Convert a list of strings into a comma-separated string.
+    '''Convert a list or tuple of strings into a comma-separated string.
 
     Args:
-        list_of_strings (list[str]): List of strings.
+        list_of_strings (list or tuple of str): List or tuple of strings.
 
     Returns:
         str: Comma-separated string.
     '''
+    if not isinstance(list_of_strings, (list, tuple)) or not all(isinstance(item, str) for item in list_of_strings):
+        raise TypeError('Input must be a list or tuple of str')
     return ','.join(list_of_strings)
 
 

generated/nidcpower/nidcpower/_converters.py

@@ -330,14 +330,16 @@ def convert_comma_separated_string_to_list(comma_separated_string):
 
 
 def convert_list_to_comma_separated_string(list_of_strings):
-    '''Convert a list of strings into a comma-separated string.
+    '''Convert a list or tuple of strings into a comma-separated string.
 
     Args:
-        list_of_strings (list[str]): List of strings.
+        list_of_strings (list or tuple of str): List or tuple of strings.
 
     Returns:
         str: Comma-separated string.
     '''
+    if not isinstance(list_of_strings, (list, tuple)) or not all(isinstance(item, str) for item in list_of_strings):
+        raise TypeError('Input must be a list or tuple of str')
     return ','.join(list_of_strings)
 
 

generated/nidigital/nidigital/_converters.py

@@ -329,14 +329,16 @@ def convert_comma_separated_string_to_list(comma_separated_string):
 
 
 def convert_list_to_comma_separated_string(list_of_strings):
-    '''Convert a list of strings into a comma-separated string.
+    '''Convert a list or tuple of strings into a comma-separated string.
 
     Args:
-        list_of_strings (list[str]): List of strings.
+        list_of_strings (list or tuple of str): List or tuple of strings.
 
     Returns:
         str: Comma-separated string.
     '''
+    if not isinstance(list_of_strings, (list, tuple)) or not all(isinstance(item, str) for item in list_of_strings):
+        raise TypeError('Input must be a list or tuple of str')
     return ','.join(list_of_strings)
 
 

generated/nidmm/nidmm/_converters.py

@@ -329,14 +329,16 @@ def convert_comma_separated_string_to_list(comma_separated_string):
 
 
 def convert_list_to_comma_separated_string(list_of_strings):
-    '''Convert a list of strings into a comma-separated string.
+    '''Convert a list or tuple of strings into a comma-separated string.
 
     Args:
-        list_of_strings (list[str]): List of strings.
+        list_of_strings (list or tuple of str): List or tuple of strings.
 
     Returns:
         str: Comma-separated string.
     '''
+    if not isinstance(list_of_strings, (list, tuple)) or not all(isinstance(item, str) for item in list_of_strings):
+        raise TypeError('Input must be a list or tuple of str')
     return ','.join(list_of_strings)
 
 

generated/nifake/nifake/_converters.py

@@ -330,14 +330,16 @@ def convert_comma_separated_string_to_list(comma_separated_string):
 
 
 def convert_list_to_comma_separated_string(list_of_strings):
-    '''Convert a list of strings into a comma-separated string.
+    '''Convert a list or tuple of strings into a comma-separated string.
 
     Args:
-        list_of_strings (list[str]): List of strings.
+        list_of_strings (list or tuple of str): List or tuple of strings.
 
     Returns:
         str: Comma-separated string.
     '''
+    if not isinstance(list_of_strings, (list, tuple)) or not all(isinstance(item, str) for item in list_of_strings):
+        raise TypeError('Input must be a list or tuple of str')
     return ','.join(list_of_strings)
 
 

generated/nifake/nifake/unit_tests/test_converters.py

@@ -413,3 +413,9 @@ def test_convert_comma_separated_string_to_list():
 def test_convert_list_to_comma_separated_string():
     out_string = _converters.convert_list_to_comma_separated_string(['PinA', 'PinB', 'PinC'])
     assert out_string == 'PinA,PinB,PinC'
+
+
+def test_convert_list_to_comma_separated_string_invalid_input():
+    with pytest.raises(TypeError) as error_info:
+        _converters.convert_list_to_comma_separated_string('PinA,PinB,PinC')
+    assert str(error_info.value) == 'Input must be a list or tuple of str'

generated/nifgen/nifgen/_converters.py

@@ -329,14 +329,16 @@ def convert_comma_separated_string_to_list(comma_separated_string):
 
 
 def convert_list_to_comma_separated_string(list_of_strings):
-    '''Convert a list of strings into a comma-separated string.
+    '''Convert a list or tuple of strings into a comma-separated string.
 
     Args:
-        list_of_strings (list[str]): List of strings.
+        list_of_strings (list or tuple of str): List or tuple of strings.
 
     Returns:
         str: Comma-separated string.
     '''
+    if not isinstance(list_of_strings, (list, tuple)) or not all(isinstance(item, str) for item in list_of_strings):
+        raise TypeError('Input must be a list or tuple of str')
     return ','.join(list_of_strings)
 
 

generated/nimodinst/nimodinst/_converters.py

@@ -329,14 +329,16 @@ def convert_comma_separated_string_to_list(comma_separated_string):
 
 
 def convert_list_to_comma_separated_string(list_of_strings):
-    '''Convert a list of strings into a comma-separated string.
+    '''Convert a list or tuple of strings into a comma-separated string.
 
     Args:
-        list_of_strings (list[str]): List of strings.
+        list_of_strings (list or tuple of str): List or tuple of strings.
 
     Returns:
         str: Comma-separated string.
     '''
+    if not isinstance(list_of_strings, (list, tuple)) or not all(isinstance(item, str) for item in list_of_strings):
+        raise TypeError('Input must be a list or tuple of str')
     return ','.join(list_of_strings)
 
 

generated/nirfsg/nirfsg/_converters.py

@@ -329,14 +329,16 @@ def convert_comma_separated_string_to_list(comma_separated_string):
 
 
 def convert_list_to_comma_separated_string(list_of_strings):
-    '''Convert a list of strings into a comma-separated string.
+    '''Convert a list or tuple of strings into a comma-separated string.
 
     Args:
-        list_of_strings (list[str]): List of strings.
+        list_of_strings (list or tuple of str): List or tuple of strings.
 
     Returns:
         str: Comma-separated string.
     '''
+    if not isinstance(list_of_strings, (list, tuple)) or not all(isinstance(item, str) for item in list_of_strings):
+        raise TypeError('Input must be a list or tuple of str')
     return ','.join(list_of_strings)
 
 

generated/niscope/niscope/_converters.py

@@ -329,14 +329,16 @@ def convert_comma_separated_string_to_list(comma_separated_string):
 
 
 def convert_list_to_comma_separated_string(list_of_strings):
-    '''Convert a list of strings into a comma-separated string.
+    '''Convert a list or tuple of strings into a comma-separated string.
 
     Args:
-        list_of_strings (list[str]): List of strings.
+        list_of_strings (list or tuple of str): List or tuple of strings.
 
     Returns:
         str: Comma-separated string.
     '''
+    if not isinstance(list_of_strings, (list, tuple)) or not all(isinstance(item, str) for item in list_of_strings):
+        raise TypeError('Input must be a list or tuple of str')
     return ','.join(list_of_strings)