ci-queue 0.85.0 → 0.86.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d7f6896ca23e9f9cd64dae9300539d0d5d88ced4c9b59a193d67824d353726cd
-  data.tar.gz: 969ba0b363dbe7e491561095587b802f24aa400685a96bcb5fda4b23e69dc02b
+  metadata.gz: 7708a4b0506c58da9ae1c6681dd335967c584869bf7b4a60546fc43f06a6cfe8
+  data.tar.gz: 22e9a6260641835f028c9952e10c3c0dfa4882156bf393b0b1a13befbee89432
 SHA512:
-  metadata.gz: dd6d91458d235f13c60df0f27374d09d3050d54354a7d87f0d427c21c558bac9338054a901a350eda42c37118af7893cfb763db96ab7be43b1f0d6a275ac2e6d
-  data.tar.gz: 833e87a9a0fd535589267462f98f81d134342f66432a43b246a0bc68632a64b85e9d80253a27d9ccf627898a5d846ea37d2c5bef7d2a602905d7c21195ddbff6
+  metadata.gz: 697d15d2ada5cae5ace00714a47b91fbb32ebf61a3abf7b181fb4293b560ba837e02c1f88aef40832a9534fcdc45a049c1ef8d19bf5b4b537f2b53d4ca1d6cf5
+  data.tar.gz: 225d1ec46ad29137111f16f18431212db237eeaa91871fb9ece7e0e35ba34270b4272b2ce26cff3b178214feb6643a75b5266ae86d41aef5ba47952064996a81

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    ci-queue (0.85.0)
+    ci-queue (0.86.0)
       logger
 
 GEM

data/ci-queue.gemspec

@@ -41,9 +41,9 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'simplecov', '~> 0.12'
   spec.add_development_dependency 'minitest-reporters', '~> 1.1'
 
+  spec.add_development_dependency 'rexml'
   spec.add_development_dependency 'snappy'
   spec.add_development_dependency 'msgpack'
   spec.add_development_dependency 'benchmark'
-  spec.add_development_dependency 'rexml'
   spec.add_development_dependency 'rubocop'
 end

data/lib/ci/queue/grind.rb

@@ -10,6 +10,12 @@ module CI
         end
       end
 
+      # Grind always eagerly loads test files and populates @index via
+      # Static#populate, so poll uses the @index path. Bypass entry_resolver
+      # to avoid JSON-formatting overhead on potentially millions of entries.
+      def entry_resolver; nil; end
+      def entry_resolver=(_); end
+
       def initialize(path, config)
         io = path == '-' ? STDIN : ::File.open(path)
 

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

@@ -13,6 +13,11 @@ module CI
       self.requeue_offset = 42
       self.max_sleep_time = 2
 
+      # Minimal wrapper returned by resolve_entry when neither @index nor entry_resolver
+      # is available. Provides the interface callers expect (.id, .queue_entry) so that
+      # downstream code doesn't crash with NoMethodError on a raw String.
+      UnresolvedEntry = Struct.new(:id, :queue_entry)
+
       class Worker < Base
         attr_accessor :entry_resolver
         attr_reader :first_reserve_at
@@ -294,7 +299,7 @@ module CI
 
           return entry_resolver.call(entry) if entry_resolver
 
-          entry
+          UnresolvedEntry.new(test_id, entry)
         end
 
         def still_streaming?

data/lib/ci/queue/version.rb

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

data/lib/minitest/queue/queue_population_strategy.rb

@@ -35,6 +35,7 @@ module Minitest
           configure_lazy_queue
           queue.stream_populate(lazy_test_enumerator, random: ordering_seed, batch_size: queue_config.lazy_load_stream_batch_size)
         else
+          configure_lazy_queue
           queue.populate(Minitest.loaded_tests, random: ordering_seed)
         end
       end

data/lib/minitest/queue/runner.rb

@@ -118,6 +118,15 @@ module Minitest
           # minitest/autorun's at_exit hook, which may not be registered since
           # test files haven't been loaded yet. exit! prevents double-execution
           # if minitest/autorun was loaded by the leader during streaming.
+          #
+          # Re-check exhausted? after booting: slow workers may arrive after the queue
+          # has been fully drained by faster workers. In that case exit cleanly (0)
+          # rather than letting Minitest return false for a 0-test run.
+          if queue.rescue_connection_errors { queue.exhausted? }
+            puts green('All tests were ran already')
+            verify_reporters!(reporters)
+            exit!(0)
+          end
           passed = Minitest.run []
           verify_reporters!(reporters)
           exit!(passed ? 0 : 1)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ci-queue
 version: !ruby/object:Gem::Version
-  version: 0.85.0
+  version: 0.86.0
 platform: ruby
 authors:
 - Jean Boussier
@@ -122,7 +122,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.1'
 - !ruby/object:Gem::Dependency
-  name: snappy
+  name: rexml
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -136,7 +136,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: msgpack
+  name: snappy
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -150,7 +150,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: benchmark
+  name: msgpack
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -164,7 +164,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rexml
+  name: benchmark
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -291,7 +291,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.8
+rubygems_version: 4.0.9
 specification_version: 4
 summary: Distribute tests over many workers using a queue
 test_files: []