marick/lein-midje

lein midje exits with status 0 on FAILURE

ccfontes opened this issue · 2 comments

ccfontes commented

I call lein midje from a pre-commit hook, and rely on the command exit status to proceed or abandon commit. Yet, when all tests succeed but there is some kind of unrelated failure like this one: FAILURE: These files have broken namespaces and will not be loaded., lein midje will exit with 0 status. I think this is undesirable.
screen shot 2014-03-01 at 22 59 22

marick commented

I agree. As far as I can tell from the code, it should be returning a non-zero exit status. Therefore, I can't promise a quick fix.