knapsack_pro 0.35.0 → 0.36.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: '094b839e2b6fef614a4c91b10579d484ee7aef37'
-  data.tar.gz: 1270f9dd4023504fa6fb4ab929ffeaccc605d529
+  metadata.gz: 8cc652e6837b671f2954c0ab3ab8119557fd7562
+  data.tar.gz: 84b5d915628454bb4de229af9e8b146d6f35a45d
 SHA512:
-  metadata.gz: c8266d9534aa7f0f93dfbce4e7a3699a9363f2a256a8c22e7dd3410d0ba50c27a73dd29cbdfa3fd53159520dca455257a1925b33f7904812f4f4cbe6f19e46a0
-  data.tar.gz: ad0ce91e958078a3884913aab721a9d08be79a0c22bbeaa387d1c16ff2a18a83f647d914e2d230a6b9aaf6627aa5e3bc173bc721ad0cb4ad6626d9e716a31dfa
+  metadata.gz: 4554fffac8f1e1c6f4f6ac4e36850e3c1f9fe8a16bdc25d8538532e0a713133c5d64adfda8bdc8733f775b697d8698e76f3817b8b882b487413399fca2a14d33
+  data.tar.gz: 51c5930337707e7a7d02e6a12577a6d366d9549b4cc2ea42820d5cec3485e950163db0501523c38b85daf0793873f9c231505cb7f3561e4417e240899f93f6e8

data/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 * TODO
 
+### 0.36.0
+
+* Show messages about not executed test files as warnings in logs.
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v0.35.0...v0.36.0
+
 ### 0.35.0
 
 * Add `RSpecQueueProfileFormatterExtension` to show profile summary only once at the very end of RSpec Queue Mode output.

data/README.md

@@ -1025,9 +1025,13 @@ Below example is for rspec. You can change `$KNAPSACK_PRO_TEST_SUITE_TOKEN_RSPEC
 if [ "$KNAPSACK_PRO_TEST_SUITE_TOKEN_RSPEC" = "" ]; then
   KNAPSACK_PRO_ENDPOINT=https://api-disabled-for-fork.knapsackpro.com \
     KNAPSACK_PRO_TEST_SUITE_TOKEN_RSPEC=disabled-for-fork \
-    bundle exec rake knapsack_pro:rspec
+    bundle exec rake knapsack_pro:rspec # use Regular Mode here always
 else
+    # Regular Mode
     bundle exec rake knapsack_pro:rspec
+
+    # You can use Queue Mode instead of Regular Mode if you like
+    # bundle exec rake knapsack_pro:queue:rspec
 fi
 ```
 

data/lib/knapsack_pro/report.rb

@@ -4,7 +4,7 @@ module KnapsackPro
       test_files = KnapsackPro.tracker.to_a
 
       if test_files.empty?
-        KnapsackPro.logger.info("No test files were executed on this CI node. When you use knapsack_pro regular mode then probably reason might be very narrowed tests list - you run only tests with specified tag and there are fewer test files with the tag than node total number.")
+        KnapsackPro.logger.warn("No test files were executed on this CI node. When you use knapsack_pro regular mode then probably reason might be very narrowed tests list - you run only tests with specified tag and there are fewer test files with the tag than node total number.")
       end
 
       create_build_subset(test_files)
@@ -35,7 +35,7 @@ module KnapsackPro
       end
 
       if test_files.empty?
-        KnapsackPro.logger.info("No test files were executed on this CI node. When you use knapsack_pro queue mode then probably reason might be that CI node was started after the test files from the queue were already executed by other CI nodes. That is why this CI node has no test files to execute.")
+        KnapsackPro.logger.warn("No test files were executed on this CI node. When you use knapsack_pro queue mode then probably reason might be that CI node was started after the test files from the queue were already executed by other CI nodes. That is why this CI node has no test files to execute.")
       end
 
       create_build_subset(test_files)

data/lib/knapsack_pro/tracker.rb

@@ -18,7 +18,7 @@ module KnapsackPro
     end
 
     def stop_timer
-      @execution_time = now_without_mock_time.to_f - @start_time
+      @execution_time = @start_time ? now_without_mock_time.to_f - @start_time : 0.0
       update_global_time
       update_test_file_time
       @execution_time

data/lib/knapsack_pro/version.rb

@@ -1,3 +1,3 @@
 module KnapsackPro
-  VERSION = '0.35.0'
+  VERSION = '0.36.0'
 end

data/spec/knapsack_pro/tracker_spec.rb

@@ -85,6 +85,22 @@ describe KnapsackPro::Tracker do
       it { expect(tracker.test_files_with_time['b_spec.rb']).to be_within(delta).of(0) }
       it_behaves_like '#to_a'
     end
+
+    # https://github.com/KnapsackPro/knapsack_pro-ruby/issues/32
+    context 'when start timer was not called (rspec-retry issue)' do
+      before do
+        test_paths.each_with_index do |test_path, index|
+          tracker.current_test_path = test_path
+          tracker.stop_timer
+        end
+      end
+
+      it { expect(tracker.global_time).to eq 0 }
+      it { expect(tracker.test_files_with_time.keys.size).to eql 2 }
+      it { expect(tracker.test_files_with_time['a_spec.rb']).to eq 0 }
+      it { expect(tracker.test_files_with_time['b_spec.rb']).to eq 0 }
+      it_behaves_like '#to_a'
+    end
   end
 
   describe '#reset!' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 0.35.0
+  version: 0.36.0
 platform: ruby
 authors:
 - ArturT
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-01 00:00:00.000000000 Z
+date: 2017-05-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake