Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix the exit status to it properly returns the exit status from your test runs. #16

Closed
wants to merge 1 commit into from

2 participants

@tecnobrat

This is almost the same fix as #12, however that fix did not properly pass the exit status for me... might be a ruby 1.9 thing. This has been tested in ruby 1.8.7 and 1.9

@tecnobrat tecnobrat Fixed the exit status when using the headless gem. Now will properly …
…exit with the same status that your tests exited with.
35238e3
@jits

+1

@tecnobrat

As stated on the other pull request, this one doesn't fix the issue. This bandaided a different issue. Closing it.

@tecnobrat tecnobrat closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 5, 2011
  1. @tecnobrat

    Fixed the exit status when using the headless gem. Now will properly …

    tecnobrat authored
    …exit with the same status that your tests exited with.
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 0 deletions.
  1. +2 −0  lib/headless.rb
View
2  lib/headless.rb
@@ -142,7 +142,9 @@ def hook_at_exit
unless @at_exit_hook_installed
@at_exit_hook_installed = true
at_exit do
+ exit_status = $?.exitstatus if $?.is_a?(Process::Status)
destroy if @destroy_at_exit
+ exit exit_status if exit_status
end
end
end
Something went wrong with that request. Please try again.