I suppose it depends whether you think the edge cases are the minimum required or whether you should include at least one "random" value too.
k
kopper
12/22/2020, 10:40 AM
sure, but right now it is used to check if the number of iterations is >= minIterations
I'd argue if someone wants to test only edge cases they will use Exhaustive instead of an Arb
s
sam
12/22/2020, 10:41 AM
I agree.
k
kopper
12/22/2020, 10:44 AM
Awesome, if I simply raise a PR that adds +1 to
this.edgecases().size
it will be a breaking change. How to proceed in such a case?
s
sam
12/22/2020, 11:44 AM
I have merged in code to fix your other issue - I think the "calculate min iterations" code should do this (the +1) but only if it's enabled by a flag in
PropertyTesting
so it's not a breaking change. In the future we can perhaps make this the default.