Merge pull request #136 from aprescott/have-enqueued-splat-args

Re-splat arguments for the have_enqueued_job alias
wspurgin · Jun 23, 2017 · 8843be9 · 8843be9
2 parents 280877e + 7934bba
commit 8843be9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/rspec/sidekiq/matchers/have_enqueued_job.rb b/lib/rspec/sidekiq/matchers/have_enqueued_job.rb
@@ -7,7 +7,7 @@ def have_enqueued_sidekiq_job(*expected_arguments)
 
       def have_enqueued_job(*expected_arguments)
         warn "[DEPRECATION] `have_enqueued_job` is deprecated.  Please use `have_enqueued_sidekiq_job` instead."
-        have_enqueued_sidekiq_job(expected_arguments)
+        have_enqueued_sidekiq_job(*expected_arguments)
       end
 
       class JobOptionParser

diff --git a/spec/rspec/sidekiq/matchers/have_enqueued_job_spec.rb b/spec/rspec/sidekiq/matchers/have_enqueued_job_spec.rb
@@ -97,6 +97,10 @@
     it 'provides deprecation warning' do
       expect { have_enqueued_job }.to output(/[DEPRECATION]/).to_stderr
     end
+
+    it 'matches the same way have_enqueued_sidekiq_job does' do
+      expect(worker).to have_enqueued_job *worker_args
+    end
   end
 
   describe '#description' do