ci-queue 0.29.0 → 0.31.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7d223652a052d2cf2828d73baa3755a832ea90bee64ea40054d3a063ddb1a517
-  data.tar.gz: 69ff5cabeca011f4fa7fee60a534eda64336e9ab4a55e7d660d7c5b0001a0268
+  metadata.gz: fdd7b371fdadda905994e7b736eddac19265db401483fad83612c3bfb5b054bb
+  data.tar.gz: 8e49ed6b35ece9ba9aa7c4876e75512f1bfa05375aa6d976a7813eb5cb304502
 SHA512:
-  metadata.gz: c00238032c5e41dc30f74138cd92dc87523fd1e0386efa839dc4013eeb77640c035df42bfa5296a0ed58fd06fb2e00e1c72a3e05eae5f9a156749128e3b5fe4c
-  data.tar.gz: b1cdabd74fad311aee9ca745d38da22fa0c086f2e8f6a4494fbbd5b4fbab0aa0672d037e2e555843947e5dc5d4a3e66e5297ed7eaf051ec88c07400d57a3b12f
+  metadata.gz: ee4e5d450227ff962c4606e759e5d15272f5c808ff690b77dfb7ff403a989c4ffd860af75fb18e15a14354523ae96b66f6b67d5035b14880e3342ee63f254f5b
+  data.tar.gz: ca3c985d44043f4348708bf43b789c3b24d988dc74aeac710bf1a6f58fb8bfb309288b0a58b98dab8ece480713e73c443dae30e2118a29e19514094d3fc17959

data/dev.yml

@@ -5,7 +5,7 @@ name: ci-queue
 up:
 - ruby: 2.6.5
 - bundler
-- isogun
+- redis
 
 commands:
   test: REDIS_URL=${REDIS_URL:-redis://ci-queue.railgun/0} bundle exec rake test TEST_FILES="$*"

data/lib/ci/queue/version.rb

@@ -2,7 +2,7 @@
 
 module CI
   module Queue
-    VERSION = '0.29.0'
+    VERSION = '0.31.0'
     DEV_SCRIPTS_ROOT = ::File.expand_path('../../../../../redis', __FILE__)
     RELEASE_SCRIPTS_ROOT = ::File.expand_path('../redis', __FILE__)
   end

data/lib/minitest/queue/local_requeue_reporter.rb

@@ -4,7 +4,7 @@ require 'minitest/reporters'
 
 module Minitest
   module Queue
-    class LocalRequeueReporter < Minitest::Reporters::BaseReporter
+    class LocalRequeueReporter < Minitest::Reporters::DefaultReporter
       include ::CI::Queue::OutputHelpers
       attr_accessor :requeues
 

data/lib/minitest/queue/runner.rb

@@ -68,7 +68,7 @@ module Minitest
         set_load_path
         Minitest.queue = queue
         reporters = [
-          LocalRequeueReporter.new,
+          LocalRequeueReporter.new(verbose: verbose),
           BuildStatusRecorder.new(build: queue.build),
           JUnitReporter.new,
           TestDataReporter.new(namespace: queue_config&.namespace),
@@ -88,9 +88,36 @@ module Minitest
           load_tests
           populate_queue
         end
+
+        at_exit {
+          verify_reporters!(reporters)
+        }
         # Let minitest's at_exit hook trigger
       end
 
+      def verify_reporters!(reporters)
+        return unless reporters.any? { |r| !Minitest::Reporters.reporters.include?(r) }
+
+        warn <<~WARNING
+          WARNING!
+
+          ci-queue requires several custom minitest reporters.
+          Please do not overwrite them.
+          If you have a statement in your test suite like this
+
+            Minitest::Reporters.use!(SomeReporter.new)
+
+          you should only run it when other reporters have not been configured
+          to avoid breaking ci-queue's functionality and getting false test summaries.
+
+          Use something like this:
+
+            if Minitest::Reporters.reporters.nil?
+              Minitest::Reporters.use!(SomeReporter.new)
+            end
+        WARNING
+      end
+
       def release_command
         require_worker_id!
         queue.release!
@@ -177,6 +204,8 @@ module Minitest
           puts
           puts "cat <<EOF |\n#{failing_order.to_a.map(&:id).join("\n")}\nEOF\n#{command.join(' ')}"
           puts
+
+          File.write('log/test_order.log', failing_order.to_a.map(&:id).join("\n"))
           exit! 0
         end
       end
@@ -255,7 +284,7 @@ module Minitest
       private
 
       attr_reader :queue_config, :options, :command, :argv
-      attr_accessor :queue, :queue_url, :grind_list, :grind_count, :load_paths
+      attr_accessor :queue, :queue_url, :grind_list, :grind_count, :load_paths, :verbose
 
       def require_worker_id!
         if queue.distributed?
@@ -419,7 +448,7 @@ module Minitest
           EOS
           opts.separator ""
           opts.on('-IPATHS', help) do |paths|
-            self.load_paths = paths
+            self.load_paths = [load_paths, paths].compact.join(':')
           end
 
           help = <<~EOS
@@ -545,6 +574,10 @@ module Minitest
             queue.config.redis_ttl = time
           end
 
+          opts.on("-v", "--verbose", "Verbose. Show progress processing files.") do
+            self.verbose = true
+          end
+
           opts.separator ""
           opts.separator "    retry: Replays a previous run in the same order."
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ci-queue
 version: !ruby/object:Gem::Version
-  version: 0.29.0
+  version: 0.31.0
 platform: ruby
 authors:
 - Jean Boussier
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-05-31 00:00:00.000000000 Z
+date: 2023-06-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler