search

mpg-project / mpg

Sane postgres defaults for clojure.jdbc
MIT License
40 stars 4 forks source link

Clojure.spec tests #23

Open jjl opened 7 years ago

jjl commented 7 years ago

I really like clojure.spec. It would be pretty awesome if we could find a way to use it to provide clearer error messages if you misconfigure things, especially since with the one-way casts we're likely to add quite a few more keys.

We can't actually release it with spec in yet because it requires an alpha clojure 1.9, but when 1.9 is out, I think we should seriously consider requiring it, not least for the generative testing.

Thoughts? @ShaneKilkelly

JuneKelly commented 7 years ago

No objections, once Clojure 1.9 is out :)

EDIT: more than just 'no objections', seems like a good idea, especially if it improves error messages.