rspec-tracer 1.0.0 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9aaeb8a00f4cebc311367e6a61d59526014849df9f7fe890455d64cef16d435f
-  data.tar.gz: b803d5104a0a35f783bdd15b7a43252a1cfddcdd56d9f0a26c5a07bb8c36ef5e
+  metadata.gz: 57dd6b7ba2dafd73bbe30d91e42c36498c7e4fb505e02c405825cf45d8ac0787
+  data.tar.gz: e8886b116e2412f566ae908d3f0e2d9213df9ff70e0c88d54a8a4fb28dab9788
 SHA512:
-  metadata.gz: 56acd78d6d4bf7b6270e8517a82d0bb1237dd78518a22d1a84a96de29e00c59379092cd725cc276d3f59ca2916306835c38426917747dcaf278e45d611c99389
-  data.tar.gz: a4e026b2c6fdaad653bd48063c9d0b771e7e7bc8876f653cc2e32279930d6a043b546ab1c7a713e12aac934f1f069497decdb5856246d4ee8033e905577be1db
+  metadata.gz: 88ea3fbe3fef4c4ab2b2d2664ac5915b4e7eaa97b5ecc0d2b77dfeb03604e7f54faf4a567f65ff31235464f351a60e8817527d51f64f5a2717bb267f1cf72261
+  data.tar.gz: fbba1feed081e49a4cef7338c05b6d1022ecac5f013b499150b42b817e1cdb13bf2dc29ad993a502d8233bfe52dc89f991ad036eaeb513cde0b5d029b6413a46

data/CHANGELOG.md

@@ -1,3 +1,113 @@
+## [1.0.2] - 2026-05-01
+
+### Fixed
+
+- **Parallel-tests merge silently dropped peer caches and left worker
+  directories behind** when the spawned-worker count exceeded
+  `ENV['PARALLEL_TEST_GROUPS']`. The merge + purge call-sites in
+  `lib/rspec_tracer.rb` (`merge_parallel_tests_reports`,
+  `merge_parallel_tests_coverage_reports`,
+  `purge_parallel_tests_reports`) iterated `1..ENV['PARALLEL_TEST_GROUPS']`
+  to construct per-worker directory names. But parallel_tests sets
+  `PARALLEL_TEST_GROUPS = num_processes.to_s` for each child, where
+  `num_processes` is the user-requested process count
+  (`Parallel.processor_count` by default) — not the actual worker
+  count. When `num_processes < spawned_worker_count` (e.g. when the
+  spec-count partition produces more non-empty groups than
+  `num_processes`, or shared-runner CPU detection drifts mid-run),
+  peer caches with `TEST_ENV_NUMBER` above the env bound were silently
+  dropped from the merge (warm-run skip decisions get made against
+  an under-sampled merged manifest) and left behind by the purge
+  (visible as straggler `parallel_tests_<N>/` directories under
+  `rspec_tracer_cache/`). The same gem behaviour was documented on
+  v1.1.1's `last_process?` fix
+  ([PR #101](https://github.com/avmnu-sng/rspec-tracer/pull/101)) but
+  the iteration call-sites kept the buggy bound. Each method now globs
+  the actual `parallel_tests_*` subdirectories under its base path,
+  making the merge + purge robust to whatever count parallel_tests
+  spawned. (from v1.2.1) No cache format change.
+
+## [1.0.1] - 2026-04-24
+
+Long-tail maintenance release. Backports high-impact crash and correctness
+fixes from 1.1.x / 1.2.x onto the v1.0.0 foundation so users on Ruby 2.5 -
+3.0 who can't upgrade can still pick them up via `gem 'rspec-tracer', '~> 1.0'`.
+
+### Fixed
+
+- `Cache#cached_examples_coverage` returns `{}` (not `nil`) when `last_run.json`
+  is present but `examples_coverage.json` is missing, preventing a
+  NoMethodError on the next consumer. (B1, from v1.1.0)
+- `Runner#generate_missed_coverage` tolerates a nil cached coverage map and
+  nil per-line strength entries. (B2, from v1.1.0)
+- `Runner#register_file_dependency` / `#register_example_file_dependency`
+  skip gracefully when `SourceFile.from_path` / `.from_name` resolves to
+  nil (e.g. gem-generated examples whose path is absent at runtime),
+  instead of crashing. (B3, from v1.1.0)
+- `CoverageReporter#merge_coverage` treats a nil existing line-coverage
+  entry as 0 when summing skipped-test contributions. (B4, from v1.1.0)
+- `parallel_tests`-mode merge-worker election no longer deadlocks under
+  slow CI: the elected worker is now picked via
+  `::ParallelTests.first_process?` (immutable at spawn) instead of the
+  lock-file max TEST_ENV_NUMBER (racy on slow runners). (from v1.1.1)
+- Pin `encoding: 'UTF-8'` on every legacy `File.read` / `File.write` JSON
+  I/O site so shells with `LANG=` unset no longer crash the tracer on
+  multibyte spec descriptions. Cache (11), report_writer (12), and nine
+  other lib/ call sites covered. (from v1.1.2)
+- `SourceFile.from_path` computes the file digest via `File.binread`, hashing
+  raw bytes regardless of Encoding.default_external. (from v1.1.2)
+- `SourceFile.file_path` returns an absolute-external path unchanged when
+  the referenced file exists on disk (e.g. shared examples from vendored
+  gems at `/opt/bundle/gems/...`), preventing silent drop of dependency
+  registration. The guard is narrow — `start_with?('/') &&
+  !start_with?(root) && File.file?(path)` — so stripped-root cache forms
+  like `/spec/foo.rb` continue through the existing expand_path branch and
+  cache `file_name` keys stay byte-identical to v1.0.0. (C1, from v1.2.0)
+- `RemoteCache::Aws#upload_dir` error message corrected from
+  "Failed to download files from" to "Failed to upload files from". (C3,
+  from v1.2.0)
+- `RemoteCache::Validator::ValidationError` declared as a proper
+  `StandardError` subclass inside `Validator`; previously the
+  `TEST_SUITE_ID ^ TEST_SUITES` XOR-guard raised a `NameError:
+  uninitialized constant` instead of the intended validation error.
+  (from v1.2.0)
+- `enviornment` → `environment` typo in the same XOR-guard raise message.
+  (from v1.2.0)
+- `RemoteCache::Validator`'s single-suite `@cached_files_regex` anchored
+  with a trailing `$` so files with extensions beyond `.json` (e.g.
+  `.json.backup`) no longer match as cache files. (from v1.2.0)
+- `RemoteCache::Repo#initialize` guards `ENV['GIT_BRANCH']` for nil
+  before calling `.chomp`; previously a `NoMethodError: undefined
+  method 'chomp' for nil:NilClass` crashed the init path and masked
+  the intended `RepoError` message when `GIT_BRANCH` was not set in
+  the environment. (from v1.1.0 PR #51)
+- `RemoteCache::Repo#download_branch_refs` uses `FileUtils.rm_f`
+  (not `File.rm_f`) to clean up a partial `branch_refs.json` on a
+  failed AWS download. `File.rm_f` is undefined — `rm_f` is a
+  FileUtils method — so the failing-download branch would crash with
+  `NoMethodError` instead of cleaning up and logging. (from v1.1.0
+  PR #65)
+
+### Note on exclusions
+
+The following items from 1.1.x / 1.2.0 are intentionally NOT in this
+release to preserve the 1.0.0 cache-format contract and Ruby 2.5+ floor:
+
+- Default-filter expansion (1.1.0 — adds `/lib/rspec_tracer/`,
+  `/usr/local/lib/ruby/`, etc. to the default filters). Changing the
+  default filter set shifts the files present in `all_files.json` for
+  most users, which would invalidate existing caches on upgrade.
+- `USE_TEST_SUITE_ID_CACHE` opt-in ENV flag (1.2.0). This is a new
+  feature, not a bug fix; users who want it can upgrade to 1.2.x.
+- The 1.1+ configuration DSL refactor (anonymous block forwarding,
+  alias_method wrapping, ENV.fetch normalizations). These are
+  Ruby-3.1-exclusive in places and orthogonal to the crash-fix scope.
+
+### Ruby support
+
+Gemspec `required_ruby_version` unchanged at `>= 2.5.0`. CI gates
+Ruby 2.5 - 4.0 inclusive on `ubuntu-latest`.
+
 ## [1.0.0] - 2021-10-21
 
 ### Added

data/lib/rspec_tracer/cache.rb

@@ -17,6 +17,7 @@ module RSpecTracer
       @pending_examples = Set.new
       @all_files = {}
       @dependency = Hash.new { |hash, key| hash[key] = Set.new }
+      @examples_coverage = {}
     end
 
     def load_cache_for_run
@@ -50,23 +51,25 @@ module RSpecTracer
     end
 
     def cached_examples_coverage
-      return @examples_coverage if defined?(@examples_coverage)
+      return @examples_coverage if @examples_coverage_loaded
+
+      @examples_coverage_loaded = true
 
       cache_path = RSpecTracer.cache_path
       cache_path = File.dirname(cache_path) if RSpecTracer.parallel_tests?
       run_id = last_run_id(cache_path)
 
-      return @examples_coverage = {} if run_id.nil?
+      return @examples_coverage if run_id.nil?
 
       starting = Process.clock_gettime(Process::CLOCK_MONOTONIC)
       cache_dir = File.join(cache_path, run_id)
-      coverage = load_examples_coverage_cache(cache_dir)
+      load_examples_coverage_cache(cache_dir)
       ending = Process.clock_gettime(Process::CLOCK_MONOTONIC)
       elapsed = RSpecTracer::TimeFormatter.format_time(ending - starting)
 
       puts "RSpec tracer loaded cached examples coverage (took #{elapsed})" if RSpecTracer.verbose?
 
-      coverage
+      @examples_coverage
     end
 
     private
@@ -76,7 +79,7 @@ module RSpecTracer
 
       return unless File.file?(file_name)
 
-      JSON.parse(File.read(file_name))['run_id']
+      JSON.parse(File.read(file_name, encoding: 'UTF-8'))['run_id']
     end
 
     def load_all_examples_cache(cache_dir, discard_run_reason: true)
@@ -84,7 +87,7 @@ module RSpecTracer
 
       return unless File.file?(file_name)
 
-      @all_examples = JSON.parse(File.read(file_name)).transform_values do |examples|
+      @all_examples = JSON.parse(File.read(file_name, encoding: 'UTF-8')).transform_values do |examples|
         examples.transform_keys(&:to_sym)
       end
 
@@ -99,7 +102,7 @@ module RSpecTracer
 
       return unless File.file?(file_name)
 
-      @duplicate_examples = JSON.parse(File.read(file_name)).transform_values do |examples|
+      @duplicate_examples = JSON.parse(File.read(file_name, encoding: 'UTF-8')).transform_values do |examples|
         examples.map { |example| example.transform_keys(&:to_sym) }
       end
     end
@@ -109,7 +112,7 @@ module RSpecTracer
 
       return unless File.file?(file_name)
 
-      @interrupted_examples = JSON.parse(File.read(file_name)).to_set
+      @interrupted_examples = JSON.parse(File.read(file_name, encoding: 'UTF-8')).to_set
     end
 
     def load_flaky_examples_cache(cache_dir)
@@ -117,7 +120,7 @@ module RSpecTracer
 
       return unless File.file?(file_name)
 
-      @flaky_examples = JSON.parse(File.read(file_name)).to_set
+      @flaky_examples = JSON.parse(File.read(file_name, encoding: 'UTF-8')).to_set
     end
 
     def load_failed_examples_cache(cache_dir)
@@ -125,7 +128,7 @@ module RSpecTracer
 
       return unless File.file?(file_name)
 
-      @failed_examples = JSON.parse(File.read(file_name)).to_set
+      @failed_examples = JSON.parse(File.read(file_name, encoding: 'UTF-8')).to_set
     end
 
     def load_pending_examples_cache(cache_dir)
@@ -133,7 +136,7 @@ module RSpecTracer
 
       return unless File.file?(file_name)
 
-      @pending_examples = JSON.parse(File.read(file_name)).to_set
+      @pending_examples = JSON.parse(File.read(file_name, encoding: 'UTF-8')).to_set
     end
 
     def load_skipped_examples_cache(cache_dir)
@@ -141,7 +144,7 @@ module RSpecTracer
 
       return unless File.file?(file_name)
 
-      @skipped_examples = JSON.parse(File.read(file_name)).to_set
+      @skipped_examples = JSON.parse(File.read(file_name, encoding: 'UTF-8')).to_set
     end
 
     def load_all_files_cache(cache_dir)
@@ -149,7 +152,7 @@ module RSpecTracer
 
       return unless File.file?(file_name)
 
-      @all_files = JSON.parse(File.read(file_name)).transform_values do |files|
+      @all_files = JSON.parse(File.read(file_name, encoding: 'UTF-8')).transform_values do |files|
         files.transform_keys(&:to_sym)
       end
     end
@@ -159,7 +162,7 @@ module RSpecTracer
 
       return unless File.file?(file_name)
 
-      @dependency = JSON.parse(File.read(file_name)).transform_values(&:to_set)
+      @dependency = JSON.parse(File.read(file_name, encoding: 'UTF-8')).transform_values(&:to_set)
     end
 
     def load_examples_coverage_cache(cache_dir)
@@ -167,7 +170,7 @@ module RSpecTracer
 
       return unless File.file?(file_name)
 
-      @examples_coverage = JSON.parse(File.read(file_name))
+      @examples_coverage = JSON.parse(File.read(file_name, encoding: 'UTF-8'))
     end
   end
 end

data/lib/rspec_tracer/configuration.rb

@@ -25,7 +25,7 @@ module RSpecTracer
     end
 
     def cache_dir
-      @cache_dir ||= (ENV['RSPEC_TRACER_CACHE_DIR'] || DEFAULT_CACHE_DIR)
+      @cache_dir ||= ENV['RSPEC_TRACER_CACHE_DIR'] || DEFAULT_CACHE_DIR
     end
 
     def cache_path
@@ -41,7 +41,7 @@ module RSpecTracer
     end
 
     def report_dir
-      @report_dir ||= (ENV['RSPEC_TRACER_REPORT_DIR'] || DEFAULT_REPORT_DIR)
+      @report_dir ||= ENV['RSPEC_TRACER_REPORT_DIR'] || DEFAULT_REPORT_DIR
     end
 
     def report_path
@@ -57,7 +57,7 @@ module RSpecTracer
     end
 
     def coverage_dir
-      @coverage_dir ||= (ENV['RSPEC_TRACER_COVERAGE_DIR'] || DEFAULT_COVERAGE_DIR)
+      @coverage_dir ||= ENV['RSPEC_TRACER_COVERAGE_DIR'] || DEFAULT_COVERAGE_DIR
     end
 
     def coverage_path
@@ -114,7 +114,7 @@ module RSpecTracer
       if ParallelTests.first_process?
         'parallel_tests_1'
       else
-        "parallel_tests_#{ENV['TEST_ENV_NUMBER']}"
+        "parallel_tests_#{ENV.fetch('TEST_ENV_NUMBER', nil)}"
       end
     end
 

data/lib/rspec_tracer/coverage_merger.rb

@@ -14,7 +14,8 @@ module RSpecTracer
       reports_dir.each do |report_dir|
         next unless File.directory?(report_dir)
 
-        cache_coverage = JSON.parse(File.read("#{report_dir}/coverage.json"))['RSpecTracer']['coverage']
+        cache_coverage = JSON.parse(File.read("#{report_dir}/coverage.json",
+                                              encoding: 'UTF-8'))['RSpecTracer']['coverage']
 
         cache_coverage.each_pair do |file_name, line_coverage|
           unless @coverage.key?(file_name)

data/lib/rspec_tracer/coverage_reporter.rb

@@ -72,7 +72,8 @@ module RSpecTracer
                             end
 
         line_coverage.each_pair do |line_number, strength|
-          line_coverage_dup[line_number.to_i] += strength
+          index = line_number.to_i
+          line_coverage_dup[index] = (line_coverage_dup[index] || 0) + strength
         end
 
         @coverage[file_path] = line_coverage_dup.freeze
@@ -164,7 +165,7 @@ module RSpecTracer
 
     def jruby_line_stub(file_path)
       lines = File.foreach(file_path).map { nil }
-      root_node = ::JRuby.parse(File.read(file_path))
+      root_node = ::JRuby.parse(File.read(file_path, encoding: 'UTF-8'))
 
       visitor = org.jruby.ast.visitor.NodeVisitor.impl do |_name, node|
         if node.newline?

data/lib/rspec_tracer/coverage_writer.rb

@@ -15,7 +15,7 @@ module RSpecTracer
         }
       }
 
-      File.write(@file_name, JSON.pretty_generate(report))
+      File.write(@file_name, JSON.pretty_generate(report), encoding: 'UTF-8')
     end
 
     def print_stats(elapsed_time)

data/lib/rspec_tracer/filter.rb

@@ -4,14 +4,6 @@ module RSpecTracer
   class Filter
     attr_reader :filter
 
-    def initialize(filter)
-      @filter = filter
-    end
-
-    def match?(_source_file)
-      raise "#{self.class.name}#match? is not intended for direct use"
-    end
-
     def self.register(filter)
       return filter if filter.is_a?(Filter)
 
@@ -32,6 +24,14 @@ module RSpecTracer
         raise ArgumentError, 'Unknow filter'
       end
     end
+
+    def initialize(filter)
+      @filter = filter
+    end
+
+    def match?(_source_file)
+      raise "#{self.class.name}#match? is not intended for direct use"
+    end
   end
 
   class ArrayFilter < RSpecTracer::Filter

data/lib/rspec_tracer/html_reporter/reporter.rb

@@ -211,7 +211,7 @@ module RSpecTracer
       end
 
       def template(name)
-        ERB.new(File.read(File.join(File.dirname(__FILE__), 'views/', "#{name}.erb")))
+        ERB.new(File.read(File.join(File.dirname(__FILE__), 'views/', "#{name}.erb"), encoding: 'UTF-8'))
       end
 
       def example_status_css_class(example_status)

data/lib/rspec_tracer/remote_cache/aws.rb

@@ -125,7 +125,7 @@ module RSpecTracer
         remote_dir = s3_dir(ref, run_id)
         local_dir = File.join(RSpecTracer.cache_path, run_id)
 
-        raise AwsError, "Failed to download files from #{local_dir}" unless system(
+        raise AwsError, "Failed to upload files from #{local_dir}" unless system(
           @aws_cli,
           's3',
           'cp',
@@ -142,7 +142,7 @@ module RSpecTracer
       private
 
       def setup_s3
-        s3_uri = ENV['RSPEC_TRACER_S3_URI']
+        s3_uri = ENV.fetch('RSPEC_TRACER_S3_URI', nil)
 
         raise AwsError, 'RSPEC_TRACER_S3_URI environment variable is not set' if s3_uri.nil?
 
@@ -165,7 +165,7 @@ module RSpecTracer
       end
 
       def s3_dir(ref, run_id = nil)
-        test_suite_id = ENV['TEST_SUITE_ID']
+        test_suite_id = ENV.fetch('TEST_SUITE_ID', nil)
 
         if test_suite_id.nil?
           "s3://#{@s3_bucket}/#{@s3_path}/#{ref}/#{run_id}/".sub(%r{/+$}, '/')

data/lib/rspec_tracer/remote_cache/cache.rb

@@ -64,7 +64,7 @@ module RSpecTracer
 
         ref_list = @repo.branch_refs.merge(@repo.branch_ref => branch_ref_time.to_i)
 
-        File.write(file_name, JSON.pretty_generate(ref_list))
+        File.write(file_name, JSON.pretty_generate(ref_list), encoding: 'UTF-8')
       end
 
       def last_run_id
@@ -72,7 +72,7 @@ module RSpecTracer
 
         raise CacheError, 'Could not find any local cache to upload' unless File.file?(file_name)
 
-        JSON.parse(File.read(file_name))['run_id']
+        JSON.parse(File.read(file_name, encoding: 'UTF-8'))['run_id']
       end
     end
   end

data/lib/rspec_tracer/remote_cache/repo.rb

@@ -8,11 +8,11 @@ module RSpecTracer
       attr_reader :branch_name, :branch_ref, :branch_refs, :ancestry_refs, :cache_refs
 
       def initialize(aws)
+        raise RepoError, 'GIT_BRANCH environment variable is not set' if ENV['GIT_BRANCH'].nil?
+
         @aws = aws
         @branch_name = ENV['GIT_BRANCH'].chomp
 
-        raise RepoError, 'GIT_BRANCH environment variable is not set' if @branch_name.nil?
-
         fetch_head_ref
         fetch_branch_ref
         fetch_ancestry_refs
@@ -135,7 +135,7 @@ module RSpecTracer
         file_name = File.join(RSpecTracer.cache_path, 'branch_refs.json')
 
         if @aws.download_branch_refs(branch_name, file_name)
-          @branch_refs = JSON.parse(File.read(file_name)).transform_values(&:to_i)
+          @branch_refs = JSON.parse(File.read(file_name, encoding: 'UTF-8')).transform_values(&:to_i)
 
           return if @branch_refs.empty?
 
@@ -144,7 +144,7 @@ module RSpecTracer
         else
           @branch_refs = {}
 
-          File.rm_f(file_name)
+          FileUtils.rm_f(file_name)
 
           puts "Failed to fetch branch refs for #{@branch_name} branch"
         end

data/lib/rspec_tracer/remote_cache/validator.rb

@@ -3,16 +3,18 @@
 module RSpecTracer
   module RemoteCache
     class Validator
+      class ValidationError < StandardError; end
+
       CACHE_FILES_PER_TEST_SUITE = 11
 
       def initialize
-        @test_suite_id = ENV['TEST_SUITE_ID']
-        @test_suites = ENV['TEST_SUITES']
+        @test_suite_id = ENV.fetch('TEST_SUITE_ID', nil)
+        @test_suites = ENV.fetch('TEST_SUITES', nil)
 
         if @test_suite_id.nil? ^ @test_suites.nil?
           raise(
             ValidationError,
-            'Both the enviornment variables TEST_SUITE_ID and TEST_SUITES are not set'
+            'Both the environment variables TEST_SUITE_ID and TEST_SUITES are not set'
           )
         end
 
@@ -36,7 +38,7 @@ module RSpecTracer
           @last_run_files_count = 1
           @last_run_files_regex = '/%<ref>s/last_run.json$'
           @cached_files_count = CACHE_FILES_PER_TEST_SUITE
-          @cached_files_regex = '/%<ref>s/[0-9a-f]{32}/.+.json'
+          @cached_files_regex = '/%<ref>s/[0-9a-f]{32}/.+.json$'
         else
           @test_suites = @test_suites.to_i
           @test_suites_regex = (1..@test_suites).to_a.join('|')

data/lib/rspec_tracer/report_merger.rb

@@ -70,7 +70,7 @@ module RSpecTracer
 
     def merge_last_run_report(cache_dir)
       file_name = File.join(cache_dir, 'last_run.json')
-      cached_last_run = JSON.parse(File.read(file_name), symbolize_names: true)
+      cached_last_run = JSON.parse(File.read(file_name, encoding: 'UTF-8'), symbolize_names: true)
       cached_last_run[:pid] = [cached_last_run[:pid]]
 
       cached_last_run.delete_if { |key, _| %i[run_id timestamp].include?(key) }

data/lib/rspec_tracer/report_writer.rb

@@ -68,74 +68,74 @@ module RSpecTracer
     def write_all_examples_report
       file_name = File.join(@cache_dir, 'all_examples.json')
 
-      File.write(file_name, JSON.pretty_generate(@reporter.all_examples))
+      File.write(file_name, JSON.pretty_generate(@reporter.all_examples), encoding: 'UTF-8')
     end
 
     def write_duplicate_examples_report
       file_name = File.join(@cache_dir, 'duplicate_examples.json')
 
-      File.write(file_name, JSON.pretty_generate(@reporter.duplicate_examples))
+      File.write(file_name, JSON.pretty_generate(@reporter.duplicate_examples), encoding: 'UTF-8')
     end
 
     def write_interrupted_examples_report
       file_name = File.join(@cache_dir, 'interrupted_examples.json')
 
-      File.write(file_name, JSON.pretty_generate(@reporter.interrupted_examples.sort.to_a))
+      File.write(file_name, JSON.pretty_generate(@reporter.interrupted_examples.sort.to_a), encoding: 'UTF-8')
     end
 
     def write_flaky_examples_report
       file_name = File.join(@cache_dir, 'flaky_examples.json')
 
-      File.write(file_name, JSON.pretty_generate(@reporter.flaky_examples.sort.to_a))
+      File.write(file_name, JSON.pretty_generate(@reporter.flaky_examples.sort.to_a), encoding: 'UTF-8')
     end
 
     def write_failed_examples_report
       file_name = File.join(@cache_dir, 'failed_examples.json')
 
-      File.write(file_name, JSON.pretty_generate(@reporter.failed_examples.sort.to_a))
+      File.write(file_name, JSON.pretty_generate(@reporter.failed_examples.sort.to_a), encoding: 'UTF-8')
     end
 
     def write_pending_examples_report
       file_name = File.join(@cache_dir, 'pending_examples.json')
 
-      File.write(file_name, JSON.pretty_generate(@reporter.pending_examples.sort.to_a))
+      File.write(file_name, JSON.pretty_generate(@reporter.pending_examples.sort.to_a), encoding: 'UTF-8')
     end
 
     def write_skipped_examples_report
       file_name = File.join(@cache_dir, 'skipped_examples.json')
 
-      File.write(file_name, JSON.pretty_generate(@reporter.skipped_examples.sort.to_a))
+      File.write(file_name, JSON.pretty_generate(@reporter.skipped_examples.sort.to_a), encoding: 'UTF-8')
     end
 
     def write_all_files_report
       file_name = File.join(@cache_dir, 'all_files.json')
 
-      File.write(file_name, JSON.pretty_generate(@reporter.all_files))
+      File.write(file_name, JSON.pretty_generate(@reporter.all_files), encoding: 'UTF-8')
     end
 
     def write_dependency_report
       file_name = File.join(@cache_dir, 'dependency.json')
 
-      File.write(file_name, JSON.pretty_generate(@reporter.dependency))
+      File.write(file_name, JSON.pretty_generate(@reporter.dependency), encoding: 'UTF-8')
     end
 
     def write_reverse_dependency_report
       file_name = File.join(@cache_dir, 'reverse_dependency.json')
 
-      File.write(file_name, JSON.pretty_generate(@reporter.reverse_dependency))
+      File.write(file_name, JSON.pretty_generate(@reporter.reverse_dependency), encoding: 'UTF-8')
     end
 
     def write_examples_coverage_report
       file_name = File.join(@cache_dir, 'examples_coverage.json')
 
-      File.write(file_name, JSON.pretty_generate(@reporter.examples_coverage))
+      File.write(file_name, JSON.pretty_generate(@reporter.examples_coverage), encoding: 'UTF-8')
     end
 
     def write_last_run_report
       file_name = File.join(@report_dir, 'last_run.json')
       last_run_data = @reporter.last_run.merge(run_id: @run_id, timestamp: Time.now.utc)
 
-      File.write(file_name, JSON.pretty_generate(last_run_data))
+      File.write(file_name, JSON.pretty_generate(last_run_data), encoding: 'UTF-8')
     end
   end
 end

data/lib/rspec_tracer/rspec_reporter.rb

@@ -6,7 +6,7 @@ module RSpecTracer
       RSpecTracer.coverage_reporter.record_coverage
       RSpecTracer.start_example_trace if RSpecTracer.trace_example?
 
-      super(example)
+      super
     end
 
     def example_finished(example)
@@ -16,28 +16,28 @@ module RSpecTracer
       example_id = example.metadata[:rspec_tracer_example_id]
       RSpecTracer.coverage_reporter.compute_diff(example_id)
 
-      super(example)
+      super
     end
 
     def example_passed(example)
       example_id = example.metadata[:rspec_tracer_example_id]
       RSpecTracer.runner.on_example_passed(example_id, example.execution_result)
 
-      super(example)
+      super
     end
 
     def example_failed(example)
       example_id = example.metadata[:rspec_tracer_example_id]
       RSpecTracer.runner.on_example_failed(example_id, example.execution_result)
 
-      super(example)
+      super
     end
 
     def example_pending(example)
       example_id = example.metadata[:rspec_tracer_example_id]
       RSpecTracer.runner.on_example_pending(example_id, example.execution_result)
 
-      super(example)
+      super
     end
   end
 end

data/lib/rspec_tracer/rspec_runner.rb

@@ -9,7 +9,7 @@ module RSpecTracer
       if RSpecTracer.no_examples
         RSpecTracer.running = true
 
-        super(example_groups)
+        super
 
         return
       end

data/lib/rspec_tracer/runner.rb

@@ -77,7 +77,7 @@ module RSpecTracer
         end
       end
 
-      @cache.cached_examples_coverage.each_pair do |example_id, example_coverage|
+      (@cache.cached_examples_coverage || {}).each_pair do |example_id, example_coverage|
         example_coverage.each_pair do |file_path, line_coverage|
           next if @reporter.example_interrupted?(example_id) ||
             @reporter.duplicate_example?(example_id)
@@ -89,7 +89,7 @@ module RSpecTracer
           next if @reporter.file_deleted?(file_name)
 
           line_coverage.each_pair do |line_number, strength|
-            missed_coverage[file_path][line_number] += strength
+            missed_coverage[file_path][line_number] += strength || 0
           end
         end
       end
@@ -271,6 +271,11 @@ module RSpecTracer
 
       source_file = RSpecTracer::SourceFile.from_name(file_name)
 
+      if source_file.nil?
+        puts "Skipping missing source file #{file_name} for example #{example_id}" if RSpecTracer.verbose?
+        return
+      end
+
       @reporter.register_source_file(source_file)
       @reporter.register_dependency(example_id, file_name)
     end
@@ -281,6 +286,11 @@ module RSpecTracer
 
       source_file = RSpecTracer::SourceFile.from_path(file_path)
 
+      if source_file.nil?
+        puts "Skipping missing source file #{file_path} for example #{example_id}" if RSpecTracer.verbose?
+        return false
+      end
+
       return false if RSpecTracer.filters.any? { |filter| filter.match?(source_file) }
 
       @reporter.register_source_file(source_file)

data/lib/rspec_tracer/source_file.rb

@@ -12,7 +12,7 @@ module RSpecTracer
       {
         file_path: file_path,
         file_name: file_name(file_path),
-        digest: Digest::MD5.hexdigest(File.read(file_path))
+        digest: Digest::MD5.hexdigest(File.binread(file_path))
       }
     end
 
@@ -25,7 +25,15 @@ module RSpecTracer
     end
 
     def file_path(file_name)
+      return file_name if absolute_external_file?(file_name)
+
       File.expand_path(file_name.sub(%r{^/}, ''), RSpecTracer.root)
     end
+
+    def absolute_external_file?(file_name)
+      file_name.start_with?('/') &&
+        !file_name.start_with?(RSpecTracer.root) &&
+        File.file?(file_name)
+    end
   end
 end

data/lib/rspec_tracer/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RSpecTracer
-  VERSION = '1.0.0'
+  VERSION = '1.0.2'
 end

data/lib/rspec_tracer.rb

@@ -111,27 +111,27 @@ module RSpecTracer
     end
 
     def runner
-      return @runner if defined?(@runner)
+      @runner if defined?(@runner)
     end
 
     def coverage_reporter
-      return @coverage_reporter if defined?(@coverage_reporter)
+      @coverage_reporter if defined?(@coverage_reporter)
     end
 
     def coverage_merger
-      return @coverage_merger if defined?(@coverage_merger)
+      @coverage_merger if defined?(@coverage_merger)
     end
 
     def report_merger
-      return @report_merger if defined?(@report_merger)
+      @report_merger if defined?(@report_merger)
     end
 
     def trace_point
-      return @trace_point if defined?(@trace_point)
+      @trace_point if defined?(@trace_point)
     end
 
     def traced_files
-      return @traced_files if defined?(@traced_files)
+      @traced_files if defined?(@traced_files)
     end
 
     def trace_example?
@@ -139,11 +139,11 @@ module RSpecTracer
     end
 
     def simplecov?
-      return @simplecov if defined?(@simplecov)
+      defined?(@simplecov) && @simplecov == true
     end
 
     def parallel_tests?
-      return @parallel_tests if defined?(@parallel_tests)
+      defined?(@parallel_tests) && @parallel_tests == true
     end
 
     private
@@ -165,7 +165,7 @@ module RSpecTracer
     end
 
     def initial_setup
-      unless setup_rspec
+      unless setup_rspec?
         puts 'Could not find a running RSpec process'
 
         return
@@ -179,7 +179,7 @@ module RSpecTracer
     end
 
     def parallel_tests_setup
-      @parallel_tests = !(ENV['TEST_ENV_NUMBER'] && ENV['PARALLEL_TEST_GROUPS']).nil?
+      @parallel_tests = !(ENV.fetch('TEST_ENV_NUMBER', nil) && ENV.fetch('PARALLEL_TEST_GROUPS', nil)).nil?
 
       return unless parallel_tests?
 
@@ -208,7 +208,7 @@ module RSpecTracer
       end
     end
 
-    def setup_rspec
+    def setup_rspec?
       runners = ObjectSpace.each_object(::RSpec::Core::Runner) do |runner|
         runner_clazz = runner.singleton_class
         clazz = RSpecTracer::RSpecRunner
@@ -339,13 +339,8 @@ module RSpecTracer
       starting = Process.clock_gettime(Process::CLOCK_MONOTONIC)
       reports_dir = []
 
-      1.upto(ENV['PARALLEL_TEST_GROUPS'].to_i) do |test_num|
-        cache_path = File.dirname(RSpecTracer.cache_path)
-        cache_dir = File.join(cache_path, "parallel_tests_#{test_num}")
-
-        next unless File.directory?(cache_dir)
-
-        run_id = JSON.parse(File.read(File.join(cache_dir, 'last_run.json')))['run_id']
+      parallel_tests_peer_dirs(File.dirname(RSpecTracer.cache_path)).each do |cache_dir|
+        run_id = JSON.parse(File.read(File.join(cache_dir, 'last_run.json'), encoding: 'UTF-8'))['run_id']
 
         reports_dir << File.join(cache_dir, run_id)
       end
@@ -374,14 +369,8 @@ module RSpecTracer
       return unless parallel_tests_executed? && !simplecov?
 
       starting = Process.clock_gettime(Process::CLOCK_MONOTONIC)
-      reports_dir = []
 
-      1.upto(ENV['PARALLEL_TEST_GROUPS'].to_i) do |test_num|
-        coverage_path = File.dirname(RSpecTracer.coverage_path)
-        coverage_dir = File.join(coverage_path, "parallel_tests_#{test_num}")
-
-        reports_dir << coverage_dir if File.directory?(coverage_dir)
-      end
+      reports_dir = parallel_tests_peer_dirs(File.dirname(RSpecTracer.coverage_path))
 
       coverage_merger.merge(reports_dir)
 
@@ -410,13 +399,34 @@ module RSpecTracer
     def purge_parallel_tests_reports
       return unless parallel_tests_executed?
 
-      1.upto(ENV['PARALLEL_TEST_GROUPS'].to_i) do |test_num|
-        [RSpecTracer.cache_path, RSpecTracer.coverage_path, RSpecTracer.report_path].each do |path|
-          FileUtils.rm_rf(File.join(File.dirname(path), "parallel_tests_#{test_num}"))
+      [RSpecTracer.cache_path, RSpecTracer.coverage_path, RSpecTracer.report_path].each do |path|
+        parallel_tests_peer_dirs(File.dirname(path)).each do |worker_dir|
+          FileUtils.rm_rf(worker_dir)
         end
       end
     end
 
+    # Returns every `parallel_tests_*` subdirectory directly under
+    # `base_path`. Used by the parallel_tests merge + purge paths.
+    #
+    # Earlier patches iterated `1..ENV['PARALLEL_TEST_GROUPS'].to_i`
+    # to construct dir names, but parallel_tests's own runner sets
+    # PARALLEL_TEST_GROUPS to the user-requested process count
+    # (`Parallel.processor_count` by default), NOT the actual worker
+    # count. When num_processes < spawned_worker_count, the upper
+    # bound was too small: peer caches with TEST_ENV_NUMBER above the
+    # bound were silently dropped from the merge AND left behind by
+    # the purge. PR #101 (v1.1.1) documented this gem behaviour for
+    # `last_process?` detection but did not extend the fix to the
+    # iteration call-sites; this method closes that gap. Globbing the
+    # actual filesystem state is robust to the env discrepancy
+    # regardless of how the gem partitions specs.
+    def parallel_tests_peer_dirs(base_path)
+      Dir.glob(File.join(base_path, 'parallel_tests_*')).select do |path|
+        File.directory?(path)
+      end
+    end
+
     def parallel_tests_executed?
       return false unless parallel_tests? && parallel_tests_last_process?
 
@@ -425,18 +435,25 @@ module RSpecTracer
       true
     end
 
+    # Elects the worker that performs the per-run merge. Delegates to
+    # `::ParallelTests.first_process?`, which returns true iff
+    # `TEST_ENV_NUMBER.to_i <= 1` — i.e. for exactly one worker per run,
+    # regardless of how many workers were spawned vs. CPU count.
+    #
+    # The prior lock-file-max election deadlocked under slow CI: a fast
+    # worker could finish before a slow worker loaded spec_helper and
+    # registered its TEST_ENV_NUMBER; both then self-elected and spun on
+    # each other's pid inside wait_for_other_processes_to_finish.
+    #
+    # `track_parallel_tests_test_env_number` and the lock-file cleanup
+    # in at_exit_behavior are retained so that users observing
+    # `parallel_tests.lock` see no behavior change — the file is still
+    # written and removed, just no longer consulted for election.
     def parallel_tests_last_process?
       return false unless parallel_tests?
+      return false unless defined?(::ParallelTests)
 
-      max_test_num = 0
-
-      File.open(RSpecTracer.parallel_tests_lock_file, 'r') do |f|
-        f.flock(File::LOCK_SH)
-
-        max_test_num = f.read.to_i
-      end
-
-      ENV['TEST_ENV_NUMBER'].to_i == max_test_num
+      ::ParallelTests.first_process?
     end
   end
 end

metadata

@@ -1,55 +1,54 @@
 --- !ruby/object:Gem::Specification
 name: rspec-tracer
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.2
 platform: ruby
 authors:
 - Abhimanyu Singh
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-10-21 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: docile
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.1.0
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.1.0
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.1'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.1.0
 - !ruby/object:Gem::Dependency
   name: rspec-core
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 3.6.0
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.6'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.6.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 3.6.0
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.6'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.6.0
 description: RSpec Tracer is a specs dependency analyzer, flaky tests detector, tests
   accelerator, and coverage reporter tool for RSpec. It maintains a list of files
   for each test, enabling itself to skip tests in the subsequent runs if none of the
@@ -119,10 +118,9 @@ licenses:
 - MIT
 metadata:
   homepage_uri: https://github.com/avmnu-sng/rspec-tracer
-  source_code_uri: https://github.com/avmnu-sng/rspec-tracer/tree/v1.0.0
+  source_code_uri: https://github.com/avmnu-sng/rspec-tracer/tree/v1.0.2
   changelog_uri: https://github.com/avmnu-sng/rspec-tracer/blob/main/CHANGELOG.md
   bug_tracker_uri: https://github.com/avmnu-sng/rspec-tracer/issues
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -137,8 +135,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.9
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: RSpec Tracer is a specs dependency analyzer, flaky tests detector, tests
   accelerator, and coverage reporter tool.