Use Minitest instead of Test::Unit

Gemfile

@@ -5,9 +5,8 @@ gem "hoe-git"
 gem "hoe-ignore"
 gem "rdoc"
 gem "rake-compiler"
-gem "test-unit", "~> 3.0"
 
 gem "rack", "< 3.0"
-gem 'minitest', '~> 5.8'
+gem "minitest", "~> 5.9"
 
 gem "jruby-openssl", :platform => "jruby"

Rakefile

@@ -1,3 +1,4 @@
+require "bundler/setup"
 require "hoe"
 require "rake/extensiontask"
 require "rake/javaextensiontask"

gemfiles/2.1-Gemfile

@@ -5,9 +5,8 @@ gem "hoe-git"
 gem "hoe-ignore"
 gem "rdoc"
 gem "rake-compiler"
-gem "test-unit", "~> 3.0"
 
 gem "rack", "~> 1.6"
-gem 'minitest', '~> 5.8'
+gem "minitest", '~> 5.9'
 
 gem "jruby-openssl", :platform => "jruby"

test/test_app_status.rb

@@ -1,8 +1,9 @@
-require 'test/unit'
-require 'rack'
-require 'puma/app/status'
+require "test_helper"
 
-class TestAppStatus < Test::Unit::TestCase
+require "puma/app/status"
+require "rack"
+
+class TestAppStatus < Minitest::Test
   class FakeServer
     def initialize
       @status = :running

test/test_binder.rb

@@ -1,27 +1,26 @@
-require "rbconfig"
-require 'test/unit'
-require 'testhelp'
+require "test_helper"
 
-require 'puma/binder'
-require 'puma/events'
+require "puma/binder"
+require "puma/events"
+require "puma/puma_http11"
 
-class TestBinder < Test::Unit::TestCase
+class TestBinder < Minitest::Test
 
   def setup
     @events = Puma::Events.new(STDOUT, STDERR)
     @binder = Puma::Binder.new(@events)
   end
 
   def test_localhost_addresses_dont_alter_listeners_for_tcp_addresses
-    omit_on_jruby
+    skip_on_jruby
 
     @binder.parse(["tcp://localhost:10001"], @events)
 
     assert_equal [], @binder.listeners
   end
 
   def test_localhost_addresses_dont_alter_listeners_for_ssl_addresses
-    omit_on_jruby
+    skip_on_jruby
 
     key =  File.expand_path "../../examples/puma/puma_keypair.pem", __FILE__
     cert = File.expand_path "../../examples/puma/cert_puma.pem", __FILE__

test/test_cli.rb

@@ -1,10 +1,8 @@
-require "rbconfig"
-require 'test/unit'
-require 'puma/cli'
-require 'tempfile'
-require 'yaml'
+require "test_helper"
 
-class TestCLI < Test::Unit::TestCase
+require "puma/cli"
+
+class TestCLI < Minitest::Test
   def setup
     @environment = 'production'
     @tmp_file = Tempfile.new("puma-test")
@@ -168,10 +166,7 @@ def test_state_file_callback_filtering
                           "--state", @tmp_path ]
     cli.launcher.write_state
 
-    data = nil
-    assert_nothing_raised do
-      data = YAML.load_file( @tmp_path )
-    end
+    data = YAML.load_file(@tmp_path)
 
     keys_not_stripped = data.keys & Puma::CLI::KEYS_NOT_TO_PERSIST_IN_STATE
     assert_empty keys_not_stripped

test/test_config.rb

@@ -1,9 +1,8 @@
-require 'test/unit'
+require "test_helper"
 
-require 'puma'
-require 'puma/configuration'
+require "puma/configuration"
 
-class TestConfigFile < Test::Unit::TestCase
+class TestConfigFile < Minitest::Test
   def test_app_from_rackup
     conf = Puma::Configuration.new do |c|
       c.rackup "test/hello-bind.ru"

test/testhelp.rb → test/test_helper.rb

@@ -1,20 +1,17 @@
 # Copyright (c) 2011 Evan Phoenix
 # Copyright (c) 2005 Zed A. Shaw
 
-require 'rubygems'
-require 'test/unit'
-require 'net/http'
-require 'digest/sha1'
-require 'uri'
-require 'stringio'
-
-require 'puma'
-require 'puma/detect'
+require "bundler/setup"
+require "minitest/autorun"
+require "minitest/pride"
+require "puma"
+require "puma/detect"
 
 # Either takes a string to do a get request against, or a tuple of [URI, HTTP] where
 # HTTP is some kind of Net::HTTP request object (POST, HEAD, etc.)
 def hit(uris)
   results = []
+
   uris.each do |u|
     res = nil
 
@@ -33,19 +30,21 @@ def hit(uris)
 end
 
 module TimeoutEveryTestCase
-  def run(*args)
+  def run(*)
     if !!ENV['CI']
       Timeout.timeout(60) { super }
     else
       super
     end
   end
 end
-Test::Unit::TestCase.prepend TimeoutEveryTestCase
 
-module OmitTestsBasedOnRubyEngine
-  def omit_on_jruby
-    omit "Omitted on JRuby" if Puma.jruby?
+Minitest::Test.prepend TimeoutEveryTestCase
+
+module SkipTestsBasedOnRubyEngine
+  def skip_on_jruby
+    skip "Skipped on JRuby" if Puma.jruby?
   end
 end
-Test::Unit::TestCase.prepend OmitTestsBasedOnRubyEngine
+
+Minitest::Test.include SkipTestsBasedOnRubyEngine

test/test_http10.rb

@@ -1,10 +1,10 @@
-require 'testhelp'
+require "test_helper"
 
-class Http10ParserTest < Test::Unit::TestCase
-  include Puma
+require "puma/puma_http11"
 
+class Http10ParserTest < Minitest::Test
   def test_parse_simple
-    parser = HttpParser.new
+    parser = Puma::HttpParser.new
     req = {}
     http = "GET / HTTP/1.0\r\n\r\n"
     nread = parser.execute(req, http, 0)
@@ -17,10 +17,10 @@ def test_parse_simple
     assert_equal '/', req['REQUEST_PATH']
     assert_equal 'HTTP/1.0', req['HTTP_VERSION']
     assert_equal '/', req['REQUEST_URI']
-    assert_equal 'GET', req['REQUEST_METHOD']    
+    assert_equal 'GET', req['REQUEST_METHOD']
     assert_nil req['FRAGMENT']
     assert_nil req['QUERY_STRING']
-    
+
     parser.reset
     assert parser.nread == 0, "Number read after reset should be 0"
   end

test/test_http11.rb

@@ -1,14 +1,14 @@
 # Copyright (c) 2011 Evan Phoenix
 # Copyright (c) 2005 Zed A. Shaw
 
-require 'testhelp'
+require "test_helper"
 
-include Puma
+require "puma/puma_http11"
 
-class Http11ParserTest < Test::Unit::TestCase
+class Http11ParserTest < Minitest::Test
 
   def test_parse_simple
-    parser = HttpParser.new
+    parser = Puma::HttpParser.new
     req = {}
     http = "GET /?a=1 HTTP/1.1\r\n\r\n"
     nread = parser.execute(req, http, 0)
@@ -30,7 +30,7 @@ def test_parse_simple
   end
 
   def test_parse_escaping_in_query
-    parser = HttpParser.new
+    parser = Puma::HttpParser.new
     req = {}
     http = "GET /admin/users?search=%27%%27 HTTP/1.1\r\n\r\n"
     nread = parser.execute(req, http, 0)
@@ -48,7 +48,7 @@ def test_parse_escaping_in_query
   end
 
   def test_parse_absolute_uri
-    parser = HttpParser.new
+    parser = Puma::HttpParser.new
     req = {}
     http = "GET http://192.168.1.96:3000/api/v1/matches/test?1=1 HTTP/1.1\r\n\r\n"
     nread = parser.execute(req, http, 0)
@@ -72,7 +72,7 @@ def test_parse_absolute_uri
   end
 
   def test_parse_dumbfuck_headers
-    parser = HttpParser.new
+    parser = Puma::HttpParser.new
     req = {}
     should_be_good = "GET / HTTP/1.1\r\naaaaaaaaaaaaa:++++++++++\r\n\r\n"
     nread = parser.execute(req, should_be_good, 0)
@@ -82,7 +82,7 @@ def test_parse_dumbfuck_headers
   end
 
   def test_parse_error
-    parser = HttpParser.new
+    parser = Puma::HttpParser.new
     req = {}
     bad_http = "GET / SsUTF/1.1"
 
@@ -99,12 +99,12 @@ def test_parse_error
   end
 
   def test_fragment_in_uri
-    parser = HttpParser.new
+    parser = Puma::HttpParser.new
     req = {}
     get = "GET /forums/1/topics/2375?page=1#posts-17408 HTTP/1.1\r\n\r\n"
-    assert_nothing_raised do
-      parser.execute(req, get, 0)
-    end
+
+    parser.execute(req, get, 0)
+
     assert parser.finished?
     assert_equal '/forums/1/topics/2375?page=1', req['REQUEST_URI']
     assert_equal 'posts-17408', req['FRAGMENT']
@@ -125,7 +125,7 @@ def rand_data(min, max, readable=true)
   end
 
   def test_max_uri_path_length
-    parser = HttpParser.new
+    parser = Puma::HttpParser.new
     req = {}
 
     # Support URI path length to a max of 2048
@@ -145,7 +145,7 @@ def test_max_uri_path_length
   end
 
   def test_horrible_queries
-    parser = HttpParser.new
+    parser = Puma::HttpParser.new
 
     # then that large header names are caught
     10.times do |c|

test/test_integration.rb

@@ -1,17 +1,11 @@
-require "rbconfig"
-require 'test/unit'
-require 'socket'
-require 'timeout'
-require 'net/http'
-require 'tempfile'
+require "test_helper"
 
-require 'puma/cli'
-require 'puma/control_cli'
-require 'puma/detect'
+require "puma/cli"
+require "puma/control_cli"
 
 # These don't run on travis because they're too fragile
 
-class TestIntegration < Test::Unit::TestCase
+class TestIntegration < Minitest::Test
   def setup
     @state_path = "test/test_puma.state"
     @bind_path = "test/test_server.sock"
@@ -116,7 +110,8 @@ def test_stop_via_pumactl
   end
 
   def test_phased_restart_via_pumactl
-    omit("Too finicky, fails 50% of the time on CI.")
+    skip("Too finicky, fails 50% of the time on CI.")
+
     if Puma.jruby? || Puma.windows?
       assert true
       return

test/test_iobuffer.rb

@@ -1,7 +1,8 @@
-require 'puma/io_buffer'
-require 'test/unit'
+require "test_helper"
 
-class TestIOBuffer < Test::Unit::TestCase
+require "puma/io_buffer"
+
+class TestIOBuffer < Minitest::Test
   attr_accessor :iobuf
   def setup
     self.iobuf = Puma::IOBuffer.new
@@ -13,7 +14,7 @@ def test_initial_size
   end
 
   def test_append_op
-    iobuf << "abc" 
+    iobuf << "abc"
     assert_equal "abc", iobuf.to_s
     iobuf << "123"
     assert_equal "abc123", iobuf.to_s

test/test_minissl.rb

@@ -1,28 +1,28 @@
-require 'test/unit'
-require 'puma'
-require 'puma/minissl'
+require "test_helper"
 
-class TestMiniSSL < Test::Unit::TestCase
+require "puma/minissl"
+
+class TestMiniSSL < Minitest::Test
 
   if defined?(JRUBY_VERSION)
     def test_raises_with_invalid_keystore_file
       ctx = Puma::MiniSSL::Context.new
 
-      exception = assert_raise(ArgumentError) { ctx.keystore = "/no/such/keystore" }
+      exception = assert_raises(ArgumentError) { ctx.keystore = "/no/such/keystore" }
       assert_equal("No such keystore file '/no/such/keystore'", exception.message)
     end
   else
     def test_raises_with_invalid_key_file
       ctx = Puma::MiniSSL::Context.new
 
-      exception = assert_raise(ArgumentError) { ctx.key = "/no/such/key" }
+      exception = assert_raises(ArgumentError) { ctx.key = "/no/such/key" }
       assert_equal("No such key file '/no/such/key'", exception.message)
     end
 
     def test_raises_with_invalid_cert_file
       ctx = Puma::MiniSSL::Context.new
 
-      exception = assert_raise(ArgumentError) { ctx.cert = "/no/such/cert" }
+      exception = assert_raises(ArgumentError) { ctx.cert = "/no/such/cert" }
       assert_equal("No such cert file '/no/such/cert'", exception.message)
     end
   end

test/test_null_io.rb

@@ -1,7 +1,8 @@
-require 'puma/null_io'
-require 'test/unit'
+require "test_helper"
 
-class TestNullIO < Test::Unit::TestCase
+require "puma/null_io"
+
+class TestNullIO < Minitest::Test
   attr_accessor :nio
   def setup
     self.nio = Puma::NullIO.new