Re-splat arguments for the have_enqueued_job alias

This makes sure that an extra level of array wrapping isn't added for tests which use the now-deprecated have_enqueued_job, which causes test failures.
wspurgin · Jun 23, 2017 · 7934bba · 7934bba
1 parent 280877e
commit 7934bba
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