Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Currently returns 0 to OS even if tests fail. #3

Merged
merged 1 commit into from

2 participants

@haberman

Hi there, I was using lunitx but noticed that it returns 0 to the OS even if tests fail. Enclosed is a simple change that returns 1 if any tests fail.

It looks like there's an error handler that does this already if the xpcall() fails. I'm not sure what situations would cause this to happen so I didn't touch that branch.

Cheers,
Josh

@dcurrie dcurrie merged commit 586eebb into dcurrie:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Apr 8, 2012
  1. @haberman
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 0 deletions.
  1. +3 −0  lunitx.lua
View
3  lunitx.lua
@@ -13,6 +13,9 @@ atexit(function()
print(emsg)
os.exit(1)
end
+ if lunit.stats.failed > 0 then
+ os.exit(1)
+ end
end)
return lunit
Something went wrong with that request. Please try again.