marick / Midje

Midje provides a migration path from clojure.test to a more flexible, readable, abstract, and gracious style of testing
MIT License
1.69k stars 128 forks source link

Improving control of colorize #459

Closed lverns closed 5 years ago

lverns commented 5 years ago

Fixes #458.

While the internal representation of the :colorize config key has changed from a string to a keyword, this should be fully backwards compatible with whatever users have in MIDJE_COLORIZE or .midje.clj.

lverns commented 5 years ago

@philomates I haven't seen that arity error before, but I also haven't tried the autotest functionality.

I'll look into it.

lverns commented 5 years ago

@philomates I was able to reproduce the issue and fix it. lein midje and lein midje :autotest now both work correctly with true, reverse, and false for MIDJE_COLORIZE.

philomates commented 5 years ago

I've released this under Midje 1.9.7 and updated the docs a little: https://github.com/marick/Midje/wiki/Configuration-files and https://github.com/marick/Midje/wiki/Colorizing