From b782c05e1beb311e7299da5f1bb48d7a20794a3f Mon Sep 17 00:00:00 2001
From: Mike Perham <mperham@gmail.com>
Date: Sat, 17 Mar 2012 21:36:33 -0700
Subject: [PATCH] Test cleanup

---
 Rakefile                | 3 ++-
 test/test_extensions.rb | 4 ----
 test/test_retry.rb      | 6 +++++-
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/Rakefile b/Rakefile
index 7a972886..20489929 100644
--- a/Rakefile
+++ b/Rakefile
@@ -1,7 +1,8 @@
 require 'rake/testtask'
 Rake::TestTask.new(:test) do |test|
   test.libs << 'test'
-  test.warning = true
+  #SO MUCH NOISE
+  #test.warning = true
   test.pattern = 'test/**/test_*.rb'
 end
 
diff --git a/test/test_extensions.rb b/test/test_extensions.rb
index 2e4f58f6..01dbbab0 100644
--- a/test/test_extensions.rb
+++ b/test/test_extensions.rb
@@ -28,10 +28,6 @@ class TestExtensions < MiniTest::Unit::TestCase
       assert_equal 1, Sidekiq.redis.llen('queue:default')
     end
 
-    it 'allows delayed exection of ActiveRecord instance methods' do
-      skip('requires a database')
-    end
-
     class UserMailer < ActionMailer::Base
       def greetings(a, b)
         raise "Should not be called!"
diff --git a/test/test_retry.rb b/test/test_retry.rb
index c5c84591..1f65433b 100644
--- a/test/test_retry.rb
+++ b/test/test_retry.rb
@@ -52,13 +52,17 @@ class TestRetry < MiniTest::Unit::TestCase
       @redis = MiniTest::Mock.new
       Sidekiq.instance_variable_set(:@redis, @redis)
 
+      fake_msg = MultiJson.encode({ 'class' => 'Bob', 'args' => [1,2], 'queue' => 'someq' })
+
       def @redis.with; yield self; end
-      @redis.expect :zremrangebyscore, [], ['retry', '-inf', String]
+      @redis.expect :zremrangebyscore, [fake_msg], ['retry', '-inf', String]
+      @redis.expect :rpush, 1, ['someq', fake_msg]
     end
 
     it 'should poll like a bad mother...SHUT YO MOUTH' do
       inst = Sidekiq::Retry::Poller.new
       inst.poll
+      @redis.verify
     end
   end