expound-printer

[olivergeorge]

It'd be nice to be able to bind expound-str to s/*explain-out*. With that stest/instrument should present expound formatted descriptions in exceptions.

(binding [s/*explain-out* expound/expound-printer]
    (s/explain-str (s/coll-of integer?) {:A 1}))

The main difference to the current code is that *explain-out* printers take explain-data as an arg.

[bhb]

@olivergeorge Agreed! I've got a local branch where I'm working on this now.

[bhb]

@olivergeorge Do you mind trying this feature before I release?

To try it:

1. git clone git@github.com:bhb/expound.git
2. cd expound
3. git checkout expound-printer
4. lein install
5. In client project, change dep to [expound "0.1.2"]

[bhb]

The output during instrumentation could be optimized for that use case, but I'll leave that for a further PR.

[bhb]

Thanks for reporting this!

[olivergeorge]

We just tested it in on our code (cljs preload) and it's working like a charm.

[bhb]

@olivergeorge I'm glad to hear that. At some point I'm going to do more work on presenting the failure in a better way (including the source code and line numbers).