Fix bug when job key is a symbol and sidekiq runs twice #108

matu1104 · 2016-05-04T23:09:12Z

This change is related to #102

coveralls · 2016-05-04T23:17:54Z

Coverage increased (+0.04%) to 95.694% when pulling a8ba612 on matu1104:master into 27c54fa on moove-it:master.

snmgian · 2016-05-06T04:09:56Z

spec/sidekiq-scheduler/schedule_spec.rb

+      let(:job_id) { :super_job }
+
+      context 'when schedule is set twice' do
+        it 'sets the schedule on redis' do


This test does not fail when running the specs with seed = 9838. Please flush Redis on a before hook (see spec/sidekiq/scheduler_spec.rb)

Thanks for the comment. I just pushed a new change.

coveralls · 2016-05-11T00:29:14Z

Coverage increased (+0.2%) to 95.839% when pulling 8830829 on matu1104:master into 27c54fa on moove-it:master.

snmgian reviewed May 6, 2016
View reviewed changes

Fix bug when job key is a symbol and sidekiq runs twice

8830829

snmgian merged commit 8043b72 into sidekiq-scheduler:master May 11, 2016

Provide feedback