knapsack_pro 3.1.1 → 3.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1a66b28a9f7bdfa3aaa0492c3099b62d020b2a01a48030865c503448ccb4ff77
-  data.tar.gz: c0fae2955b72521decd338c64be57c31eacc2f33d41dbce39610be0584183aab
+  metadata.gz: 5baae9cb1ea264d73e085e9f475b4f775e06801992ffd715f33a90fc93fd974f
+  data.tar.gz: e74b748147d57817546675c86ea534274f65cf287e52884d3960f09aa44efbb7
 SHA512:
-  metadata.gz: c142d923945cb4ed649535b93d0a632f5ed705513c9b3528e776784e0f6e5959ce08b81c084e69a89dd42027e453f32c2830743e8f366afe2d0fb4ddca5d8ae7
-  data.tar.gz: 6ca8aa6ac8ad655bb267e17ba93564b5f44585b0dcaac74e195c7e818323f9aa954da067753c7c0b9b7c3782d3d3ecbf2e3829e7c147a9f054cde3af56898ad8
+  metadata.gz: 8f6102eef985cb8cb6c810053be5afecefcf309eb6a8367e3bcacdde09fb6bd10aa8d6ea10f68cdcdaf9b02a83117f2720ba6613562183c1ec9d52751ca87d56
+  data.tar.gz: b8c3d75e4267141135723fd29de4d5b97dffc1485dc762d7a9df0ba449c0827fa184261ffc101435b94678a7fc89bc8ed75876937a96d90ee6fd6103d7e38362

data/CHANGELOG.md

@@ -1,5 +1,29 @@
 # Change Log
 
+### 3.2.0
+
+* Add an error message to `KnapsackPro::Adapters::RspecAdapter#bind`
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/165
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v3.1.3...v3.2.0
+
+### 3.1.3
+
+* Run Fallback Mode when `Errno::ECONNRESET` exception happens
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/164
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v3.1.2...v3.1.3
+
+### 3.1.2
+
+* Fix bug when test files have no recorded time execution then they should not be detected as slow test files for RSpec split by test examples feature
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/163
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v3.1.1...v3.1.2
+
 ### 3.1.1
 
 * Rephrase log outputs in the Queue Mode RSpec runner

data/lib/knapsack_pro/adapters/rspec_adapter.rb

@@ -101,6 +101,11 @@ module KnapsackPro
     # In case someone is doing switch from knapsack gem to the knapsack_pro gem
     # and didn't notice the class name changed
     class RspecAdapter < RSpecAdapter
+      def self.bind
+        error_message = 'You have attempted to call KnapsackPro::Adapters::RspecAdapter.bind. Please switch to using the new class name: KnapsackPro::Adapters::RSpecAdapter. See https://docs.knapsackpro.com/knapsack_pro-ruby/guide for up-to-date configuration instructions.'
+        KnapsackPro.logger.error(error_message)
+        raise error_message
+      end
     end
   end
 end

data/lib/knapsack_pro/client/connection.rb

@@ -119,7 +119,7 @@ module KnapsackPro
         end
 
         response_body
-      rescue ServerError, Errno::ECONNREFUSED, Errno::ETIMEDOUT, Errno::EPIPE, EOFError, SocketError, Net::OpenTimeout, Net::ReadTimeout, OpenSSL::SSL::SSLError => e
+      rescue ServerError, Errno::ECONNREFUSED, Errno::ECONNRESET, Errno::ETIMEDOUT, Errno::EPIPE, EOFError, SocketError, Net::OpenTimeout, Net::ReadTimeout, OpenSSL::SSL::SSLError => e
         logger.warn("#{action.http_method.to_s.upcase} #{endpoint_url}")
         logger.warn('Request failed due to:')
         logger.warn(e.inspect)

data/lib/knapsack_pro/slow_test_file_determiner.rb

@@ -8,7 +8,8 @@ module KnapsackPro
       time_threshold = (time_execution / KnapsackPro::Config::Env.ci_node_total) * TIME_THRESHOLD_PER_CI_NODE
 
       test_files.select do |test_file|
-        test_file.fetch('time_execution') >= time_threshold
+        time_execution = test_file.fetch('time_execution')
+        time_execution >= time_threshold && time_execution > 0
       end
     end
 

data/lib/knapsack_pro/version.rb

@@ -1,3 +1,3 @@
 module KnapsackPro
-  VERSION = '3.1.1'
+  VERSION = '3.2.0'
 end

data/spec/knapsack_pro/slow_test_file_determiner_spec.rb

@@ -1,16 +1,6 @@
 describe KnapsackPro::SlowTestFileDeterminer do
   describe '.call' do
     let(:node_total) { 4 }
-    let(:time_execution) { 20.0 }
-    let(:test_files) 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 },
-      ]
-    end
 
     before do
       expect(KnapsackPro::Config::Env).to receive(:ci_node_total).and_return(node_total)
@@ -18,11 +8,38 @@ describe KnapsackPro::SlowTestFileDeterminer do
 
     subject { described_class.call(test_files, time_execution) }
 
-    it do
-      expect(subject).to eq([
-        { 'path' => 'c_spec.rb', 'time_execution' => 3.5 },
-        { 'path' => 'd_spec.rb', 'time_execution' => 5.9 },
-      ])
+    context 'when test files have recorded time execution' do
+      let(:time_execution) { 20.0 }
+      let(:test_files) 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 },
+        ]
+      end
+
+      it do
+        expect(subject).to eq([
+          { 'path' => 'c_spec.rb', 'time_execution' => 3.5 },
+          { 'path' => 'd_spec.rb', 'time_execution' => 5.9 },
+        ])
+      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 },
+          { 'path' => 'b_spec.rb', 'time_execution' => 0.0 },
+        ]
+      end
+
+      it do
+        expect(subject).to eq([])
+      end
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 3.1.1
+  version: 3.2.0
 platform: ruby
 authors:
 - ArturT
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-09-10 00:00:00.000000000 Z
+date: 2022-03-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -405,7 +405,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.22
+rubygems_version: 3.2.32
 signing_key:
 specification_version: 4
 summary: Knapsack Pro splits tests across parallel CI nodes and ensures each parallel