Refactor: extract execute_master_setup to remove duplication

Extract shared master setup logic (reorder tests, store chunk metadata,
push to queue) into execute_master_setup method. Used by both initial
master setup in populate() and takeover in run_master_setup().

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: ssteele110

ruby/lib/ci/queue/redis/base.rb

@@ -70,7 +70,7 @@ def wait_for_master(timeout: 120)
                 if respond_to?(:attempt_master_takeover, true) && attempt_master_takeover
                   # Takeover succeeded - run master setup
                   if respond_to?(:run_master_setup, true)
-                    run_master_setup
+                    execute_master_setup
                     return true
                   end
                 end

ruby/lib/ci/queue/redis/worker.rb

@@ -39,20 +39,9 @@ def populate(tests, random: Random.new)
           # All workers need an index of tests to resolve IDs
           @index = tests.map { |t| [t.id, t] }.to_h
           @total = tests.size
+          @random = random
 
-          if acquire_master_role?
-            with_master_setup_heartbeat do
-              executables = reorder_tests(tests, random: random)
-
-              chunks = executables.select { |e| e.is_a?(CI::Queue::TestChunk) }
-              individual_tests = executables.reject { |e| e.is_a?(CI::Queue::TestChunk) }
-
-              store_chunk_metadata(chunks) if chunks.any?
-
-              all_ids = chunks.map(&:id) + individual_tests.map(&:id)
-              push(all_ids)
-            end
-          end
+          execute_master_setup(tests) if acquire_master_role?
 
           register_worker_presence
 
@@ -589,18 +578,12 @@ def register_worker_presence
           raise if master?
         end
 
-        # Run master setup after a successful takeover.
-        # Reconstructs the work the original master would have done.
-        def run_master_setup
+        # Shared logic for master setup - reorders tests, stores chunk metadata, and pushes to queue.
+        # Used by both initial master setup (populate) and takeover.
+        def execute_master_setup(tests)
           return unless @master && @index
-
-          # Reconstruct tests array from index
-          tests = @index.values
-
           with_master_setup_heartbeat do
-            # Use the same seed for deterministic ordering
-            random = Random.new(Digest::MD5.hexdigest(config.seed).to_i(16))
-            executables = reorder_tests(tests, random: random)
+            executables = reorder_tests(tests, random: @random)
 
             chunks = executables.select { |e| e.is_a?(CI::Queue::TestChunk) }
             individual_tests = executables.reject { |e| e.is_a?(CI::Queue::TestChunk) }