minitest-distributed 0.2.11 → 0.2.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3755320df95c6d8e5ba5e32b04b44351043588d43c309d4d746cc36f96567132
-  data.tar.gz: 785503f39eadcfab72b7fc3df6e8f19d3ad0fbaea793e68f9f233ed08c4f6452
+  metadata.gz: 722ec2cc00da4f9c5f5efc0591e39c2becc2338ce7fe171c60339d284116242b
+  data.tar.gz: aa1d3c52afa3b16c649a0b12d27e83ccb19ae11dd80fb68d3e57a669e0a081e5
 SHA512:
-  metadata.gz: fd42edc7fd402f38c08f22ff17fa8430b65cae6d1c53b27afa55446ab5cb6e90f0c0d8a92c6fa7a7665d7aba4a81768a52d9a5c1007da2ebf6407414c2538510
-  data.tar.gz: e3b5119c06eaec10a7f8befacec8b3fde636e17eb9c8a4b539ae4aedf95d00217a97911ccf067e14f08850e1dbc3a6d072723811b565ea06aff46be8fb2607ae
+  metadata.gz: c2cfc291aeb202bb680b04a6e579272fd907efe47ef1d5e5ea2b66ca5ffe23967ded9ea38458e485e1e9b42609d84a2058fd2df3250217cf76a60272f3b3255c
+  data.tar.gz: cb0934c54f7c3bdcc76abe8c0593b08e2e7ddfacd9cf2c4e7ce3a71e586b2e75170be91ebe3ac8b520fb7c8c7e4e961ae4a9a218c582bf1301dc916f41427847

data/dev.yml

@@ -5,3 +5,4 @@ type: ruby
 up:
   - ruby
   - bundler
+  - redis

data/lib/minitest/distributed/coordinators/redis_coordinator.rb

@@ -3,6 +3,7 @@
 
 require "redis"
 require "set"
+require "logger"
 
 module Minitest
   module Distributed
@@ -268,7 +269,25 @@ module Minitest
 
         sig { returns(Redis) }
         def redis
-          @redis ||= Redis.new(url: configuration.coordinator_uri.to_s)
+          @redis ||= Redis.new(
+            url: configuration.coordinator_uri.to_s,
+            middlewares: custom_middlewares,
+            custom: custom_config,
+            timeout: 2,
+          )
+        end
+
+        sig { returns(T.nilable(T::Array[Module])) }
+        def custom_middlewares
+          return unless ENV.key?("MINITEST_DISTRIBUTED_REDIS_LOG")
+
+          require_relative "redis_instrumentation_middleware"
+          [RedisInstrumentationMiddleware]
+        end
+
+        sig { returns(T.nilable(T::Hash[Symbol, File])) }
+        def custom_config
+          { log_file: logger }.compact
         end
 
         sig { returns(String) }
@@ -517,6 +536,13 @@ module Minitest
           adjust_combined_results(batch_result_aggregate)
         end
 
+        sig { returns(T.nilable(Logger)) }
+        def logger
+          return unless (log_path = ENV["MINITEST_DISTRIBUTED_REDIS_LOG"])
+
+          @logger ||= T.let(Logger.new(log_path), T.nilable(Logger))
+        end
+
         INITIAL_BACKOFF = 10 # milliseconds
         private_constant :INITIAL_BACKOFF
       end

data/lib/minitest/distributed/coordinators/redis_instrumentation_middleware.rb

@@ -0,0 +1,39 @@
+# typed: strict
+# frozen_string_literal: true
+
+require "redis"
+
+module Minitest
+  module Distributed
+    module Coordinators
+      # Redis middleware that logs all Redis commands to a file for debugging.
+      module RedisInstrumentationMiddleware
+        extend T::Sig
+
+        sig { params(command: T::Array[T.untyped], redis_config: T.untyped).returns(T.untyped) }
+        def call(command, redis_config)
+          log_file = redis_config.custom[:log_file]
+          log_file.info("EXEC: #{command.inspect}")
+          result = super
+          log_file.info("RESULT: #{result.inspect}")
+          result
+        rescue => e
+          log_file.info("ERROR: #{e.class}")
+          Kernel.raise
+        end
+
+        sig { params(commands: T::Array[T.untyped], redis_config: T.untyped).returns(T.untyped) }
+        def call_pipelined(commands, redis_config)
+          log_file = redis_config.custom[:log_file]
+          log_file.info("EXEC PIPELINED: #{commands.inspect}")
+          result = super
+          log_file.info("RESULT PIPELINED: #{result.inspect}")
+          result
+        rescue => e
+          log_file.info("ERROR PIPELINED: #{e.class}")
+          Kernel.raise
+        end
+      end
+    end
+  end
+end

data/lib/minitest/distributed/version.rb

@@ -3,6 +3,6 @@
 
 module Minitest
   module Distributed
-    VERSION = "0.2.11"
+    VERSION = "0.2.12"
   end
 end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: minitest-distributed
 version: !ruby/object:Gem::Version
-  version: 0.2.11
+  version: 0.2.12
 platform: ruby
 authors:
 - Willem van Bergen
-autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-09-16 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest
@@ -116,6 +115,7 @@ files:
 - lib/minitest/distributed/coordinators/coordinator_interface.rb
 - lib/minitest/distributed/coordinators/memory_coordinator.rb
 - lib/minitest/distributed/coordinators/redis_coordinator.rb
+- lib/minitest/distributed/coordinators/redis_instrumentation_middleware.rb
 - lib/minitest/distributed/enqueued_runnable.rb
 - lib/minitest/distributed/filters/exclude_file_filter.rb
 - lib/minitest/distributed/filters/exclude_filter.rb
@@ -169,7 +169,6 @@ metadata:
   allowed_push_host: https://rubygems.org
   homepage_uri: https://github.com/Shopify/minitest-distributed
   source_code_uri: https://github.com/Shopify/minitest-distributed
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -184,8 +183,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.18
-signing_key:
+rubygems_version: 3.7.2
 specification_version: 4
 summary: Distributed test executor plugin for Minitest
 test_files: []