ci-queue 0.30.0 → 0.32.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 38ef31e58e85eed042bdfd90ceb5b6bb02adc6a842ad6a553730325235433af2
-  data.tar.gz: 629e6617553d6a59468b45eda582502960e335312be212edd39e931ebc883a93
+  metadata.gz: ffbfc433f36b0b8dcb19d9c2f17ab4227815dc1a7211b111d33a98fbc0ae6bf8
+  data.tar.gz: b84dc1389720a40520c74d57ae65d3161e5b00b91762d8ffc3a6cb4734d2d7dd
 SHA512:
-  metadata.gz: d1a67bfd6f6e3943a11e0033c0f1d11f8d187fa082c6f9de3c0bcb97049a6093cde2ac49dfb39a66cd79b352f20227a35aa0a011223a61fd4bb213d225bf1fe4
-  data.tar.gz: 833ba58c0c2b05780f5edc709eae9f2e1b78c72a8c49dbadb1ebc376d5cde0352d7fdc3f0e9b8e96dc27f8a53c01ec3ba8527b6435d50f085a0f8042abb88a86
+  metadata.gz: 3c7207ab7fa5995ea934212f5c5a2dac179b8bcdf67221040fc8e6e41e4653570b34e9aff18e4bbb647797ea4792e93d962c71d4a354ba57399a4114c6de7e61
+  data.tar.gz: '027848c07795e66c33fccb820465e9df3ff1fe2c75864192159e9e8902bbc679fcb21963a272b4f0d15ad0844b7524799ade11a02e3a8e45c276ec705f367a7c'

data/Gemfile

@@ -3,4 +3,4 @@ source 'https://rubygems.org'
 # Specify your gem's dependencies in ci-queue.gemspec
 gemspec
 
-gem 'activesupport', '~> 5.2.0'
+gem 'activesupport'

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.32.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!
@@ -148,6 +175,7 @@ module Minitest
         unless run_tests_in_fork(queue.failing_test)
           puts reopen_previous_step
           puts red("The test fail when ran alone, no need to bisect.")
+          File.write('log/test_order.log', queue_config.failing_test)
           exit! 0
         end
 
@@ -167,6 +195,7 @@ module Minitest
         step("Final validation")
         status = if run_tests_in_fork(failing_order)
           step(yellow("The bisection was inconclusive, there might not be any leaky test here."))
+          File.write('log/test_order.log', "")
           exit! 1
         else
           step(green('The following command should reproduce the leak on your machine:'), collapsed: false)
@@ -257,7 +286,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 +450,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 +576,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.30.0
+  version: 0.32.0
 platform: ruby
 authors:
 - Jean Boussier
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-06-14 00:00:00.000000000 Z
+date: 2023-07-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -239,7 +239,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.13
+rubygems_version: 3.4.16
 signing_key: 
 specification_version: 4
 summary: Distribute tests over many workers using a queue