fix:boolean parameter without defaullt value defaults to False

[DimitriKwihangana]

Fix the behavior of boolean parameters in the pipeline launcher widget. When a boolean parameter is defined without a default value, userswere getting a validation error ("parameter is required") instead of the parameter defaulting to False.

Changes

• Modified _validate_single() in openhexa/sdk/pipelines/parameter.py
• Added special handling for boolean parameters: when no default is provided, they now default to False instead of raising a validation
  error

How to test

• Create a pipeline with a boolean parameter that has no default value (e.g., @parameter("verbose", type=bool, required=True)).
• Run the pipeline without passing the flag. Previously this would raise a ParameterValueError: verbose is required error. After the fix, the pipeline should run successfully with the parameter defaulting to False.

[yolanfery]

Can't this be moved to

        if normalized_value is None:
            if self.required:
                raise ParameterValueError(f"{self.code} is required")

To avoid duplication of condition check ?

[bramj]

Agreed, also: we can always default booleans that are None to False, regardless of if they're required

[yolanfery]

I propose to add a unit test to cover this edge case

[yolanfery]

(This ensures that we avoid regressions in the future but also it greatly simplifies code reviews, if the test coverage is good and the intent of tests are good we might not need to pull up the code and test)

[bramj]

Agreed

[bramj]

Great! Works perfectly 👍