rspec-tracer 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b7f283b4a6ab3d0d2616f7dca4bac721906d36b4296ba1d64446937db1503dd5
-  data.tar.gz: 2922f96420e344603a472b4e67bb5f2c58c16e8bf798e13098eec8f8096434c5
+  metadata.gz: 5e89ead4a61c2b6d881fe8ca3fbcfec3ab94e7d8b78f759de5fe64e4e22b693d
+  data.tar.gz: 8993bc089a6185bdd7f0e20f71ad69b13fbd96c679bf67878c69b4f52dacd12f
 SHA512:
-  metadata.gz: 222031b5d7b0a6b01cad104fda83347a9e10af957c7a5d1dfede9f46cf8a275435c3e0c9dfb02ad7788b3dfc2d4c3b9513a60ee3ccf0d3bfd68a692758a38f1e
-  data.tar.gz: f2828567b26b580ffc66568f8b14581cb160f4bdf14a61622b605b5832d5d3bc86cff211190628c5480604f0ec1c758609ec8aa490b357016e3565615c635be2
+  metadata.gz: 91dc8066a66b706cfc02e9cb1050c61fc2e6bfa15d7353d056b11edb854ac493a99770516ce57e9ef4649494a0d99573b9dc8e441ead128b9e28abce9e2e21c1
+  data.tar.gz: 21a2cceab3b2cd3f1c3051542a1267d03a7cb499417f9501502a8c1bac3ac62e96821a1c07ea977768adca4b806a1fffca13b0e2946db95f031e877bc5bc4ebd

data/CHANGELOG.md

@@ -1,3 +1,20 @@
+## [1.1.1] - 2026-04-23
+
+### Fixed
+
+- **parallel_tests at-exit deadlock** — `parallel_tests_last_process?`
+  relied on a lock file written during `RSpecTracer.start` to identify
+  the last worker. If a fast worker reached `at_exit` before a slower
+  peer had loaded `spec_helper` and registered its `TEST_ENV_NUMBER`,
+  both workers could self-elect as the last process, both entered
+  `::ParallelTests.wait_for_other_processes_to_finish`, and deadlocked
+  on each other's pid. The elector now delegates to
+  `::ParallelTests.first_process?`, which reads immutable env vars set
+  by the parent at worker spawn. Exactly one worker is elected per run,
+  regardless of boot-time ordering or runner CPU count. No public-API
+  change — the `rspec_tracer.lock` file is still written and cleaned
+  up, just no longer consulted.
+
 ## [1.1.0] - 2026-04-20
 
 ### Added

data/lib/rspec_tracer/version.rb

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

data/lib/rspec_tracer.rb

@@ -416,18 +416,47 @@ 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
+    # (TEST_ENV_NUMBER == '' or '1'), regardless of how many workers
+    # were actually spawned vs. how many CPUs the runner reports.
+    #
+    # Two previously attempted approaches do NOT work here:
+    #
+    #   1. The lock-file scheme below (each worker writing its
+    #      TEST_ENV_NUMBER to `rspec_tracer.lock` via
+    #      `track_parallel_tests_test_env_number`; last_process picked
+    #      the max) deadlocked under slow CI: worker 1 could finish
+    #      its examples before worker 2 even loaded spec_helper,
+    #      observe itself as the max, and enter
+    #      `::ParallelTests.wait_for_other_processes_to_finish`
+    #      concurrently with worker 2's own self-election — both
+    #      workers then spun on each other's pid.
+    #
+    #   2. `::ParallelTests.last_process?` compares TEST_ENV_NUMBER
+    #      against PARALLEL_TEST_GROUPS, which parallel_rspec sets to
+    #      the CPU-based *intended* process count — NOT the actual
+    #      worker count. When spec files < CPU count (common), no
+    #      TEST_ENV_NUMBER ever matches PARALLEL_TEST_GROUPS and the
+    #      merge is silently skipped.
+    #
+    # `first_process?` avoids both: set by the parent at spawn,
+    # immutable thereafter, and identifies exactly one worker
+    # regardless of CPU count. The elected worker still calls
+    # `wait_for_other_processes_to_finish` before merging so peer
+    # caches are guaranteed on disk.
+    #
+    # `track_parallel_tests_test_env_number` and the lock-file
+    # cleanup in `at_exit_behavior` are retained for backward
+    # compatibility with users who observe `rspec_tracer.lock` /
+    # set `RSPEC_TRACER_LOCK_FILE`; the file is still written and
+    # removed but is no longer consulted.
     def parallel_tests_last_process?
       return false unless parallel_tests?
+      return false unless defined?(::ParallelTests)
 
-      max_test_num = 0
-
-      File.open(RSpecTracer.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,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rspec-tracer
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Abhimanyu Singh
@@ -111,7 +111,7 @@ licenses:
 - MIT
 metadata:
   homepage_uri: https://github.com/avmnu-sng/rspec-tracer
-  source_code_uri: https://github.com/avmnu-sng/rspec-tracer/tree/v1.1.0
+  source_code_uri: https://github.com/avmnu-sng/rspec-tracer/tree/v1.1.1
   changelog_uri: https://github.com/avmnu-sng/rspec-tracer/blob/main/CHANGELOG.md
   bug_tracker_uri: https://github.com/avmnu-sng/rspec-tracer/issues
 rdoc_options: []