Tests: Change ListParamType to check value type instead of setting an…

… attribute when parsing an argument twice

click_params/base.py

@@ -115,7 +115,7 @@ def convert(self, value, param, ctx):
         # For an unknown reason, when a user is prompted for a value using "prompt" argument from click.option
         # the convert method seems to be called more than once, so this is necessary to avoid an error
         # when calling self._strip_separator below since the value passed will be already converted to a list
-        if self._convert_called:
+        if isinstance(value, list):
             return value
 
         if self._ignore_empty and value == "":
@@ -125,7 +125,6 @@ def convert(self, value, param, ctx):
         if errors:
             self.fail(self._error_message.format(errors=errors), param, ctx)
 
-        self._convert_called = True
         return converted_list
 
     def __repr__(self):

tests/test_base.py

@@ -189,7 +189,6 @@ def test_should_return_correct_items_when_giving_correct_expression(self, expres
         # this is to test the scenario when a user is prompted for a value, it seems that
         # the convert method is called more than once
         # after the first call, all subsequent calls will have the converted value as first argument
-        assert base_list._convert_called
         assert values == base_list.convert(values, None, None)
 
     @pytest.mark.parametrize('param_type', [click.INT, click.FLOAT, click.STRING, FRACTION, COMPLEX])

tests/test_miscellaneous.py

@@ -166,6 +166,21 @@ def test_miscellaneous_list_param_types_ignore_empty_string(param_type):
     assert misc_list_type.convert("", None, None) == []
 
 
+def test_cli_with_multiple_similar_string_list_param_types(runner):
+    @click.command()
+    @click.option('-v', 'values', type=StringListParamType(","))
+    def cli(values):
+        click.echo(values)
+
+    result = runner.invoke(cli, ['-v', "abc,def"])
+
+    assert result.output == "['abc', 'def']\n"
+
+    result = runner.invoke(cli, ['-v', "abc,def"])
+    # TODO: the following will currently fail!
+    assert result.output == "['abc', 'def']\n"
+
+
 class TestJsonParamType:
     """Tests JsonParamType specific cases"""