Merge pull request resque#683 from jeremy/minitest-only

Shed test/unit. Use minitest exclusively.

Author: steveklabnik

test/redis_queue_test.rb

@@ -2,8 +2,6 @@
 require 'resque/queue'
 
 describe "Resque::Queue" do
-  include Test::Unit::Assertions
-
   class Thing
     attr_reader :inside
 
@@ -122,7 +120,7 @@ def == other
     queue1 << x
     queue1.destroy
 
-    assert_raise Resque::QueueDestroyed do
+    assert_raises Resque::QueueDestroyed do
       queue1 << x
     end
   end

test/resque_test.rb

@@ -1,8 +1,6 @@
 require 'test_helper'
 
 describe "Resque" do
-  include Test::Unit::Assertions
-
   before do
     Resque.redis.flushall
 
@@ -115,11 +113,11 @@
 
     assert Resque::Job.create(:jobs, 'SomeMethodJob', 20, '/tmp')
     assert Resque::Job.create(:jobs, 'SomeJob', 20, '/tmp')
-    assert_not_equal Resque.reserve(:jobs), Resque.reserve(:jobs)
+    refute_equal Resque.reserve(:jobs), Resque.reserve(:jobs)
 
     assert Resque::Job.create(:jobs, 'SomeJob', 20, '/tmp')
     assert Resque::Job.create(:jobs, 'SomeJob', 30, '/tmp')
-    assert_not_equal Resque.reserve(:jobs), Resque.reserve(:jobs)
+    refute_equal Resque.reserve(:jobs), Resque.reserve(:jobs)
   end
 
   it "can put jobs on a queue by way of a method" do

test/test_helper.rb

@@ -1,16 +1,14 @@
 require 'rubygems'
 require 'timeout'
 require 'bundler/setup'
-
-$dir = File.dirname(File.expand_path(__FILE__))
-$LOAD_PATH.unshift $dir + '/../lib'
-$TESTING = true
+require 'redis/namespace'
 require 'minitest/unit'
 require 'minitest/spec'
-require 'test/unit'
 
-require 'redis/namespace'
+$dir = File.dirname(File.expand_path(__FILE__))
+$LOAD_PATH.unshift $dir + '/../lib'
 require 'resque'
+$TESTING = true
 
 #
 # make sure we can run redis
@@ -31,11 +29,7 @@
 at_exit do
   next if $!
 
-  if defined?(MiniTest)
-    exit_code = MiniTest::Unit.new.run(ARGV)
-  else
-    exit_code = Test::Unit::AutoRunner.run
-  end
+  exit_code = MiniTest::Unit.new.run(ARGV)
 
   processes = `ps -A -o pid,command | grep [r]edis-test`.split("\n")
   pids = processes.map { |process| process.split(" ")[0] }

test/worker_test.rb

@@ -1,8 +1,6 @@
 require 'test_helper'
 
 describe "Resque::Worker" do
-  include Test::Unit::Assertions
-
   before do
     Resque.redis = Resque.redis # reset state in Resque object
     Resque.redis.flushall
@@ -235,7 +233,7 @@ def self.perform
   end
 
   it "complains if no queues are given" do
-    assert_raise Resque::NoQueueError do
+    assert_raises Resque::NoQueueError do
       Resque::Worker.new
     end
   end
@@ -503,7 +501,7 @@ def self.perform
   it "reconnects to redis after fork" do
     original_connection = Resque.redis.client.connection.instance_variable_get("@sock")
     @worker.work(0)
-    assert_not_equal original_connection, Resque.redis.client.connection.instance_variable_get("@sock")
+    refute_equal original_connection, Resque.redis.client.connection.instance_variable_get("@sock")
   end
 
   it "will call before_pause before it is paused" do
@@ -594,7 +592,7 @@ def self.perform( run_time, rescue_time=nil )
 
             # ensure the worker is started
             start_status = Resque.redis.blpop( 'sigterm-test:start', 5 )
-            assert_not_nil start_status
+            refute_nil start_status
             child_pid = start_status[1].to_i
             assert_operator child_pid, :>, 0
 
@@ -604,7 +602,7 @@ def self.perform( run_time, rescue_time=nil )
 
             # wait to see how it all came down
             result = Resque.redis.blpop( 'sigterm-test:result', 5 )
-            assert_not_nil result
+            refute_nil result
             assert !result[1].start_with?('Finished Normally'), 'Job Finished normally. Sleep not long enough?'
             assert result[1].start_with? 'Caught SignalException', 'Signal exception not raised in child.'