Add test Helper::Skipable for skip support in 1.8.x

Author: yaauie

test/fork_safety_test.rb

@@ -5,6 +5,7 @@
 class TestForkSafety < Test::Unit::TestCase
 
   include Helper::Client
+  include Helper::Skipable
 
   driver(:ruby, :hiredis) do
     def test_fork_safety
@@ -31,7 +32,7 @@ def test_fork_safety
 
     rescue NotImplementedError => error
       raise unless error.message =~ /fork is not available/
-      skip(error.message)
+      return skip(error.message)
     end
 
     def test_fork_safety_with_enabled_inherited_socket
@@ -58,7 +59,7 @@ def test_fork_safety_with_enabled_inherited_socket
 
     rescue NotImplementedError => error
       raise unless error.message =~ /fork is not available/
-      skip(error.message)
+      return skip(error.message)
     end
   end
 end

test/helper.rb

@@ -215,4 +215,17 @@ def _new_client(options = {})
       Redis::Distributed.new(NODES, _format_options(options).merge(:driver => ENV["conn"]))
     end
   end
+
+  # Basic support for `skip` in 1.8.x
+  # Note: YOU MUST use `return skip(message)` in order to appropriately bail
+  # from a running test.
+  module Skipable
+    Skipped = Class.new(RuntimeError)
+
+    def skip(message = nil, bt = caller)
+      return super if defined?(super)
+
+      $stderr.puts("SKIPPED: #{self} #{message || 'no reason given'}")
+    end
+  end
 end