search

typelevel / weaver-test

A test framework that runs everything in parallel.
https://typelevel.org/weaver-test/
Other
48 stars 8 forks source link

Cats instances for Scalacheck `Gen`? #98

Open zainab-ali opened 3 weeks ago

zainab-ali commented 3 weeks ago

This issue was copied over from: https://github.com/disneystreaming/weaver-test/issues/589 It was opened by: armanbilge

Specifically, to help with this: https://github.com/disneystreaming/weaver-test/blob/850a8a6aeb4875d916663092e362eb0bfc24bc99/docs/scalacheck.md?plain=1#L46-L53

Which is a reasonable setup IMO. Personally I'm just missing some of the handy combinators for composition. E.g. it would be nice to express that example using .product or .tupled instead of needing the multi-line for ... yield.

There's options like cats-scalacheck but it would be very convenient to be able to do this out-of-the-box.

zainab-ali commented 3 weeks ago

This comment was copied over from: https://github.com/disneystreaming/weaver-test/issues/589#issuecomment-1264716824 It was written by: Baccata

So you're suggesting inlining the instances in weaver ? I'm not fundamentally opposed to it