diff options
author | Thomas Grindinger <thomas@linux-ei28.site> | 2016-04-28 16:04:06 -0500 |
---|---|---|
committer | Jeremy Daer <jeremydaer@gmail.com> | 2016-04-29 17:18:28 -0700 |
commit | f3a108619037d7f1e51230a8437af8f23764eca7 (patch) | |
tree | 7622b2d458ce587ee000fd0ed2b589d89dac1bd6 | |
parent | a893d50ce83e616f06ce24fcac8b902edbee3e31 (diff) | |
download | rack-f3a108619037d7f1e51230a8437af8f23764eca7.tar.gz |
Tests: check exitstatus outside the conditional as a workaround
`$?` may be `nil` here, some quirk on 2.4.0-dev. Split it up to fix. Signed-off-by: Jeremy Daer <jeremydaer@gmail.com>
-rw-r--r-- | test/helper.rb | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/test/helper.rb b/test/helper.rb index 7af0f27b..aa9c0e0a 100644 --- a/test/helper.rb +++ b/test/helper.rb @@ -2,7 +2,10 @@ require 'minitest/autorun' module Rack class TestCase < Minitest::Test - if `which lighttpd` && $?.success? + # Check for Lighttpd and launch it for tests if available. + `which lighttpd` + + if $?.success? begin # Keep this first. LIGHTTPD_PID = fork { |