forking_test_runner 1.16.0 → 1.18.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 347d5e63755b11aefa879f5c697e9cc6cab2ca169051ec12f326d10037c38a7f
-  data.tar.gz: 5368d418bad4d28203064fbcb31434a92789a3063be0384b5b7c690478c2c8c7
+  metadata.gz: a248306cb1a71ac76fe62cff26f468f3aa8ae39f701f6c2d6ab8ed43bc2c0491
+  data.tar.gz: a4683438d4fa23251d609568fec91e147c77325548d6d0c616e5034c6bc26efe
 SHA512:
-  metadata.gz: 588c1a0b82819d48e8c3cfc3cdf63d3443c4f64ef20950a110cd3103a150ea42721ed4c71916face7940805ecf1b70071431f88937b03beefb18d033621342c1
-  data.tar.gz: c25e5abe903f105a56cf7b79c50e00ea8285653329b804813c44de25f2c584c38eb3c6cdde8340f66aba5efe52489161917b1a3c0087be4a13c55effde4a1925
+  metadata.gz: a63845433bc9bef04de653f116dc20377ad1c16d7014e81f2bc16b32e02b68f6bab9b46cb9de0ea6740d86c74021381c5f457deb6fc6c212e3555f6b095948c2
+  data.tar.gz: b80c1bcb90d560d74af870dff0b44bffee28bc1a60c0a1ee0607d96361c4b00e8fb210d96734482495aabc003fe1a7ef01464e6f1838730dfed317add6871188

data/lib/forking_test_runner/cli.rb

@@ -11,16 +11,16 @@ module ForkingTestRunner
       [:merge_coverage, "--merge-coverage", "Merge base code coverage into individual files coverage and summarize coverage report"],
       [
         :record_runtime,
-        "--record-runtime=MODE",
+        "--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",
         String
       ],
-      [:runtime_log, "--runtime-log=FILE", "File to store runtime log in or runtime.log", String],
-      [:parallel, "--parallel=NUM", "Number of parallel groups to run at once", Integer],
-      [:group, "--group=NUM[,NUM]", "What group this is (use with --groups / starts at 1)", String],
-      [:groups, "--groups=NUM", "How many groups there are in total (use with --group)", Integer],
+      [:runtime_log, "--runtime-log FILE", "File to store runtime log in or runtime.log", String],
+      [:parallel, "--parallel [NUM]", "Number of parallel groups to run at once", Integer],
+      [:group, "--group NUM[,NUM]", "What group this is (use with --groups / starts at 1)", String],
+      [:groups, "--groups NUM", "How many groups there are in total (use with --group)", Integer],
       [:version, "--version", "Show version"],
       [:help, "--help", "Show help"]
     ].freeze
@@ -28,9 +28,13 @@ module ForkingTestRunner
     class << self
       def parse_options(argv)
         options = OPTIONS.each_with_object({}) do |(setting, flag, _, type), all|
-          all[setting] = delete_argv(flag.split('=', 2)[0], argv, type:)
+          optional = flag.include?("[")
+          all[setting] = delete_argv(flag.split(' ', 2)[0], argv, type: type, optional: optional)
         end
 
+        # default parallel to processor count
+        options[:parallel] = Parallel.processor_count if options[:parallel] == true
+
         # show version
         if options.fetch(:version)
           puts VERSION
@@ -76,18 +80,31 @@ module ForkingTestRunner
       # so minitest / rspec can read their own options (--seed / -v ...)
       #  - keep our options clear / unambiguous to avoid overriding
       #  - read all serial non-flag arguments as tests and leave only unknown options behind
-      #  - use .fetch everywhere to make sure nothing is misspelled
-      # GOOD: test --ours --theirs
-      # OK: --ours test --theirs
-      # BAD: --theirs test --ours
-      def delete_argv(name, argv, type: nil)
-        return unless index = argv.index(name)
-        argv.delete_at(index)
-        if type
-          found = argv.delete_at(index) || raise("Missing argument for #{name}")
-          send(type.name, found) # case found
-        else
-          true
+      def delete_argv(name, argv, type: nil, optional: false)
+        if type # value needed
+          value =
+            if (index = argv.index(name)) # user used `--foo bar` style ?
+              argv.delete_at(index)
+              next_arg = argv[index]
+              if next_arg && !next_arg.start_with?("-")
+                argv.delete_at(index)
+              elsif optional
+                return true
+              else
+                raise(ArgumentError, "Missing argument for #{name}")
+              end
+            else # user used `--foo=bar` style ?
+              prefix = "#{name}="
+              return unless (index = argv.index { |arg| arg.start_with?(prefix) })
+              argv.delete_at(index).delete_prefix(prefix)
+            end
+
+          send(type.name, value) # for example Integer(found)
+        else # no value needed
+          if (index = argv.index(name))
+            argv.delete_at(index)
+            true
+          end
         end
       end
     end

data/lib/forking_test_runner/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module ForkingTestRunner
-  VERSION = "1.16.0"
+  VERSION = "1.18.0"
 end

data/lib/forking_test_runner.rb

@@ -66,8 +66,8 @@ module ForkingTestRunner
         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"}" }
+            .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
@@ -82,7 +82,7 @@ module ForkingTestRunner
         puts "Time: #{diff.round(2)} diff to expected"
       end
 
-      if mode = @options.fetch(:record_runtime)
+      if (mode = @options.fetch(:record_runtime))
         # store runtime log
         log = runtime_log || 'runtime.log'
         record_test_runtime(mode, results, log)
@@ -162,7 +162,7 @@ module ForkingTestRunner
       when 'simple'
         File.write(log, data)
       when 'amend'
-        if id = ENV["BUILDKITE_JOB_ID"]
+        if (id = ENV["BUILDKITE_JOB_ID"])
           slug = "#{ENV.fetch("BUILDKITE_ORG_SLUG")}-#{ENV.fetch("BUILDKITE_PIPELINE_SLUG")}"
         else
           slug = ENV.fetch("TRAVIS_REPO_SLUG").sub("/", "-")
@@ -278,10 +278,11 @@ module ForkingTestRunner
       wpipe.close
 
       buffer = +""
+      tee = !@options.fetch(:quiet) && !@options.fetch(:parallel)
 
-      while ch = rpipe.read(1)
+      while (ch = rpipe.read(tee ? 1 : 4096))
         buffer << ch
-        $stdout.write(ch) if !@options.fetch(:quiet) && !@options.fetch(:parallel) # tee
+        $stdout.write(ch) if tee
       end
 
       Process.wait(child)
@@ -344,8 +345,8 @@ module ForkingTestRunner
       tests = ParallelTests::Test::Runner.send(
         :tests_with_size,
         tests,
-        runtime_log:,
-        group_by:
+        runtime_log: runtime_log,
+        group_by: group_by
       )
       groups = ParallelTests::Grouper.in_even_groups_by_size(tests, group_count, {})
       group = groups[group - 1] || raise("Group #{group} not found")

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: forking_test_runner
 version: !ruby/object:Gem::Version
-  version: 1.16.0
+  version: 1.18.0
 platform: ruby
 authors:
 - Michael Grosser
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-11-22 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark
@@ -56,22 +55,16 @@ dependencies:
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.4'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.4.22
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.4'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.4.22
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: drb
   requirement: !ruby/object:Gem::Requirement
@@ -104,16 +97,16 @@ dependencies:
   name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '6.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: mutex_m
   requirement: !ruby/object:Gem::Requirement
@@ -174,17 +167,16 @@ dependencies:
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.4'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.4'
-description:
+        version: '0'
 email: michael@grosser.it
 executables:
 - forking-test-runner
@@ -201,7 +193,6 @@ homepage: https://github.com/grosser/forking_test_runner
 licenses:
 - MIT
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -209,15 +200,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 3.1.0
+      version: 3.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.10
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Run every test in a fork to avoid pollution and get clean output per test
 test_files: []