knapsack_pro 7.13.1 → 7.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 199f5a710b27c9e76985316a9ac5dfeb95b6f7970a9246f26cdfaee8fb06adcd
-  data.tar.gz: 2b3421149a4284e32abf8f3351e6efafac9fc2bcd26a0225228b6f1a1ba5753d
+  metadata.gz: 113e9ca31d058ab9623d6a0bb278768ad0a864db8bb8776440d00ceb2a648b37
+  data.tar.gz: '084be6d88bf072c8ece112b3de2a6707b240816e993f039701cd68e7920f7988'
 SHA512:
-  metadata.gz: 983ffb0e290dba0d7a3e93bcf025fa0fd380e9457bc9c93c477783386c79dfeb1195a726f1513aea2970c882c46621d929e5ad204e343afa81eaa9ca73906790
-  data.tar.gz: 3f5031d341b563fbf231e38506c70521fc3cbd3b7ff55a529c725491b330cbd7bed105fbd6540c97fe76842d6c84b63dbaf8369937a4cbddb10ebde754ec6a1a
+  metadata.gz: 97015f7b3db931f3a9f68e7ef59aecb99f0befa5202b148d5aa56736d256048c842c00babc265c47c583534f2a0dafc488c49b748b4b3084f93a2d42f6013d84
+  data.tar.gz: 62fd8be74b14f402840ea4b9b6866a9ac008daa09d279e2cee21bd3f513d8f746f635c770c83e65db36139aa6e9ebb362301794e6aad70a9d59796dd86167d90

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+### 7.14.0
+
+* Improve debugging for hanging CI nodes: show hanging spec files in the RSpec output and a command to reproduce the current batch of tests.
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/287
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v7.13.1...v7.14.0
+
 ### 7.13.1
 
 * Fix handling signals for non-RSpec test runners

data/lib/knapsack_pro/runners/queue/base_runner.rb

@@ -77,6 +77,8 @@ module KnapsackPro
           puts 'Use the following backtrace(s) to find the line of code that got stuck if the CI node hung and terminated your tests.'
           puts 'How to read the backtrace: https://knapsackpro.com/perma/ruby/backtrace-debugging'
 
+          log_current_tests(threads)
+
           threads.each do |thread|
             puts
             if thread == Thread.main
@@ -95,6 +97,10 @@ module KnapsackPro
 
           $stdout.flush
         end
+
+        def log_current_tests(threads)
+          # implement in a child class if you need to log more info
+        end
       end
     end
   end

data/lib/knapsack_pro/runners/queue/rspec_runner.rb

@@ -119,6 +119,27 @@ module KnapsackPro
 
         def post_trap_signals
           RSpec.world.wants_to_quit = true
+
+          log_current_batch_rspec_command
+        end
+
+        def log_current_tests(threads)
+          threads.each do |thread|
+            next unless thread.backtrace
+
+            spec_file_lines = thread.backtrace.select { |line| line.include?('_spec.rb') }
+            next if spec_file_lines.empty?
+
+            puts
+            if thread == Thread.main
+              puts "Running specs in the main thread:"
+            else
+              puts "Non-main thread inspect: #{thread.inspect}"
+              puts "Running specs in non-main thread:"
+            end
+            puts spec_file_lines.join("\n")
+            puts
+          end
         end
 
         def pre_run_setup
@@ -165,6 +186,21 @@ module KnapsackPro
           test_file_paths
         end
 
+        def log_current_batch_rspec_command
+          test_file_paths = @queue.current_batch&.test_file_paths
+          return unless test_file_paths
+
+          puts
+          puts '=' * 80
+
+          order_option = @adapter_class.order_option(@cli_args)
+          printable_args = @rspec_pure.args_with_seed_option_added_when_viable(order_option, @rspec_runner.knapsack__seed, @cli_args)
+          messages = @rspec_pure.rspec_command(printable_args, test_file_paths, :batch_finished)
+          messages.each do |message|
+            puts message
+          end
+        end
+
         def log_rspec_batch_command(test_file_paths)
           order_option = @adapter_class.order_option(@cli_args)
           printable_args = @rspec_pure.args_with_seed_option_added_when_viable(order_option, @rspec_runner.knapsack__seed, @cli_args)

data/lib/knapsack_pro/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module KnapsackPro
-  VERSION = '7.13.1'
+  VERSION = '7.14.0'
 end

data/spec/integration/runners/queue/rspec_runner_spec.rb

@@ -1403,12 +1403,16 @@ describe "#{KnapsackPro::Runners::Queue::RSpecRunner} - Integration tests", :cle
         OUTPUT
       )
 
+      expect(actual.stdout).to include('To retry the last batch of tests fetched from the Queue API, please run the following command on your machine:')
+      expect(actual.stdout).to include('bundle exec rspec --format documentation --default-path spec_integration "spec_integration/b_spec.rb" "spec_integration/c_spec.rb"')
 
       expect(actual.stdout).to include('Use the following backtrace(s) to find the line of code that got stuck if the CI node hung and terminated your tests.')
+      expect(actual.stdout).to include('Running specs in the main thread:')
+      expect(actual.stdout).to include('Running specs in non-main thread:')
       expect(actual.stdout).to include('Main thread backtrace:')
-      expect(actual.stdout).to match(/spec_integration\/b_spec\.rb:7:in .*kill/)
+      expect(actual.stdout.scan(/spec_integration\/b_spec\.rb:7:in .*kill/).size).to eq 2
       expect(actual.stdout).to include('Non-main thread backtrace:')
-      expect(actual.stdout).to match(/spec_integration\/a_spec\.rb:6:in .*sleep/)
+      expect(actual.stdout.scan(/spec_integration\/a_spec\.rb:6:in .*sleep/).size).to eq 2
 
 
       expect(actual.exit_code).to eq 1

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 7.13.1
+  version: 7.14.0
 platform: ruby
 authors:
 - ArturT
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-04 00:00:00.000000000 Z
+date: 2025-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake