Skip to content

Commit

Permalink
fix test_int_signal_with_background_thread_in_jruby per review
Browse files Browse the repository at this point in the history
  • Loading branch information
MSP-Greg committed Sep 18, 2019
1 parent 1bd0711 commit d1c74e5
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions test/test_integration_single.rb
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ def test_usr2_restart_restores_environment
end

def test_term_exit_code
skip_on :windows # no TERM
skip_unless_signal_exist? :TERM

cli_server "test/rackup/hello.ru"
_, status = stop_server
Expand All @@ -34,7 +34,7 @@ def test_term_exit_code
end

def test_term_suppress
skip_on :windows # no TERM
skip_unless_signal_exist? :TERM

cli_server "-C test/config/suppress_exception.rb test/rackup/hello.ru"
_, status = stop_server
Expand All @@ -43,7 +43,8 @@ def test_term_suppress
end

def test_term_not_accepts_new_connections
skip_on :jruby, :windows
skip_unless_signal_exist? :TERM
skip_on :jruby

cli_server 'test/rackup/sleep.ru'

Expand All @@ -70,9 +71,7 @@ def test_term_not_accepts_new_connections
@server = nil # prevent `#teardown` from killing already killed server
end

def test_int_signal_with_background_thread_in_jruby
skip_unless :jruby

def test_int_refuse
cli_server 'test/rackup/hello.ru'
begin
sock = TCPSocket.new(HOST, @tcp_port)
Expand Down

0 comments on commit d1c74e5

Please sign in to comment.