knapsack_pro 8.1.1 → 8.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5ad155a65c67bffcb3dad75d114396d2e53cc8896f21a84de7325a56f9a582ba
-  data.tar.gz: 6a6e0958d5596b150cdc6ee46453d5ff9991255e98113990015e34a8fef8aa1d
+  metadata.gz: 0cb3592a7a5f0e3f69138e53d5659954e0e84190ad84380d823e776268d4e900
+  data.tar.gz: b40c96cebfb9c4446998291b01c435f6e29cd937a51848593510ed633e56b14b
 SHA512:
-  metadata.gz: ee9feec10df81e17331b3664af0a30f8d75c4d436bb9c897829588518bb3468e5c00a3729d0e5c9f731ce535ab2b02dc5747a52a743e400bd49ffd046bb7ef51
-  data.tar.gz: d982189a7880df48104810628f1ff6e416c420325ace9fc2827c988c0289bfe27d2e78ff4bdffd94fe20891e37333805f2d530ab40aa822632847daed5b9596d
+  metadata.gz: 8308e34071c72b055335ded1422bb49762c600fffbc86f83a54911dde1cc23db59e472233bd53c1ec558fbf612c11b537587e241774dd8824d14e6665fd914e8
+  data.tar.gz: 35829c1ba92de81be2d7b3f5bd428b9fb2a66a8ad3da940f21e196b02b7962ce13f0c453f27c85c3acfce681223b3a83145baf2b26ef12b10a66dcf67f0d9fa3

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+### 8.1.2
+
+* Allow running RSpec with `--force-color` (and the default Split by Test Examples)
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/298
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v8.1.1...v8.1.2
+
 ### 8.1.1
 
 * Do not load Rake tasks on behalf of the user for RSpec and Minitest in Queue Mode

data/lib/knapsack_pro/test_case_detectors/rspec_test_example_detector.rb

@@ -33,32 +33,22 @@ module KnapsackPro
         # Apply a --format option which overrides formatters from the RSpec custom option files like `.rspec`.
         cli_args = cli_args_without_formatters + cli_format + [
           '--dry-run',
-          '--no-color',
           '--out', report_path,
-          '--default-path', test_dir,
+          '--default-path', test_dir
         ] + KnapsackPro::TestFilePresenter.paths(test_file_entities)
-        options = ::RSpec::Core::ConfigurationOptions.new(cli_args)
-        exit_code = ::RSpec::Core::Runner.new(options).run($stderr, $stdout)
-        if exit_code != 0
-          debug_cmd = ([
-            'bundle exec rspec',
-          ] + cli_args).join(' ')
-
-          KnapsackPro.logger.error('-'*10 + ' START of actionable error message ' + '-'*50)
-          KnapsackPro.logger.error('RSpec (with a dry-run option) had a problem generating the report with test examples for the slow test files. Here is what you can do:')
-
-          KnapsackPro.logger.error("a) Please look for an error message from RSpec in the output above or below. If you don't see anything, that is fine. Sometimes RSpec does not produce any errors in the output.")
-
-          KnapsackPro.logger.error("b) Check if RSpec generated the report file #{report_path}. If the report exists, it may contain an error message. Here is a preview of the report file:")
-          KnapsackPro.logger.error(report_content || 'N/A')
-
-          KnapsackPro.logger.error('c) To reproduce the error manually, please run the following RSpec command. This way, you can find out what is causing the error. Please ensure you run the command in the same environment where the error occurred. For instance, if the error happens on the CI server, you should run the command in the CI environment:')
-          KnapsackPro.logger.error(debug_cmd)
+        exit_code = begin
+          options = ::RSpec::Core::ConfigurationOptions.new(cli_args)
+          ::RSpec::Core::Runner.new(options).run($stderr, $stdout)
+        rescue SystemExit => e
+          e.status
+        end
 
-          KnapsackPro.logger.error('-'*10 + ' END of actionable error message ' + '-'*50)
+        return if exit_code.zero?
 
-          raise 'There was a problem while generating test examples for the slow test files. Please read the actionable error message above.'
-        end
+        report.fetch('messages', []).each { |message| puts message }
+        command = (['bundle exec rspec'] + cli_args).join(' ')
+        KnapsackPro.logger.error("Failed to generate the slow test files report: #{command}")
+        exit exit_code
       end
 
       def test_file_example_paths
@@ -92,8 +82,9 @@ module KnapsackPro
         "#{report_dir}/rspec_dry_run_json_report_node_#{KnapsackPro::Config::Env.ci_node_index}.json"
       end
 
-      def report_content
-        File.read(report_path) if File.exist?(report_path)
+      def report
+        return {} unless File.exist?(report_path)
+        JSON.parse(File.read(report_path))
       end
 
       def adapter_class

data/lib/knapsack_pro/version.rb

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

data/spec/knapsack_pro/test_case_detectors/rspec_test_example_detector_spec.rb

@@ -48,7 +48,6 @@ describe KnapsackPro::TestCaseDetectors::RSpecTestExampleDetector do
           expect(RSpec::Core::ConfigurationOptions).to receive(:new).with(expected_args + [
             '--format', expected_format,
             '--dry-run',
-            '--no-color',
             '--out', report_path,
             '--default-path', test_dir,
             'spec/a_spec.rb', 'spec/b_spec.rb',
@@ -78,7 +77,7 @@ describe KnapsackPro::TestCaseDetectors::RSpecTestExampleDetector do
           end
 
           it do
-            expect { subject }.to raise_error(RuntimeError, 'There was a problem while generating test examples for the slow test files. Please read the actionable error message above.')
+            expect { subject }.to raise_error(SystemExit) { |error| expect(error.status).to eq exit_code }
           end
         end
       end
@@ -123,6 +122,36 @@ describe KnapsackPro::TestCaseDetectors::RSpecTestExampleDetector do
         expect { subject }.to raise_error("The internal KNAPSACK_PRO_RSPEC_OPTIONS environment variable is unset. Ensure it is not overridden accidentally. Otherwise, please report this as a bug: https://knapsackpro.com/perma/ruby/support")
       end
     end
+
+    context 'with --force-color' do
+      let(:rspec_args) { '--force-color' }
+      let(:expected_args) { ['--force-color'] }
+      let(:expected_format) { 'json' }
+
+      it_behaves_like 'generate_json_report runs RSpec::Core::Runner'
+    end
+
+    context 'with --no-color and --force-color' do
+      let(:rspec_args) { '--no-color --force-color' }
+
+      after { KnapsackPro.reset_logger! }
+
+      it do
+        subject_class = Class.new(KnapsackPro::TestCaseDetectors::RSpecTestExampleDetector) do
+          define_method(:slow_test_files) do
+            [{ 'path' => 'spec/a_spec.rb' }]
+          end
+        end
+
+        expect do
+          KnapsackPro.logger = ::Logger.new($stdout)
+          subject_class.new.generate_json_report(rspec_args)
+        end
+          .to output(/Please only use one of `--force-color` and `--no-color`/).to_stderr
+          .and output(%r{ERROR -- : \[knapsack_pro\] Failed to generate the slow test files report: bundle exec rspec --no-color --force-color --format json --dry-run --out .knapsack_pro/test_case_detectors/rspec/rspec_dry_run_json_report_node_0.json --default-path spec spec/a_spec.rb}).to_stdout
+          .and raise_error(SystemExit) { |error| expect(error.status).to eq 1 }
+      end
+    end
   end
 
   describe '#test_file_example_paths' do

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 8.1.1
+  version: 8.1.2
 platform: ruby
 authors:
 - ArturT
 bindir: bin
 cert_chain: []
-date: 2025-04-10 00:00:00.000000000 Z
+date: 2025-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake