knapsack_pro 7.8.2 → 7.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b5ab7ad3dc194e05627836c95059415cc305c763b4e14facfd7139ea3888265e
-  data.tar.gz: 6e8f3bdf4bff758695e2a11398dddfe0bf03f38a3c3e687b13ca69f96de63cab
+  metadata.gz: 173300dc2ec82a1b5a15005c91ac3ceb942c40acbb14fd2ba57e64d2a32b67d8
+  data.tar.gz: c0c856044ce6773dce0968ebd620f4c49e1fea170c30d880af90ace2e1fb66c1
 SHA512:
-  metadata.gz: b6655218987352b0916596ed6320e900df07e7fbdeb8390fdd167d3577d9b5a2fd9c6f3917b06dc7c1496435dbc808fed314e23b661eab792efa9e68c5b4946a
-  data.tar.gz: 5c171a12bc010155e4401df17a87de1bc7059fc8454b897fa16ab589211fbefb637e8545ab38ec4e73b630d3262e524b2f9f49c0c40a99156cae26a0a63c0084
+  metadata.gz: e078dce53b72374ef499087c9675eea34272769a46154997e55dc06ab23e4da7fa016206297eed8b3cc660c5238a99b16177abb6f45c191e3bd38dc60c9cee6f
+  data.tar.gz: 5a80b8cbc3dc3273e297693263a9223fab250d1ee6adbd2c6c909f904d35430f3403a1f9da92dc2709adc3f30f8311b8d05d34b6292e8952e1aae42c88f8f977

data/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+### 7.10.0
+
+* Improve the RSpec split by examples feature. Use test file execution times for existing test files on the disk to determine slow test files. This fixes issue with detecting slow test files when API token is shared between multiple test suites.
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/277
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v7.9.0...v7.10.0
+
+### 7.9.0
+
+* A more readable error message for the RSpec split by examples JSON report (remove ANSI codes that are not human-readable)
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/275
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v7.8.2...v7.9.0
+
 ### 7.8.2
 
 * Set `RSpec.world.wants_to_quit` to true when any signal is received by the knapsack_pro gem to allow graceful exit.

data/lib/knapsack_pro/slow_test_file_determiner.rb

@@ -5,9 +5,9 @@ module KnapsackPro
     TIME_THRESHOLD_PER_CI_NODE = 0.7 # 70%
 
     # test_files: { 'path' => 'a_spec.rb', 'time_execution' => 0.0 }
-    # time_execution: of build distribution (total time of CI build run)
-    def self.call(test_files, time_execution)
-      time_threshold = (time_execution / KnapsackPro::Config::Env.ci_node_total) * TIME_THRESHOLD_PER_CI_NODE
+    def self.call(test_files)
+      total_execution_time = test_files.sum { |test_file| test_file.fetch('time_execution') }
+      time_threshold = (total_execution_time / KnapsackPro::Config::Env.ci_node_total) * TIME_THRESHOLD_PER_CI_NODE
 
       test_files.select do |test_file|
         time_execution = test_file.fetch('time_execution')

data/lib/knapsack_pro/slow_test_file_finder.rb

@@ -19,7 +19,7 @@ module KnapsackPro
 
       test_files_existing_on_disk = KnapsackPro::TestFileFinder.select_test_files_that_can_be_run(adapter_class, merged_test_files_from_api)
 
-      slow_test_files = KnapsackPro::SlowTestFileDeterminer.call(test_files_existing_on_disk, build_distribution_entity.time_execution)
+      slow_test_files = KnapsackPro::SlowTestFileDeterminer.call(test_files_existing_on_disk)
 
       KnapsackPro::SlowTestFileDeterminer.save_to_json_report(slow_test_files)
 

data/lib/knapsack_pro/test_case_detectors/rspec_test_example_detector.rb

@@ -27,6 +27,7 @@ module KnapsackPro
 
         cli_args = cli_format + [
           '--dry-run',
+          '--no-color',
           '--out', report_path,
           '--default-path', test_dir,
         ] + KnapsackPro::TestFilePresenter.paths(test_file_entities)

data/lib/knapsack_pro/version.rb

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

data/spec/knapsack_pro/slow_test_file_determiner_spec.rb

@@ -6,7 +6,7 @@ describe KnapsackPro::SlowTestFileDeterminer do
       expect(KnapsackPro::Config::Env).to receive(:ci_node_total).and_return(node_total)
     end
 
-    subject { described_class.call(test_files, time_execution) }
+    subject { described_class.call(test_files) }
 
     context 'when test files have recorded time execution' do
       let(:time_execution) { 20.0 }
@@ -14,22 +14,20 @@ describe KnapsackPro::SlowTestFileDeterminer do
         [
           { 'path' => 'a_spec.rb', 'time_execution' => 1.0 },
           { 'path' => 'b_spec.rb', 'time_execution' => 3.4 },
-          # slow tests are above 3.5s threshold (20.0 / 4 * 0.7 = 3.5)
           { 'path' => 'c_spec.rb', 'time_execution' => 3.5 },
-          { 'path' => 'd_spec.rb', 'time_execution' => 5.9 },
+          { 'path' => 'd_spec.rb', 'time_execution' => 12.1 },
         ]
       end
 
-      it do
+      it 'detects slow tests above 3.5s threshold (20.0 / 4 nodes * 70% threshold = 3.5)' do
         expect(subject).to eq([
           { 'path' => 'c_spec.rb', 'time_execution' => 3.5 },
-          { 'path' => 'd_spec.rb', 'time_execution' => 5.9 },
+          { 'path' => 'd_spec.rb', 'time_execution' => 12.1 },
         ])
       end
     end
 
     context 'when test files have no recorded time execution' do
-      let(:time_execution) { 0.0 }
       let(:test_files) do
         [
           { 'path' => 'a_spec.rb', 'time_execution' => 0.0 },
@@ -41,6 +39,14 @@ describe KnapsackPro::SlowTestFileDeterminer do
         expect(subject).to eq([])
       end
     end
+
+    context 'when there are no test files' do
+      let(:test_files) { [] }
+
+      it do
+        expect(subject).to eq([])
+      end
+    end
   end
 
   describe '.save_to_json_report', :clear_tmp do

data/spec/knapsack_pro/slow_test_file_finder_spec.rb

@@ -13,8 +13,7 @@ describe KnapsackPro::SlowTestFileFinder do
 
       it do
         test_files_from_api = double
-        time_execution = double
-        build_distribution_entity = instance_double(KnapsackPro::BuildDistributionFetcher::BuildDistributionEntity, test_files: test_files_from_api, time_execution: time_execution)
+        build_distribution_entity = instance_double(KnapsackPro::BuildDistributionFetcher::BuildDistributionEntity, test_files: test_files_from_api)
         expect(KnapsackPro::BuildDistributionFetcher).to receive(:call).and_return(build_distribution_entity)
 
         merged_test_files_from_api = double
@@ -24,7 +23,7 @@ describe KnapsackPro::SlowTestFileFinder do
         expect(KnapsackPro::TestFileFinder).to receive(:select_test_files_that_can_be_run).with(adapter_class, merged_test_files_from_api).and_return(test_files_existing_on_disk)
 
         slow_test_files = double
-        expect(KnapsackPro::SlowTestFileDeterminer).to receive(:call).with(test_files_existing_on_disk, time_execution).and_return(slow_test_files)
+        expect(KnapsackPro::SlowTestFileDeterminer).to receive(:call).with(test_files_existing_on_disk).and_return(slow_test_files)
 
         expect(KnapsackPro::SlowTestFileDeterminer).to receive(:save_to_json_report).with(slow_test_files)
 

data/spec/knapsack_pro/test_case_detectors/rspec_test_example_detector_spec.rb

@@ -48,6 +48,7 @@ describe KnapsackPro::TestCaseDetectors::RSpecTestExampleDetector do
           expect(RSpec::Core::ConfigurationOptions).to receive(:new).with([
             '--format', expected_format,
             '--dry-run',
+            '--no-color',
             '--out', report_path,
             '--default-path', test_dir,
             'spec/a_spec.rb', 'spec/b_spec.rb',

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 7.8.2
+  version: 7.10.0
 platform: ruby
 authors:
 - ArturT
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-10-10 00:00:00.000000000 Z
+date: 2024-10-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake