Reduce collection generator default size from 20

Question

In general I find that it is very easy (especially with nested or recursive collections) to have a check run OOME due to generating very large nested collections. Currently the default is 20 - I think we should change it to 3.

The attached patch just changes the default from 20 to 3. An alternate approach would be to change it to a dynvar setting.

Patch: clj-2102-3.patch

Answer 1

Comment made by: alexmiller

Updated patch to apply to master

Answer 2

Comment made by: alexmiller

Updated patch to apply to spec.alpha

Answer 3

Comment made by: stu

I have definitely seen the pain here -- nested collections can get big fast. OTOH for non-nested collections the larger generator is nice. Not sure moving the default is a help.

Answer 4

Comment made by: alexmiller

Use dynamic var and reduce default. Also consider ways to avoid this kind of problem in test.check itself (how does quickcheck deal with this?).

Answer 5

Reference: https://clojure.atlassian.net/browse/CLJ-2102 (reported by alexmiller)