Merge pull request resque#750 from ismailasci/requeue_and_remove_index_error

/failed/requeue/all throws exception if has multiple failed jobs

Author: steveklabnik

lib/resque/server.rb

@@ -192,7 +192,7 @@ def show_for_polling(page)
 
     post "/failed/requeue/all" do
       Resque::Failure.count.times do |num|
-        Resque::Failure.requeue_and_remove(num)
+        Resque::Failure.requeue_and_remove(0)
       end
       redirect u('failed')
     end

lib/resque/server/test_helper.rb

@@ -9,6 +9,13 @@ def app
         Resque::Server.new
       end
 
+      def add_failed_jobs
+        Resque::Failure.create(:exception => Exception.new, :worker => Resque::Worker.new(:test), :queue => "queue", :payload => {'class' => 'TestClass'})
+        Resque::Failure.create(:exception => Exception.new, :worker => Resque::Worker.new(:test), :queue => "queue", :payload => {'class' => 'TestClass'})
+        Resque::Failure.create(:exception => Exception.new, :worker => Resque::Worker.new(:test), :queue => "queue", :payload => {'class' => 'TestClass'})
+        Resque::Failure.create(:exception => Exception.new, :worker => Resque::Worker.new(:test), :queue => "queue", :payload => {'class' => 'TestClass'})
+      end
+
       def self.should_respond_with_success
         it "should respond with success" do
           assert last_response.ok?, last_response.errors

test/resque-web_test.rb

@@ -57,3 +57,15 @@
 
   should_respond_with_success
 end
+
+describe "on POST to /failed/requeue/all" do
+  before {
+    add_failed_jobs
+    post "/failed/requeue/all"
+  }
+
+  it "should redirect to /failed and contain '0 jobs'" do
+    follow_redirect!
+    assert last_response.body.include?('<b>0</b> jobs')
+  end
+end