From 40d568c9894af165bea1fce6644b6fd8358fcd8b Mon Sep 17 00:00:00 2001 From: David Stosik Date: Fri, 22 Aug 2025 10:00:32 +0900 Subject: [PATCH] Don't reserve `nil` when there are no tests left --- ruby/lib/ci/queue/redis/worker.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ruby/lib/ci/queue/redis/worker.rb b/ruby/lib/ci/queue/redis/worker.rb index 9445f339..d311be39 100644 --- a/ruby/lib/ci/queue/redis/worker.rb +++ b/ruby/lib/ci/queue/redis/worker.rb @@ -168,9 +168,9 @@ def raise_on_mismatching_test(test) end def reserve - test = (try_to_reserve_lost_test || try_to_reserve_test) - reserved_tests << test - test + (try_to_reserve_lost_test || try_to_reserve_test).tap do |test| + reserved_tests << test if test + end end def try_to_reserve_test