Unverified Commit bb403767 authored by Marius van den Beek's avatar Marius van den Beek Committed by GitHub
Browse files

Merge pull request #17983 from mvdbeek/fix_LengthValidator 

[24.0] Fix `LengthValidator` if no value passed
parents eb472dbf 92dfa466

lib/galaxy/tools/parameters/validation.py

+3 −1
Original line number Diff line number Diff line
@@ -186,7 +186,9 @@ class LengthValidator(InRangeValidator): 
        super().__init__(message, range_min=length_min, range_max=length_max, negate=negate)



    def validate(self, value, trans=None):

        super().validate(len(value), trans)

        if value is None:

            raise ValueError("No value provided")

        super().validate(len(value) if value else 0, trans)





class DatasetOkValidator(Validator):

test/unit/app/tools/test_parameter_validation.py

+8 −0
Original line number Diff line number Diff line
@@ -186,6 +186,14 @@ class TestParameterValidation(BaseParameterTestCase): 
            p.validate("bar")

        p.validate("f")

        p.validate("foobarbaz")

        p = self._parameter_for(

            xml="""

<param name="blah" type="text" optional="false">

    <validator type="length" min="2" max="8"/>

</param>"""

        )

        with self.assertRaisesRegex(ValueError, "No value provided"):

            p.validate(None)



    def test_InRangeValidator(self):

        p = self._parameter_for(