2014-12-30 15:54:58 -05:00
|
|
|
require_relative 'helper'
|
2012-05-25 23:21:42 -04:00
|
|
|
require 'sidekiq/scheduled'
|
|
|
|
|
2013-09-22 17:38:33 -04:00
|
|
|
class TestScheduling < Sidekiq::Test
|
2012-05-25 23:21:42 -04:00
|
|
|
describe 'middleware' do
|
|
|
|
before do
|
2014-03-24 13:20:16 -04:00
|
|
|
Sidekiq::Client.instance_variable_set(:@default, nil)
|
2013-05-12 17:25:30 -04:00
|
|
|
@redis = Minitest::Mock.new
|
2012-05-25 23:21:42 -04:00
|
|
|
# Ugh, this is terrible.
|
|
|
|
Sidekiq.instance_variable_set(:@redis, @redis)
|
2013-09-29 03:58:16 -04:00
|
|
|
def @redis.multi; [yield] * 2 if block_given?; end
|
2012-05-25 23:21:42 -04:00
|
|
|
def @redis.with; yield self; end
|
|
|
|
end
|
|
|
|
|
2014-03-24 13:20:16 -04:00
|
|
|
after do
|
|
|
|
Sidekiq::Client.instance_variable_set(:@default, nil)
|
2014-05-13 17:00:20 -04:00
|
|
|
Sidekiq.instance_variable_set(:@redis, REDIS)
|
2014-03-24 13:20:16 -04:00
|
|
|
end
|
|
|
|
|
2012-05-25 23:21:42 -04:00
|
|
|
class ScheduledWorker
|
|
|
|
include Sidekiq::Worker
|
2013-09-29 03:58:16 -04:00
|
|
|
sidekiq_options :queue => :custom_queue
|
2012-05-25 23:21:42 -04:00
|
|
|
def perform(x)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'schedules a job via interval' do
|
2013-03-24 20:42:43 -04:00
|
|
|
@redis.expect :zadd, true, ['schedule', Array]
|
2012-08-11 14:47:25 -04:00
|
|
|
assert ScheduledWorker.perform_in(600, 'mike')
|
2012-05-25 23:21:42 -04:00
|
|
|
@redis.verify
|
|
|
|
end
|
|
|
|
|
MyWorker.perform_in(1.month) does not always schedule job in one month.
At the moment, `MyWorker.perform_in(1.month)` always schedules a job in 30
days. When added to a date, `1.month` will add 1 to the date's month count,
which means that it will add 28, 29, 30, or 31 days depending on the month and
year.
When I call `MyWorker.perform_in(1.month)`, I would expect the job to be
scheduled next month, same day of the month, all the time. At the moment, it is
true only four months in the year.
My pull request tries to fix this by converting the interval object to a Float
at the last possible moment.
Plaese note that the test I wrote will fail only during months that do not have
30 days. Ideally, I would add a dependency to Timecop and freeze time to any
day in a month of 28, 29 or 31 days. This could also avoid using
`#assert_in_delta`, in favour of `#assert_equal`.
Feel free to read my blog post [Rails' `1.month` has a variable
length](http://dstosik.github.io/rails/2015/02/19/rails-1month-variable-length/)
for more details.
2015-02-19 19:25:46 -05:00
|
|
|
it 'schedules a job in one month' do
|
|
|
|
@redis.expect :zadd, true do |key, args|
|
|
|
|
assert_equal 'schedule', key
|
|
|
|
assert_in_delta 1.month.since.to_f, args[0][0].to_f, 1
|
|
|
|
end
|
|
|
|
assert ScheduledWorker.perform_in(1.month, 'mike')
|
|
|
|
@redis.verify
|
|
|
|
end
|
|
|
|
|
2012-05-25 23:21:42 -04:00
|
|
|
it 'schedules a job via timestamp' do
|
2013-03-24 20:42:43 -04:00
|
|
|
@redis.expect :zadd, true, ['schedule', Array]
|
2012-08-11 14:47:25 -04:00
|
|
|
assert ScheduledWorker.perform_in(5.days.from_now, 'mike')
|
2012-05-25 23:21:42 -04:00
|
|
|
@redis.verify
|
|
|
|
end
|
2012-11-20 00:31:14 -05:00
|
|
|
|
2013-09-29 03:58:16 -04:00
|
|
|
it 'schedules job right away on negative timestamp/interval' do
|
2013-11-14 22:12:27 -05:00
|
|
|
@redis.expect :sadd, true, ['queues', 'custom_queue']
|
2013-09-29 03:58:16 -04:00
|
|
|
@redis.expect :lpush, true, ['queue:custom_queue', Array]
|
|
|
|
assert ScheduledWorker.perform_in(-300, 'mike')
|
|
|
|
@redis.verify
|
|
|
|
end
|
|
|
|
|
2012-11-20 00:31:14 -05:00
|
|
|
it 'schedules multiple jobs at once' do
|
|
|
|
@redis.expect :zadd, true, ['schedule', Array]
|
2013-03-24 20:42:43 -04:00
|
|
|
assert Sidekiq::Client.push_bulk('class' => ScheduledWorker, 'args' => [['mike'], ['mike']], 'at' => 600)
|
2012-11-20 00:31:14 -05:00
|
|
|
@redis.verify
|
|
|
|
end
|
2015-06-03 06:45:35 -04:00
|
|
|
|
|
|
|
it 'removes the enqueued_at field when scheduling' do
|
|
|
|
@redis.expect :zadd, true do |key, args|
|
|
|
|
job = Sidekiq.load_json(args.first.last)
|
|
|
|
job.key?('created_at') && !job.key?('enqueued_at')
|
|
|
|
end
|
|
|
|
assert ScheduledWorker.perform_in(1.month, 'mike')
|
|
|
|
@redis.verify
|
|
|
|
end
|
2012-05-25 23:21:42 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
end
|