fix: range-constraint and lenght-constraint work with only one bound

jvalue / jayvee

Jayvee is a domain-specific language and runtime for automated processing of data pipelines

https://jvalue.github.io/jayvee/

150 stars 15 forks source link

fix: range-constraint and lenght-constraint work with only one bound #548

Closed TungstnBallon closed 6 months ago

TungstnBallon commented 7 months ago

It is now possible to only specify one bound when using lenght-constraint. However one bound minimum is still required closes #533

I went ahead and took the liberty to also implement this fix for range-constraint.

TungstnBallon commented 7 months ago

I might be missing something but there are no functional changes in this PR, only tests and hints now? Did you merge the functional changes in a different PR?

It seems to work without functional changes. I think this is because all the required properties like lowerBound or lowerBoundInclusive already have defaults that are used if the property isn't specified in the .jv file. I followed the steps to reproduce from #533 and the error described there is gone too.