From bc7389daba0394a0243ef1e1efdb428d065053ab Mon Sep 17 00:00:00 2001 From: Adam Niedzielski Date: Sat, 5 Feb 2022 23:25:42 +0100 Subject: [PATCH] Further improve tests related to bulk scheduling of jobs (#5171) * Use realistic values in scheduling tests * Add one more failing test case * Add failing test for regular push * Remove the new tests as they expect too much --- test/test_scheduling.rb | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/test/test_scheduling.rb b/test/test_scheduling.rb index debaa57e..b007b019 100644 --- a/test/test_scheduling.rb +++ b/test/test_scheduling.rb @@ -38,7 +38,7 @@ describe 'job scheduling' do assert_equal 3, ss.size assert_equal qs+1, q.size - assert Sidekiq::Client.push_bulk('class' => SomeScheduledWorker, 'args' => [['mike'], ['mike']], 'at' => 600) + assert Sidekiq::Client.push_bulk('class' => SomeScheduledWorker, 'args' => [['mike'], ['mike']], 'at' => Time.now.to_f + 100) assert_equal 5, ss.size assert SomeScheduledWorker.perform_in(TimeDuck.new, 'samwise') @@ -59,11 +59,10 @@ describe 'job scheduling' do ss = Sidekiq::ScheduledSet.new ss.clear - assert Sidekiq::Client.push_bulk('class' => SomeScheduledWorker, 'args' => [['mike'], ['mike']], 'at' => 600) + assert Sidekiq::Client.push_bulk('class' => SomeScheduledWorker, 'args' => [['mike'], ['mike']], 'at' => Time.now.to_f + 100) job = ss.first assert job['created_at'] refute job['enqueued_at'] end end - end