ci-queue 0.30.0 → 0.31.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 38ef31e58e85eed042bdfd90ceb5b6bb02adc6a842ad6a553730325235433af2
-  data.tar.gz: 629e6617553d6a59468b45eda582502960e335312be212edd39e931ebc883a93
+  metadata.gz: fdd7b371fdadda905994e7b736eddac19265db401483fad83612c3bfb5b054bb
+  data.tar.gz: 8e49ed6b35ece9ba9aa7c4876e75512f1bfa05375aa6d976a7813eb5cb304502
 SHA512:
-  metadata.gz: d1a67bfd6f6e3943a11e0033c0f1d11f8d187fa082c6f9de3c0bcb97049a6093cde2ac49dfb39a66cd79b352f20227a35aa0a011223a61fd4bb213d225bf1fe4
-  data.tar.gz: 833ba58c0c2b05780f5edc709eae9f2e1b78c72a8c49dbadb1ebc376d5cde0352d7fdc3f0e9b8e96dc27f8a53c01ec3ba8527b6435d50f085a0f8042abb88a86
+  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.30.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!
@@ -257,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?
@@ -421,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
@@ -547,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,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ci-queue
 version: !ruby/object:Gem::Version
-  version: 0.30.0
+  version: 0.31.0
 platform: ruby
 authors:
 - Jean Boussier