forking_test_runner 1.12.0 → 1.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: be87df3e12024204491610717130bc0b71a3b7fc02d6a60c82da34ebedbf74c6
-  data.tar.gz: 4e6fb9c449dc7e00c46e39fdbc7686bcfe0bf1df439c800e78767950a517f6c4
+  metadata.gz: e38d2bbf3dd185f3f8c912537be54a0068c98db3c45b17edda9a53a01ccd63d8
+  data.tar.gz: 63393812f02d8841d6ee56c000d9400e0062f405d7eb95eccbd766a93877e7d8
 SHA512:
-  metadata.gz: 323f765b32f2ed37f62a82dfff10b6bdc7c0c8670a441f340f2a910d8dfb17a9660726698b8c34dfb89522246003e8c7fde4a205d17b8511c3ec432027604a65
-  data.tar.gz: ad243eb959c09b3bac8d58da961adec4c34b92844b239da2e541d599e2f1b78d738bdc3cc5db7b44c4817e18aa99be86d5adb74bac2e5614c01b57c027a0b562
+  metadata.gz: 1cf9450f7cf1fc7259bf5ec9a51f88141ce7e937aca94152577919117ff1fbec3311591f045b01864cd644b6ada45e71265656c91c067242da7c4195e987df27
+  data.tar.gz: ac79e34dab39a08445773c394b1912f7119605187cf2a09af670e94f54b2c2d786dc43a09968e50443881220d7d62f7bfcebf167521569b497d96603714962d5

data/bin/forking-test-runner

@@ -1,7 +1,8 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 # enable local usage from cloned repo
-root = File.expand_path("../..", __FILE__)
+root = File.expand_path('..', __dir__)
 $LOAD_PATH << "#{root}/lib" if File.exist?("#{root}/Gemfile")
 
 require 'forking_test_runner'

data/lib/forking_test_runner/cli.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module ForkingTestRunner
   # read and delete options we support and pass the rest through to the underlying test runner (-v / --seed etc)
   module CLI
@@ -11,9 +12,9 @@ module ForkingTestRunner
       [
         :record_runtime,
         "--record-runtime=MODE",
-        "\n      Record test runtime:\n" <<
-          "        simple = write to disk at --runtime-log)\n" <<
-          "        amend  = write from multiple remote workers via http://github.com/grosser/amend, needs TRAVIS_REPO_SLUG & TRAVIS_BUILD_NUMBER",
+        "\n      Record test runtime:\n        " \
+        "simple = write to disk at --runtime-log)\n        " \
+        "amend  = write from multiple remote workers via http://github.com/grosser/amend, needs TRAVIS_REPO_SLUG & TRAVIS_BUILD_NUMBER",
         String
       ],
       [:runtime_log, "--runtime-log=FILE", "File to store runtime log in or runtime.log", String],
@@ -22,7 +23,7 @@ module ForkingTestRunner
       [:groups, "--groups=NUM", "How many groups there are in total (use with --group)", Integer],
       [:version, "--version", "Show version"],
       [:help, "--help", "Show help"]
-    ]
+    ].freeze
 
     class << self
       def parse_options(argv)
@@ -43,8 +44,8 @@ module ForkingTestRunner
         end
 
         # check if we can use merge_coverage
-        if options.fetch(:merge_coverage)
-          abort "merge_coverage does not work on ruby prior to 2.3" if RUBY_VERSION < "2.3.0"
+        if options.fetch(:merge_coverage) && "2.3.0" > RUBY_VERSION
+          abort "merge_coverage does not work on ruby prior to 2.3"
         end
 
         if !!options.fetch(:group) ^ !!options.fetch(:groups)

data/lib/forking_test_runner/coverage_capture.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module ForkingTestRunner
   module CoverageCapture
     # override Coverage.result to add pre-fork captured coverage
@@ -7,8 +8,7 @@ module ForkingTestRunner
     end
 
     # deprecated, single_cov checks for this, so leave it here
-    def capture_coverage!
-    end
+    def capture_coverage!; end
 
     class << self
       attr_accessor :coverage
@@ -49,7 +49,7 @@ module ForkingTestRunner
       def merge_lines_coverage(a, b)
         b.each_with_index.map do |b_count, i|
           a_count = a[i]
-          (a_count.nil? && b_count.nil?) ? nil : a_count.to_i + b_count.to_i
+          a_count.nil? && b_count.nil? ? nil : a_count.to_i + b_count.to_i
         end
       end
 

data/lib/forking_test_runner/version.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module ForkingTestRunner
-  VERSION = "1.12.0"
+  VERSION = "1.14.0"
 end

data/lib/forking_test_runner.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'benchmark'
 require 'optparse'
 require 'forking_test_runner/version'
@@ -11,6 +12,8 @@ module ForkingTestRunner
   CONVERAGE_REPORT_PREFIX = "coverage/fork-"
 
   class << self
+    attr_accessor :before_fork_callbacks, :after_fork_callbacks
+
     def cli(argv)
       @options, tests = CLI.parse_options(argv)
 
@@ -34,21 +37,24 @@ module ForkingTestRunner
         puts "Running tests #{all_tests.map(&:first).join(" ")}"
       end
 
+      @before_fork_callbacks = []
+      @after_fork_callbacks = []
+
       # run all the tests
       results = with_lock do |lock|
-        Parallel.map_with_index(test_groups, in_processes: parallel || 0) do |tests, env_index|
+        Parallel.map_with_index(test_groups, in_processes: parallel || 0) do |tests_group, env_index|
           if parallel
             ENV["TEST_ENV_NUMBER"] = (env_index == 0 ? '' : (env_index + 1).to_s) # NOTE: does not support first_is_1 option
           end
 
           reraise_clean_ar_error { load_test_env }
 
-          tests.map do |file, expected|
+          tests_group.map do |file, expected|
             print_started file unless parallel
             result = [file, expected, *benchmark { run_test(file) }]
             sync_stdout lock do
               print_started file if parallel
-              print_finished *result
+              print_finished(*result)
             end
             result
           end
@@ -58,9 +64,11 @@ module ForkingTestRunner
       unless @options.fetch(:quiet)
         # pretty print the results
         puts "\nResults:"
-        puts results.
-          sort_by { |_,_,_,r,_| r ? 0 : 1 }. # failures should be last so they are easy to find
-          map { |f,_,_,r,_| "#{f}: #{r ? "OK" : "Fail"}"}
+        puts(
+          results
+                    .sort_by { |_, _, _, r, _| r ? 0 : 1 } # failures should be last so they are easy to find
+                    .map { |f, _, _, r, _| "#{f}: #{r ? "OK" : "Fail"}" }
+        )
         puts
       end
 
@@ -90,7 +98,7 @@ module ForkingTestRunner
 
     def with_lock(&block)
       return yield unless @options.fetch(:parallel)
-      Tempfile.open"forking-test-runner-lock", &block
+      Tempfile.open "forking-test-runner-lock", &block
     end
 
     def sync_stdout(lock)
@@ -155,7 +163,7 @@ module ForkingTestRunner
         File.write(log, data)
       when 'amend'
         if id = ENV["BUILDKITE_JOB_ID"]
-          slug = ENV.fetch("BUILDKITE_ORG_SLUG") + "-" + ENV.fetch("BUILDKITE_PIPELINE_SLUG")
+          slug = "#{ENV.fetch("BUILDKITE_ORG_SLUG")}-#{ENV.fetch("BUILDKITE_PIPELINE_SLUG")}"
         else
           slug = ENV.fetch("TRAVIS_REPO_SLUG").sub("/", "-")
           id = ENV.fetch("TRAVIS_BUILD_NUMBER")
@@ -195,6 +203,7 @@ module ForkingTestRunner
         preload_fixtures
         ActiveRecord::Base.connection.disconnect!
       end
+      @before_fork_callbacks.each(&:call)
 
       CoverageCapture.capture! if @options.fetch(:merge_coverage)
     end
@@ -205,7 +214,7 @@ module ForkingTestRunner
       e = begin
         yield
         nil
-      rescue
+      rescue StandardError
         $!
       end
 
@@ -216,7 +225,7 @@ module ForkingTestRunner
     def load_test_helper
       disable_test_autorun
       require 'rspec/core' if @options.fetch(:rspec)
-      helper =  @options.fetch(:helper) || (@options.fetch(:rspec) ? "spec/spec_helper" : "test/test_helper")
+      helper = @options.fetch(:helper) || (@options.fetch(:rspec) ? "spec/spec_helper" : "test/test_helper")
       require "./#{helper}"
     end
 
@@ -261,7 +270,7 @@ module ForkingTestRunner
 
       wpipe.close
 
-      buffer = ""
+      buffer = +""
 
       while ch = rpipe.read(1)
         buffer << ch
@@ -276,7 +285,11 @@ module ForkingTestRunner
     def preserve_tty
       was_tty = $stdout.tty?
       yield
-      def $stdout.tty?; true; end if was_tty
+      if was_tty
+        def $stdout.tty?;
+          true;
+        end
+      end
     end
 
     def run_test(file)
@@ -289,6 +302,9 @@ module ForkingTestRunner
           if partial_reports_for_single_cov?
             SingleCov.coverage_report = "#{CONVERAGE_REPORT_PREFIX}#{Process.pid}.json"
           end
+
+          @after_fork_callbacks.each(&:call)
+
           if active_record?
             key = (ActiveRecord::VERSION::STRING >= "4.1.0" ? :test : "test")
             ActiveRecord::Base.establish_connection key
@@ -307,7 +323,8 @@ module ForkingTestRunner
     def change_program_name_to(name)
       return yield if @options.fetch(:parallel)
       begin
-        old, $0 = $0, name
+        old = $0
+        $0 = name
         yield
       ensure
         $0 = old
@@ -328,7 +345,7 @@ module ForkingTestRunner
       group = groups[group - 1] || raise("Group #{group} not found")
 
       # return tests with runtime
-      tests = Hash[tests]
+      tests = tests.to_h
       group.map { |test| [test, (tests[test] if group_by == :runtime)] }
     end
 
@@ -344,7 +361,7 @@ module ForkingTestRunner
       end
     end
 
-    def toggle_test_autorun(value, file=nil)
+    def toggle_test_autorun(value, file = nil)
       if @options.fetch(:rspec)
         if value
           exit(RSpec::Core::Runner.run([file] + ARGV))
@@ -381,7 +398,7 @@ module ForkingTestRunner
         File.unlink(report) # do not leave junk behind
       end
 
-      data = JSON.pretty_generate(key => {"coverage" => coverage, "timestamp" => Time.now.to_i })
+      data = JSON.pretty_generate(key => { "coverage" => coverage, "timestamp" => Time.now.to_i })
       File.write(SingleCov.coverage_report, data)
 
       # make it not override our report when it finishes for main process

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: forking_test_runner
 version: !ruby/object:Gem::Version
-  version: 1.12.0
+  version: 1.14.0
 platform: ruby
 authors:
 - Michael Grosser
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-19 00:00:00.000000000 Z
+date: 2024-08-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parallel_tests
@@ -39,21 +39,27 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.3'
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.3.27
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.3'
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.3.27
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,7 +73,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: sqlite3
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -81,7 +87,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: minitest
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -94,7 +100,35 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.65.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.65.1
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.6.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.6.0
+description:
 email: michael@grosser.it
 executables:
 - forking-test-runner
@@ -111,7 +145,7 @@ homepage: https://github.com/grosser/forking_test_runner
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -119,15 +153,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: 2.7.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.16
-signing_key: 
+rubygems_version: 3.4.10
+signing_key:
 specification_version: 4
 summary: Run every test in a fork to avoid pollution and get clean output per test
 test_files: []