Handle deletion of queued jobs when iterating over a Sidekiq::Queue

lib/sidekiq/api.rb

@@ -117,18 +117,23 @@ module Sidekiq
     end
 
     def each(&block)
+      initial_size = size
+      deleted_size = 0
       page = 0
       page_size = 50
 
       loop do
+        range_start = page * page_size - deleted_size
+        range_end   = page * page_size - deleted_size + (page_size - 1)
         entries = Sidekiq.redis do |conn|
-          conn.lrange @rname, page * page_size, (page * page_size) + page_size - 1
+          conn.lrange @rname, range_start, range_end
         end
         break if entries.empty?
         page += 1
         entries.each do |entry|
           block.call Job.new(entry, @name)
         end
+        deleted_size = initial_size - size
       end
     end
 

test/test_api.rb

@@ -202,6 +202,16 @@ class TestApi < Sidekiq::Test
       assert_equal set.size, 0
     end
 
+    it 'can remove jobs when iterating over a queue' do
+      # initial queue size must be greater than Queue#each underlying page size
+      51.times do
+        ApiWorker.perform_async(1, 'aaron')
+      end
+      q = Sidekiq::Queue.new
+      q.map(&:delete)
+      assert_equal q.size, 0
+    end
+
     it 'can find job by id in queues' do
       q = Sidekiq::Queue.new
       job_id = ApiWorker.perform_async(1, 'jason')