Attempt to reduce flakiness of scheduler_test

Author: p-schlickmann

test/integration/instrumentation_test.rb

@@ -53,7 +53,7 @@ class InstrumentationTest < ActiveSupport::TestCase
   end
 
   test "stopping a worker with claimed executions emits release_claimed events" do
-    StoreResultJob.perform_later(42, pause: SolidQueue.shutdown_timeout + 100.second)
+    StoreResultJob.perform_later(42, pause: SolidQueue.shutdown_timeout + 10.seconds)
     process = nil
 
     events = subscribed(/release.*_claimed\.solid_queue/) do
@@ -88,7 +88,7 @@ class InstrumentationTest < ActiveSupport::TestCase
   end
 
   test "starting and stopping a worker emits register_process and deregister_process events" do
-    StoreResultJob.perform_later(42, pause: SolidQueue.shutdown_timeout + 100.second)
+    StoreResultJob.perform_later(42, pause: SolidQueue.shutdown_timeout + 10.seconds)
     process = nil
 
     events = subscribed(/(register|deregister)_process\.solid_queue/) do

test/unit/scheduler_test.rb

@@ -24,7 +24,7 @@ class SchedulerTest < ActiveSupport::TestCase
     end
 
     schedulers.each(&:start)
-    sleep 2
+    wait_while_with_timeout(2.5.seconds) { SolidQueue::RecurringExecution.count != 2 }
     schedulers.each(&:stop)
 
     assert_equal SolidQueue::Job.count, SolidQueue::RecurringExecution.count