Skip to content
This repository has been archived by the owner on Oct 9, 2019. It is now read-only.

Add Fuzz.set, Fuzz.dict #188

Open
avh4 opened this issue Jul 6, 2017 · 1 comment
Open

Add Fuzz.set, Fuzz.dict #188

avh4 opened this issue Jul 6, 2017 · 1 comment

Comments

@avh4
Copy link
Member

avh4 commented Jul 6, 2017

Proposing we add

Fuzz.set : Fuzzer a -> Fuzzer (Set a)
Fuzz.dict : Fuzzer comparable -> Fuzzer value -> Fuzzer (Dict comparable value)

You can currently use Fuzz.list a |> Fuzz.map Set.fromList (and similarly for Dict), but shrinking is extremely inefficient when the number of possible values a can produce is small (in my case, each test evaluation takes more than 5 seconds, so evaluating shrinks that are the same as previous shrinks adds up quickly).

Edit: maybe the inefficient shrinking is simply a result of #187 ?

@mgold
Copy link
Member

mgold commented Jul 7, 2017

If the only benefit of this is shrinking performance, and we might completely overhaul how shrinking works, let's wait on this one.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants