knapsack_pro 5.3.3 → 5.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 388e3d47f2da2b0d9f1377677b912a44af5919111a86869115713488a83d84e2
-  data.tar.gz: '08fd59b79cc652a4045f11f24a363cac8928644a2cf2a705b766e558e77d104f'
+  metadata.gz: 202eb37b7550c8556fadd902ed204ea71d5c928b2ed79f3452aa2f7dabee1a35
+  data.tar.gz: 846ec0dd9f9953bbbfeed2b9f77535a25e5182de8b3798dcff88a48fce1ba0f9
 SHA512:
-  metadata.gz: 881dc5a753a3af8ffc1320eab7dd5a50ba38909812e971a1aeb2e282a3566f27afea3b601d99784d27d9959d7cf13f2d93b8db7634f73ccfb72a82b33e4243b2
-  data.tar.gz: d7887b2f0bb7854b1d926f422b70ab1e4ac21904026e01863aa9b83e7a690a4df0d576f6ed863fe86e8f9dfb574db106553a7a61fc0913b1dc26ac22903c422b
+  metadata.gz: 6e9e1bffd253bade681efd59cdb43a13c8ae99bb43981dbf8aaea0f7adc3e5d2cce530488e6308a7eb35821abfc9245f26236e3e3dc323b68dbd3fd457905865
+  data.tar.gz: aedf904bdf8f765e1fdcbe651ee9d1f839201c48fdad7750668c88cb2f70bb1f61fff1fa633062f0be3bf31770656d48b0467b61b970f6d03526c430b7dc12c8

data/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+### 5.3.5
+
+* Handle RSpec exceptions when running RSpec in Queue Mode
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/214
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/215
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v5.3.4...v5.3.5
+
+### 5.3.4
+
+* fix(Queue Mode): handle OS signals and RSpec internal `wants_to_quit` and `rspec_is_quitting` states to stop consuming tests from the Queue API when the CI node is terminated
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/207
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v5.3.3...v5.3.4
+
 ### 5.3.3
 
 * Fix hanging CI when `git fetch --shallow-since` takes too long

data/lib/knapsack_pro/formatters/rspec_queue_summary_formatter.rb

@@ -62,11 +62,39 @@ module KnapsackPro
         unless most_recent_pending.empty?
           registered_output.puts('All pending tests on this CI node:')
           registered_output.puts(most_recent_pending)
+          registered_output.puts('')
         end
 
         unless most_recent_failures_summary.empty?
           registered_output.puts('All failed tests on this CI node:')
           registered_output.puts(most_recent_failures_summary)
+          registered_output.puts('')
+        end
+
+        registered_output.puts(most_recent_summary)
+      end
+
+      def self.print_exit_summary
+        registered_output.puts('Knapsack Pro Queue exited/aborted!')
+        registered_output.puts('')
+
+        unexecuted_test_files = KnapsackPro.tracker.unexecuted_test_files
+        unless unexecuted_test_files.empty?
+          registered_output.puts('Unexecuted tests on this CI node:')
+          registered_output.puts(unexecuted_test_files)
+          registered_output.puts('')
+        end
+
+        unless most_recent_pending.empty?
+          registered_output.puts('All pending tests on this CI node:')
+          registered_output.puts(most_recent_pending)
+          registered_output.puts('')
+        end
+
+        unless most_recent_failures_summary.empty?
+          registered_output.puts('All failed tests on this CI node:')
+          registered_output.puts(most_recent_failures_summary)
+          registered_output.puts('')
         end
 
         registered_output.puts(most_recent_summary)

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

@@ -2,6 +2,10 @@ module KnapsackPro
   module Runners
     module Queue
       class BaseRunner
+        TERMINATION_SIGNALS = %w(HUP INT TERM ABRT QUIT USR1 USR2)
+
+        @@terminate_process = false
+
         def self.run(args)
           raise NotImplementedError
         end
@@ -13,6 +17,7 @@ module KnapsackPro
         def initialize(adapter_class)
           @allocator_builder = KnapsackPro::QueueAllocatorBuilder.new(adapter_class)
           @allocator = allocator_builder.allocator
+          trap_signals
         end
 
         def test_file_paths(args)
@@ -33,6 +38,23 @@ module KnapsackPro
         def self.child_status
           $?
         end
+
+        def self.handle_signal!
+          raise 'Knapsack Pro process was terminated!' if @@terminate_process
+        end
+
+        def self.set_terminate_process
+          @@terminate_process = true
+        end
+
+        def trap_signals
+          TERMINATION_SIGNALS.each do |signal|
+            Signal.trap(signal) {
+              puts "#{signal} signal has been received. Terminating Knapsack Pro..."
+              @@terminate_process = true
+            }
+          end
+        end
       end
     end
   end

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

@@ -22,6 +22,7 @@ module KnapsackPro
             all_test_file_paths: [],
           }
           while accumulator[:status] == :next
+            handle_signal!
             accumulator = run_tests(accumulator)
           end
 

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

@@ -29,6 +29,7 @@ module KnapsackPro
             all_test_file_paths: [],
           }
           while accumulator[:status] == :next
+            handle_signal!
             accumulator = run_tests(accumulator)
           end
 

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

@@ -37,6 +37,7 @@ module KnapsackPro
             all_test_file_paths: [],
           }
           while accumulator[:status] == :next
+            handle_signal!
             accumulator = run_tests(accumulator)
           end
 
@@ -91,26 +92,44 @@ module KnapsackPro
             options = ::RSpec::Core::ConfigurationOptions.new(cli_args)
             rspec_runner = ::RSpec::Core::Runner.new(options)
 
-            exit_code = rspec_runner.run($stderr, $stdout)
-            exitstatus = exit_code if exit_code != 0
-
-            printable_args = args_with_seed_option_added_when_viable(args, rspec_runner)
-            log_rspec_command(printable_args, test_file_paths, :subset_queue)
-
-            rspec_clear_examples
-
-            KnapsackPro::Hooks::Queue.call_after_subset_queue
-
-            KnapsackPro::Report.save_subset_queue_to_file
-
-            return {
-              status: :next,
-              runner: runner,
-              can_initialize_queue: false,
-              args: args,
-              exitstatus: exitstatus,
-              all_test_file_paths: all_test_file_paths,
-            }
+            begin
+              exit_code = rspec_runner.run($stderr, $stdout)
+              exitstatus = exit_code if exit_code != 0
+            rescue Exception => exception
+              KnapsackPro.logger.error("Having exception when running RSpec: #{exception.inspect}")
+              KnapsackPro::Formatters::RSpecQueueSummaryFormatter.print_exit_summary
+              KnapsackPro::Hooks::Queue.call_after_subset_queue
+              KnapsackPro::Hooks::Queue.call_after_queue
+              Kernel.exit(1)
+              raise
+            else
+              if rspec_runner.world.wants_to_quit
+                KnapsackPro.logger.warn('RSpec wants to quit.')
+                set_terminate_process
+              end
+              if rspec_runner.world.rspec_is_quitting
+                KnapsackPro.logger.warn('RSpec is quitting.')
+                set_terminate_process
+              end
+
+              printable_args = args_with_seed_option_added_when_viable(args, rspec_runner)
+              log_rspec_command(printable_args, test_file_paths, :subset_queue)
+
+              rspec_clear_examples
+
+              KnapsackPro::Hooks::Queue.call_after_subset_queue
+
+              KnapsackPro::Report.save_subset_queue_to_file
+
+              return {
+                status: :next,
+                runner: runner,
+                can_initialize_queue: false,
+                args: args,
+                exitstatus: exitstatus,
+                all_test_file_paths: all_test_file_paths,
+              }
+            end
           end
         end
 

data/lib/knapsack_pro/tracker.rb

@@ -68,6 +68,12 @@ module KnapsackPro
       @prerun_tests_loaded = true
     end
 
+    def unexecuted_test_files
+      @test_files_with_time.map do |path, hash|
+        path unless hash[:measured_time]
+      end.compact
+    end
+
     def to_a
       # When the test files are not loaded in the memory then load them from the disk.
       # Useful for the Regular Mode when the memory is not shared between tracker instances.

data/lib/knapsack_pro/version.rb

@@ -1,3 +1,3 @@
 module KnapsackPro
-  VERSION = '5.3.3'
+  VERSION = '5.3.5'
 end

data/lib/tasks/queue/cucumber.rake

@@ -4,7 +4,13 @@ namespace :knapsack_pro do
   namespace :queue do
     task :cucumber, [:cucumber_args] do |_, args|
       Kernel.system("RAILS_ENV=test RACK_ENV=test #{$PROGRAM_NAME} 'knapsack_pro:queue:cucumber_go[#{args[:cucumber_args]}]'")
-      Kernel.exit($?.exitstatus)
+      exitstatus = $?.exitstatus
+      if exitstatus.nil?
+        puts 'Something went wrong. Most likely, the process has been killed. Knapsack Pro has been terminated.'
+        Kernel.exit(1)
+      else
+        Kernel.exit(exitstatus)
+      end
     end
 
     task :cucumber_go, [:cucumber_args] do |_, args|

data/lib/tasks/queue/minitest.rake

@@ -4,7 +4,13 @@ namespace :knapsack_pro do
   namespace :queue do
     task :minitest, [:minitest_args] do |_, args|
       Kernel.system("RAILS_ENV=test RACK_ENV=test #{$PROGRAM_NAME} 'knapsack_pro:queue:minitest_go[#{args[:minitest_args]}]'")
-      Kernel.exit($?.exitstatus)
+      exitstatus = $?.exitstatus
+      if exitstatus.nil?
+        puts 'Something went wrong. Most likely, the process has been killed. Knapsack Pro has been terminated.'
+        Kernel.exit(1)
+      else
+        Kernel.exit(exitstatus)
+      end
     end
 
     task :minitest_go, [:minitest_args] do |_, args|

data/lib/tasks/queue/rspec.rake

@@ -4,7 +4,13 @@ namespace :knapsack_pro do
   namespace :queue do
     task :rspec, [:rspec_args] do |_, args|
       Kernel.system("RAILS_ENV=test RACK_ENV=test #{$PROGRAM_NAME} 'knapsack_pro:queue:rspec_go[#{args[:rspec_args]}]'")
-      Kernel.exit($?.exitstatus)
+      exitstatus = $?.exitstatus
+      if exitstatus.nil?
+        puts 'Something went wrong. Most likely, the process has been killed. Knapsack Pro has been terminated.'
+        Kernel.exit(1)
+      else
+        Kernel.exit(exitstatus)
+      end
     end
 
     task :rspec_go, [:rspec_args] do |_, args|

data/spec/knapsack_pro/runners/queue/cucumber_runner_spec.rb

@@ -41,6 +41,7 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
           exitstatus: 0,
           all_test_file_paths: [],
         }
+        expect(described_class).to receive(:handle_signal!)
         expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
 
         expect(Kernel).to receive(:exit).with(expected_exitstatus)
@@ -66,6 +67,7 @@ describe KnapsackPro::Runners::Queue::CucumberRunner do
           exitstatus: 0,
           all_test_file_paths: [],
         }
+        expect(described_class).to receive(:handle_signal!)
         expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
 
         expect(Kernel).to receive(:exit).with(expected_exitstatus)

data/spec/knapsack_pro/runners/queue/minitest_runner_spec.rb

@@ -43,6 +43,7 @@ describe KnapsackPro::Runners::Queue::MinitestRunner do
           exitstatus: 0,
           all_test_file_paths: [],
         }
+        expect(described_class).to receive(:handle_signal!)
         expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
 
         expect(Kernel).to receive(:exit).with(expected_exitstatus)
@@ -68,6 +69,7 @@ describe KnapsackPro::Runners::Queue::MinitestRunner do
           exitstatus: 0,
           all_test_file_paths: [],
         }
+        expect(described_class).to receive(:handle_signal!)
         expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
 
         expect(Kernel).to receive(:exit).with(expected_exitstatus)

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

@@ -49,6 +49,7 @@ describe KnapsackPro::Runners::Queue::RSpecRunner do
             exitstatus: 0,
             all_test_file_paths: [],
           }
+          expect(described_class).to receive(:handle_signal!)
           expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
 
           expect(Kernel).to receive(:exit).with(expected_exitstatus)
@@ -74,6 +75,7 @@ describe KnapsackPro::Runners::Queue::RSpecRunner do
             exitstatus: 0,
             all_test_file_paths: [],
           }
+          expect(described_class).to receive(:handle_signal!)
           expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
 
           expect(Kernel).to receive(:exit).with(expected_exitstatus)
@@ -99,6 +101,7 @@ describe KnapsackPro::Runners::Queue::RSpecRunner do
             exitstatus: 0,
             all_test_file_paths: [],
           }
+          expect(described_class).to receive(:handle_signal!)
           expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
 
           expect(Kernel).to receive(:exit).with(expected_exitstatus)
@@ -124,6 +127,7 @@ describe KnapsackPro::Runners::Queue::RSpecRunner do
             exitstatus: 0,
             all_test_file_paths: [],
           }
+          expect(described_class).to receive(:handle_signal!)
           expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
 
           expect(Kernel).to receive(:exit).with(expected_exitstatus)
@@ -165,6 +169,7 @@ describe KnapsackPro::Runners::Queue::RSpecRunner do
           exitstatus: 0,
           all_test_file_paths: [],
         }
+        expect(described_class).to receive(:handle_signal!)
         expect(described_class).to receive(:run_tests).with(accumulator).and_return(expected_accumulator)
 
         expect(Kernel).to receive(:exit).with(expected_exitstatus)
@@ -200,76 +205,190 @@ describe KnapsackPro::Runners::Queue::RSpecRunner do
       let(:test_file_paths) { ['a_spec.rb', 'b_spec.rb'] }
       let(:logger) { double }
       let(:rspec_seed) { 7771 }
+      let(:exit_code) { [0, 1].sample }
+      let(:rspec_wants_to_quit) { false }
+      let(:rspec_is_quitting) { false }
+      let(:rspec_core_runner) do
+        double(world: double(wants_to_quit: rspec_wants_to_quit, rspec_is_quitting: rspec_is_quitting))
+      end
 
-      before do
-        subset_queue_id = 'fake-subset-queue-id'
-        expect(KnapsackPro::Config::EnvGenerator).to receive(:set_subset_queue_id).and_return(subset_queue_id)
+      context 'having no exception when running RSpec' do
+        before do
+          subset_queue_id = 'fake-subset-queue-id'
+          expect(KnapsackPro::Config::EnvGenerator).to receive(:set_subset_queue_id).and_return(subset_queue_id)
 
-        expect(ENV).to receive(:[]=).with('KNAPSACK_PRO_SUBSET_QUEUE_ID', subset_queue_id)
+          expect(ENV).to receive(:[]=).with('KNAPSACK_PRO_SUBSET_QUEUE_ID', subset_queue_id)
 
-        tracker = instance_double(KnapsackPro::Tracker)
-        expect(KnapsackPro).to receive(:tracker).twice.and_return(tracker)
-        expect(tracker).to receive(:reset!)
-        expect(tracker).to receive(:set_prerun_tests).with(test_file_paths)
+          tracker = instance_double(KnapsackPro::Tracker)
+          expect(KnapsackPro).to receive(:tracker).twice.and_return(tracker)
+          expect(tracker).to receive(:reset!)
+          expect(tracker).to receive(:set_prerun_tests).with(test_file_paths)
 
-        expect(described_class).to receive(:ensure_spec_opts_have_rspec_queue_summary_formatter)
-        options = double
-        expect(RSpec::Core::ConfigurationOptions).to receive(:new).with([
-          '--no-color',
-          '--default-path', 'fake-test-dir',
-          'a_spec.rb', 'b_spec.rb',
-        ]).and_return(options)
+          expect(described_class).to receive(:ensure_spec_opts_have_rspec_queue_summary_formatter)
+          options = double
+          expect(RSpec::Core::ConfigurationOptions).to receive(:new).with([
+            '--no-color',
+            '--default-path', 'fake-test-dir',
+            'a_spec.rb', 'b_spec.rb',
+          ]).and_return(options)
 
-        rspec_core_runner = double
-        expect(RSpec::Core::Runner).to receive(:new).with(options).and_return(rspec_core_runner)
-        expect(rspec_core_runner).to receive(:run).with($stderr, $stdout).and_return(exit_code)
+          expect(RSpec::Core::Runner).to receive(:new).with(options).and_return(rspec_core_runner)
+          expect(rspec_core_runner).to receive(:run).with($stderr, $stdout).and_return(exit_code)
 
-        expect(described_class).to receive(:rspec_clear_examples)
+          expect(described_class).to receive(:rspec_clear_examples)
 
-        expect(KnapsackPro::Hooks::Queue).to receive(:call_before_subset_queue)
+          expect(KnapsackPro::Hooks::Queue).to receive(:call_before_subset_queue)
 
-        expect(KnapsackPro::Hooks::Queue).to receive(:call_after_subset_queue)
+          expect(KnapsackPro::Hooks::Queue).to receive(:call_after_subset_queue)
 
-        expect(KnapsackPro::Report).to receive(:save_subset_queue_to_file)
+          expect(KnapsackPro::Report).to receive(:save_subset_queue_to_file)
 
-        configuration = double
-        expect(rspec_core_runner).to receive(:configuration).twice.and_return(configuration)
-        expect(configuration).to receive(:seed_used?).and_return(true)
-        expect(configuration).to receive(:seed).and_return(rspec_seed)
+          configuration = double
+          expect(rspec_core_runner).to receive(:configuration).twice.and_return(configuration)
+          expect(configuration).to receive(:seed_used?).and_return(true)
+          expect(configuration).to receive(:seed).and_return(rspec_seed)
 
-        expect(KnapsackPro).to receive(:logger).twice.and_return(logger)
-        expect(logger).to receive(:info)
-          .with("To retry the last batch of tests fetched from the API Queue, please run the following command on your machine:")
-        expect(logger).to receive(:info).with(/#{args.join(' ')} --seed #{rspec_seed}/)
-      end
+          expect(KnapsackPro).to receive(:logger).at_least(2).and_return(logger)
+          expect(logger).to receive(:info)
+            .with("To retry the last batch of tests fetched from the API Queue, please run the following command on your machine:")
+          expect(logger).to receive(:info).with(/#{args.join(' ')} --seed #{rspec_seed}/)
+        end
 
-      context 'when exit code is zero' do
-        let(:exit_code) { 0 }
+        context 'when the exit code is zero' do
+          let(:exit_code) { 0 }
 
-        it do
-          expect(subject).to eq({
-            status: :next,
-            runner: runner,
-            can_initialize_queue: false,
-            args: args,
-            exitstatus: exitstatus,
-            all_test_file_paths: test_file_paths,
-          })
+          it do
+            expect(subject).to eq({
+              status: :next,
+              runner: runner,
+              can_initialize_queue: false,
+              args: args,
+              exitstatus: exitstatus,
+              all_test_file_paths: test_file_paths,
+            })
+          end
+        end
+
+        context 'when the exit code is not zero' do
+          let(:exit_code) { double }
+
+          it do
+            expect(subject).to eq({
+              status: :next,
+              runner: runner,
+              can_initialize_queue: false,
+              args: args,
+              exitstatus: exit_code,
+              all_test_file_paths: test_file_paths,
+            })
+          end
+        end
+
+        context 'when RSpec wants to quit' do
+          let(:exit_code) { 0 }
+          let(:rspec_wants_to_quit) { true }
+
+          after do
+            described_class.class_variable_set(:@@terminate_process, false)
+          end
+
+          it 'terminates the process' do
+            expect(logger).to receive(:warn).with('RSpec wants to quit.')
+
+            expect(described_class.class_variable_get(:@@terminate_process)).to be false
+
+            expect(subject).to eq({
+              status: :next,
+              runner: runner,
+              can_initialize_queue: false,
+              args: args,
+              exitstatus: exitstatus,
+              all_test_file_paths: test_file_paths,
+            })
+
+            expect(described_class.class_variable_get(:@@terminate_process)).to be true
+          end
+        end
+
+        context 'when RSpec is quitting' do
+          let(:exit_code) { 0 }
+          let(:rspec_is_quitting) { true }
+
+          after do
+            described_class.class_variable_set(:@@terminate_process, false)
+          end
+
+          it 'terminates the process' do
+            expect(logger).to receive(:warn).with('RSpec is quitting.')
+
+            expect(described_class.class_variable_get(:@@terminate_process)).to be false
+
+            expect(subject).to eq({
+              status: :next,
+              runner: runner,
+              can_initialize_queue: false,
+              args: args,
+              exitstatus: exitstatus,
+              all_test_file_paths: test_file_paths,
+            })
+
+            expect(described_class.class_variable_get(:@@terminate_process)).to be true
+          end
         end
       end
 
-      context 'when exit code is not zero' do
-        let(:exit_code) { double }
+      context 'having exception when running RSpec' do
+        before do
+          subset_queue_id = 'fake-subset-queue-id'
+          expect(KnapsackPro::Config::EnvGenerator).to receive(:set_subset_queue_id).and_return(subset_queue_id)
+
+          expect(ENV).to receive(:[]=).with('KNAPSACK_PRO_SUBSET_QUEUE_ID', subset_queue_id)
+
+          tracker = instance_double(KnapsackPro::Tracker)
+          expect(KnapsackPro).to receive(:tracker).twice.and_return(tracker)
+          expect(tracker).to receive(:reset!)
+          expect(tracker).to receive(:set_prerun_tests).with(test_file_paths)
+
+          expect(described_class).to receive(:ensure_spec_opts_have_rspec_queue_summary_formatter)
+          options = double
+          expect(RSpec::Core::ConfigurationOptions).to receive(:new).with([
+            '--no-color',
+            '--default-path', 'fake-test-dir',
+            'a_spec.rb', 'b_spec.rb',
+          ]).and_return(options)
+
+          rspec_core_runner = double
+          expect(RSpec::Core::Runner).to receive(:new).with(options).and_return(rspec_core_runner)
+          expect(rspec_core_runner).to receive(:run).with($stderr, $stdout).and_raise SystemExit
+          expect(KnapsackPro::Hooks::Queue).to receive(:call_before_subset_queue)
+          allow(KnapsackPro::Report).to receive(:save_subset_queue_to_file)
+          allow(KnapsackPro::Hooks::Queue).to receive(:call_after_subset_queue)
+          allow(KnapsackPro::Hooks::Queue).to receive(:call_after_queue)
+          allow(KnapsackPro::Formatters::RSpecQueueSummaryFormatter).to receive(:print_exit_summary)
+          expect(Kernel).to receive(:exit).with(1)
+        end
 
-        it do
-          expect(subject).to eq({
-            status: :next,
-            runner: runner,
-            can_initialize_queue: false,
-            args: args,
-            exitstatus: exit_code,
-            all_test_file_paths: test_file_paths,
-          })
+        it 'does not call #save_subset_queue_to_file or #rspec_clear_examples' do
+          expect(described_class).not_to receive(:rspec_clear_examples)
+          expect(KnapsackPro::Report).not_to receive(:save_subset_queue_to_file)
+          expect { subject }.to raise_error SystemExit
+        end
+
+        it 'logs the exception' do
+          expect(KnapsackPro).to receive(:logger).once.and_return(logger)
+          expect(logger).to receive(:error).with("Having exception when running RSpec: #<SystemExit: SystemExit>")
+          expect { subject }.to raise_error SystemExit
+        end
+
+        it 'calls #print_exit_summary' do
+          expect(KnapsackPro::Formatters::RSpecQueueSummaryFormatter).to receive(:print_exit_summary)
+          expect { subject }.to raise_error SystemExit
+        end
+
+        it 'calls #call_after_subset_queue and #call_after_queue' do
+          expect(KnapsackPro::Hooks::Queue).to receive(:call_after_subset_queue)
+          expect(KnapsackPro::Hooks::Queue).to receive(:call_after_queue)
+          expect { subject }.to raise_error SystemExit
         end
       end
     end

data/spec/knapsack_pro/tracker_spec.rb

@@ -185,4 +185,21 @@ describe KnapsackPro::Tracker do
       expect(tracker.prerun_tests_loaded).to be false
     end
   end
+
+
+  describe '#unexecuted_test_files' do
+    before do
+      tracker.set_prerun_tests(['a_spec.rb', 'b_spec.rb', 'c_spec.rb'])
+
+      # measure execution time for b_spec.rb
+      tracker.current_test_path = 'b_spec.rb'
+      tracker.start_timer
+      sleep 0.1
+      tracker.stop_timer
+    end
+
+    it 'returns test files without measured time' do
+      expect(tracker.unexecuted_test_files).to eq(['a_spec.rb', 'c_spec.rb'])
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 5.3.3
+  version: 5.3.5
 platform: ruby
 authors:
 - ArturT
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-07-21 00:00:00.000000000 Z
+date: 2023-08-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake