simplecov 1.0.0.rc1 → 1.0.0.rc2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aa29d54833537aa0434a697ada8befbfbb81c97c608afc1e7cddd6971009ec79
-  data.tar.gz: df39d6ee8257c5cc7575621d83a428a96f80fc1db762bf3e247927ace5d1050d
+  metadata.gz: 95cc757bf172ce50cd60e636048c88e30fade408c962d8fe8c9478465b6b2d67
+  data.tar.gz: 9ee4b98d8f3f790289415b03c3e37c436b32d1e2a9d193fd435dfdb966d4b658
 SHA512:
-  metadata.gz: be38ab39b2d235b3a2fbdd7fcd298684f1ff43d40e966a43174489da2b2680ac99dfba059b70ebcd774353265f5feab852a126ca242c97703f6e0bffea29f3c1
-  data.tar.gz: 91e057b46a0f7be3d914af750bedd7fcc3abe174db97970a7babedd85b047bfe69a682b34723f0d60115d6d4b1f674e854c431f854ff4f054b6194c6e9ab07d3
+  metadata.gz: 719bcdfee0e3e79de52e7da4fdd955d85416b2c33c3b8bcd437d13187b9d883c6430ea55e0d52ca17c392cfe4e506f6a433278d5d285091858702bd669eb43bd
+  data.tar.gz: 4e70c6fabe5ed02ea4ece1ca811434c96b760e871507cc607a05af64710a9f0c14ee87c16dbcf82d19cdaaf23f3896d78ea0500e051130dbc15b7c6046e4b6b8

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+1.0.0.rc2 (2026-06-10)
+======================
+
+## Bugfixes
+* Forked subprocesses (for example Rails' `parallelize`) are now named by a stable per-run serial rather than the child's OS process id. Because the pid changed every run, a re-run's worker results were never named the same as the previous run's and so never overwrote them. They accumulated in `.resultset.json` until `merge_timeout` dropped them, and while several runs sat inside that window together SimpleCov merged them all. When the set of files had drifted between those runs (a deleted file, a changed filter) the stale results leaked into the report, inflating the denominator and changing the coverage percentage from one run to the next. The serial sequence is identical from one run to the next, so a re-run's workers now overwrite the previous run's entries and the resultset stays bounded. See #1171.
+* The "Excluded N result(s) older than `merge_timeout`" warning is now emitted at most once, from the reporting process, instead of once per forked worker. Every worker merges the resultset too, and the default `at_fork` sets `print_errors false` for them, but this one warning did not honor the flag, so an eight-worker run printed eight copies. See #1171.
+* Legacy-API deprecation warnings (`track_files`, `add_filter`, `add_group`, and the rest) are now deduplicated by call site and emitted at most once per source location. A deprecated method called in a loop, or a configuration block re-evaluated once per parallel worker or spec file, previously repeated the same notice until the surrounding output was unreadable. See #1204.
+
 1.0.0.rc1 (2026-06-02)
 ======================
 

data/lib/simplecov/configuration/coverage_criteria.rb

@@ -110,8 +110,8 @@ module SimpleCov
 
     # DEPRECATED: prefer `enable_coverage :eval`.
     def enable_coverage_for_eval
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov.enable_coverage_for_eval` is deprecated. " \
-           "Replace with `SimpleCov.enable_coverage :eval`."
+      SimpleCov::Deprecation.warn("`SimpleCov.enable_coverage_for_eval` is deprecated. " \
+                                  "Replace with `SimpleCov.enable_coverage :eval`.")
       enable_eval_coverage
     end
 

data/lib/simplecov/configuration/filters.rb

@@ -58,8 +58,8 @@ module SimpleCov
     # historical `track_files` behavior) and restricts the report to the
     # matching set.
     def track_files(glob)
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov.track_files` is deprecated. " \
-           "#{track_files_replacement_hint(glob)}"
+      SimpleCov::Deprecation.warn("`SimpleCov.track_files` is deprecated. " \
+                                  "#{track_files_replacement_hint(glob)}")
       @tracked_files = glob
     end
 
@@ -102,8 +102,8 @@ module SimpleCov
     # DEPRECATED: alias for `skip`. Same matcher grammar, identical behavior.
     def add_filter(filter_argument = nil, &block)
       example = block ? "`SimpleCov.skip { ... }`" : "`SimpleCov.skip #{filter_argument.inspect}`"
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov.add_filter` is deprecated. " \
-           "Replace with `SimpleCov.skip` (same arguments, same behavior). Example: #{example}."
+      SimpleCov::Deprecation.warn("`SimpleCov.add_filter` is deprecated. " \
+                                  "Replace with `SimpleCov.skip` (same arguments, same behavior). Example: #{example}.")
       skip(filter_argument, &block)
     end
 
@@ -141,8 +141,10 @@ module SimpleCov
                 else
                   "`SimpleCov.group #{group_name.inspect}, #{filter_argument.inspect}`"
                 end
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov.add_group` is deprecated. " \
-           "Replace with `SimpleCov.group` (same arguments, same behavior). Example: #{example}."
+      SimpleCov::Deprecation.warn(
+        "`SimpleCov.add_group` is deprecated. " \
+        "Replace with `SimpleCov.group` (same arguments, same behavior). Example: #{example}."
+      )
       group(group_name, filter_argument, &block)
     end
 

data/lib/simplecov/configuration/formatting.rb

@@ -89,8 +89,8 @@ module SimpleCov
 
     # DEPRECATED: alias for `print_errors`. Same value, same behavior.
     def print_error_status
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov.print_error_status` is deprecated. " \
-           "Replace with `SimpleCov.print_errors` (same value)."
+      SimpleCov::Deprecation.warn("`SimpleCov.print_error_status` is deprecated. " \
+                                  "Replace with `SimpleCov.print_errors` (same value).")
       defined?(@print_error_status) ? @print_error_status : true
     end
 
@@ -101,8 +101,8 @@ module SimpleCov
     # be removed in a future release.
     #
     def nocov_token(nocov_token = nil)
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov.nocov_token` and `SimpleCov.skip_token` are deprecated. " \
-           "Replace with `# simplecov:disable` / `# simplecov:enable` block comments."
+      SimpleCov::Deprecation.warn("`SimpleCov.nocov_token` and `SimpleCov.skip_token` are deprecated. " \
+                                  "Replace with `# simplecov:disable` / `# simplecov:enable` block comments.")
       current_nocov_token(nocov_token)
     end
     alias skip_token nocov_token

data/lib/simplecov/configuration/merging.rb

@@ -36,8 +36,8 @@ module SimpleCov
 
     # DEPRECATED: alias for `merge_subprocesses`. Same value/behavior.
     def enable_for_subprocesses(value = nil)
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov.enable_for_subprocesses` is deprecated. " \
-           "Replace with `SimpleCov.merge_subprocesses` (same value, same behavior)."
+      SimpleCov::Deprecation.warn("`SimpleCov.enable_for_subprocesses` is deprecated. " \
+                                  "Replace with `SimpleCov.merge_subprocesses` (same value, same behavior).")
       return @enable_for_subprocesses if defined?(@enable_for_subprocesses) && value.nil?
 
       @enable_for_subprocesses = value || false
@@ -56,8 +56,8 @@ module SimpleCov
 
     # DEPRECATED: alias for `merging`. Same value, same behavior.
     def use_merging(use = nil)
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov.use_merging` is deprecated. " \
-           "Replace with `SimpleCov.merging` (same value, same behavior)."
+      SimpleCov::Deprecation.warn("`SimpleCov.use_merging` is deprecated. " \
+                                  "Replace with `SimpleCov.merging` (same value, same behavior).")
       @use_merging = use unless use.nil?
       @use_merging = true unless defined?(@use_merging) && @use_merging == false
     end

data/lib/simplecov/configuration/thresholds.rb

@@ -76,8 +76,8 @@ module SimpleCov
       coverage = {primary_coverage => coverage} if coverage.is_a?(Numeric)
       defaults, overrides = partition_per_file_thresholds(coverage)
 
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov.minimum_coverage_by_file` is deprecated. " \
-           "Replace it with:\n#{per_file_coverage_replacement(defaults, overrides)}"
+      SimpleCov::Deprecation.warn("`SimpleCov.minimum_coverage_by_file` is deprecated. " \
+                                  "Replace it with:\n#{per_file_coverage_replacement(defaults, overrides)}")
 
       raise_on_invalid_coverage(defaults, "minimum_coverage_by_file")
       overrides.each_value { |criteria| raise_on_invalid_coverage(criteria, "minimum_coverage_by_file") }
@@ -98,8 +98,8 @@ module SimpleCov
     def minimum_coverage_by_group(coverage = nil)
       return @minimum_coverage_by_group ||= {} unless coverage
 
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov.minimum_coverage_by_group` is deprecated. " \
-           "Replace it with:\n#{per_group_coverage_replacement(coverage)}"
+      SimpleCov::Deprecation.warn("`SimpleCov.minimum_coverage_by_group` is deprecated. " \
+                                  "Replace it with:\n#{per_group_coverage_replacement(coverage)}")
       @minimum_coverage_by_group = coverage.dup.transform_values do |group_coverage|
         group_coverage = {primary_coverage => group_coverage} if group_coverage.is_a?(Numeric)
         raise_on_invalid_coverage(group_coverage, "minimum_coverage_by_group")

data/lib/simplecov/configuration.rb

@@ -113,14 +113,19 @@ module SimpleCov
 
     #
     # Gets or sets the behavior to start a new forked Process.
-    # Defaults to adding " (subprocess: #{pid})" to command_name and
+    # Defaults to adding " (subprocess: #{serial})" to command_name and
     # starting SimpleCov in quiet mode.
     #
     def at_fork(&block)
       @at_fork = block if block
-      @at_fork ||= lambda { |pid|
-        # This needs a unique name so it won't be overwritten
-        SimpleCov.command_name "#{SimpleCov.command_name} (subprocess: #{pid})"
+      @at_fork ||= lambda { |_pid|
+        # Needs a name that's unique per worker within a run yet identical
+        # across runs. Build it from SimpleCov's stable fork serial rather
+        # than the OS pid: with the pid, every run produced uniquely-named
+        # results that never overwrote the previous run's, so they piled up
+        # in .resultset.json until merge_timeout and the merged report's
+        # file set drifted from run to run. See issue #1171.
+        SimpleCov.command_name "#{SimpleCov.command_name} (subprocess: #{SimpleCov.subprocess_serial})"
         # be quiet, the parent process will use the regular formatter
         SimpleCov.print_errors false
         SimpleCov.formatter SimpleCov::Formatter::SimpleFormatter

data/lib/simplecov/deprecation.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+require "set"
+
+module SimpleCov
+  # Emits legacy-API deprecation warnings, deduplicated by the source
+  # location that triggered them. A deprecated method called in a loop —
+  # or a config block re-evaluated once per parallel worker / spec file —
+  # otherwise repeats the same notice until stderr is unreadable. Keying
+  # on the caller location collapses those repeats to a single line while
+  # still warning separately about each distinct call site the user needs
+  # to fix. See issue #1204.
+  module Deprecation
+  module_function
+
+    # Warn about a deprecated API. `message` is the notice without the
+    # `[DEPRECATION]` tag or location prefix (both are added here).
+    #
+    # `location` defaults to the caller of the deprecated method that
+    # called us — every shipped call site is a one-level alias such as
+    # `track_files`, so the frame two up is the user code. Pass `location:`
+    # explicitly when the relevant site isn't that frame (e.g. a source
+    # file and line discovered while parsing).
+    # `Array(...)` coerces a missing backtrace (nil) to `[]` so `.first`
+    # yields nil rather than raising — and, unlike `&.`, adds no branch for
+    # the unreachable no-caller case to the project's 100% coverage target.
+    def warn(message, location: Array(Kernel.caller(2..2)).first)
+      # Key on location when we have one (collapses a deprecated call in a
+      # loop to a single warning); fall back to the message so a missing
+      # backtrace never silently swallows every notice.
+      return unless emitted.add?(location || message)
+
+      Kernel.warn "#{"#{location}: " if location}[DEPRECATION] #{message}"
+    end
+
+    # Already-emitted dedup keys for this process. Parallel workers are
+    # separate processes with their own set, so each warns at most once.
+    def emitted
+      @emitted ||= Set.new
+    end
+
+    # @api private — reset emitted state between tests.
+    def reset!
+      @emitted = Set.new
+    end
+  end
+end

data/lib/simplecov/process.rb

@@ -15,8 +15,14 @@ module SimpleCov
   # the child.
   module ProcessForkHook
     def _fork
+      active = defined?(SimpleCov) && Coverage.running?
+      # Assign the next serial in the PARENT, before the fork, so the child
+      # inherits its own stable ordinal via copy-on-write. The default
+      # at_fork uses it (instead of the child pid) to name the subprocess's
+      # result, keeping that name identical across runs. See issue #1171.
+      SimpleCov.next_subprocess_serial! if active
       pid = super
-      SimpleCov.at_fork.call(::Process.pid) if pid.zero? && defined?(SimpleCov) && Coverage.running?
+      SimpleCov.at_fork.call(::Process.pid) if pid.zero? && active
       pid
     end
   end

data/lib/simplecov/result_merger.rb

@@ -68,6 +68,14 @@ module SimpleCov
       end
 
       def warn_about_expired_results(expired_command_names)
+        # Subprocesses merge the resultset too (each forked worker calls
+        # `SimpleCov.result` to store its slice), and the default `at_fork`
+        # sets `print_errors false` for them. Without this guard the warning
+        # is emitted once per worker — N copies of the same message for an
+        # N-worker run. Gate on `print_errors` like every other SimpleCov
+        # warning so only the reporting process speaks up.
+        return unless SimpleCov.print_errors
+
         warn "[SimpleCov]: Excluded #{expired_command_names.size} result(s) older than " \
              "merge_timeout (#{SimpleCov.merge_timeout}s) from the merged report: " \
              "#{expired_command_names.sort.join(', ')}. " \

data/lib/simplecov/source_file.rb

@@ -123,8 +123,8 @@ module SimpleCov
 
     # DEPRECATED: use `covered_percent(:branch)`.
     def branches_coverage_percent
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov::SourceFile#branches_coverage_percent` is deprecated. " \
-           "Use `covered_percent(:branch)`."
+      SimpleCov::Deprecation.warn("`SimpleCov::SourceFile#branches_coverage_percent` is deprecated. " \
+                                  "Use `covered_percent(:branch)`.")
       covered_percent(:branch)
     end
 
@@ -176,8 +176,8 @@ module SimpleCov
 
     # DEPRECATED: use `covered_percent(:method)`.
     def methods_coverage_percent
-      warn "#{Kernel.caller.first}: [DEPRECATION] `SimpleCov::SourceFile#methods_coverage_percent` is deprecated. " \
-           "Use `covered_percent(:method)`."
+      SimpleCov::Deprecation.warn("`SimpleCov::SourceFile#methods_coverage_percent` is deprecated. " \
+                                  "Use `covered_percent(:method)`.")
       covered_percent(:method)
     end
 

data/lib/simplecov/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SimpleCov
-  VERSION = "1.0.0.rc1"
+  VERSION = "1.0.0.rc2"
 end

data/lib/simplecov.rb

@@ -21,6 +21,22 @@ module SimpleCov
     # so JSONFormatter can detect when an existing coverage.json was written
     # by a sibling process running concurrently.
     attr_accessor :process_start_time
+
+    # A monotonically increasing serial the parent assigns to each forked
+    # subprocess (see SimpleCov::ProcessForkHook). The default `at_fork`
+    # builds the worker's command_name from this rather than the OS pid:
+    # the serial sequence is the same from one run to the next, so a re-run
+    # overwrites the previous run's resultset entries instead of writing
+    # uniquely-named ones that pile up until merge_timeout. See issue #1171.
+    def subprocess_serial
+      @subprocess_serial ||= 0
+    end
+
+    # @api private — bump the serial in the parent before a fork so the
+    # child inherits its own ordinal via copy-on-write.
+    def next_subprocess_serial!
+      @subprocess_serial = subprocess_serial + 1
+    end
     # Should we take care of at_exit behavior or something else? Used by the
     # minitest plugin. See lib/minitest/simplecov_plugin.rb.
     attr_accessor :external_at_exit
@@ -186,6 +202,7 @@ end
 require "set"
 require "forwardable"
 require_relative "simplecov/color"
+require_relative "simplecov/deprecation"
 require_relative "simplecov/configuration"
 SimpleCov.extend SimpleCov::Configuration
 require_relative "simplecov/coverage_statistics"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: simplecov
 version: !ruby/object:Gem::Version
-  version: 1.0.0.rc1
+  version: 1.0.0.rc2
 platform: ruby
 authors:
 - Erik Berlin
@@ -71,6 +71,7 @@ files:
 - lib/simplecov/coverage_statistics.rb
 - lib/simplecov/coverage_violations.rb
 - lib/simplecov/defaults.rb
+- lib/simplecov/deprecation.rb
 - lib/simplecov/directive.rb
 - lib/simplecov/exit_codes.rb
 - lib/simplecov/exit_codes/exit_code_handling.rb
@@ -149,9 +150,9 @@ licenses:
 metadata:
   bug_tracker_uri: https://github.com/simplecov-ruby/simplecov/issues
   changelog_uri: https://github.com/simplecov-ruby/simplecov/blob/main/CHANGELOG.md
-  documentation_uri: https://www.rubydoc.info/gems/simplecov/1.0.0.rc1
+  documentation_uri: https://www.rubydoc.info/gems/simplecov/1.0.0.rc2
   mailing_list_uri: https://groups.google.com/forum/#!forum/simplecov
-  source_code_uri: https://github.com/simplecov-ruby/simplecov/tree/v1.0.0.rc1
+  source_code_uri: https://github.com/simplecov-ruby/simplecov/tree/v1.0.0.rc2
   rubygems_mfa_required: 'true'
 rdoc_options: []
 require_paths:
@@ -167,7 +168,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.12
+rubygems_version: 4.0.14
 specification_version: 4
 summary: Code coverage for Ruby
 test_files: []