knapsack_pro 1.22.1 → 1.22.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ff5e0f88a008abb13c1ca010f4b430be290ecf4e349719cac39611eebc575a4
-  data.tar.gz: 83650913e5d6cd21201ebce5ae48c731d61a0bc27bb37f566fcdc3d96a9b0598
+  metadata.gz: e71ab6f3ce5b553af4903e555e1d44cdaa5d3f5e6a6e27a67a0f651546f3c979
+  data.tar.gz: bc4cc3f898e1e02ffec511a62163b66659239ad4ecc2f2bdd0a0043dc2bdf478
 SHA512:
-  metadata.gz: 3b3d2dc24c6592040f559f77f59168502bb949b135406a06f59c836494902412e202a28bc781760e0862f416ccaa2d2c9a246269fee87e9fd3d9120ba803f4ab
-  data.tar.gz: 9d4005922f963d765b494ab2b35642fa3be85787a6deb8b490ed3e1f64e5055d474d4cdae5b1bba4d4668b4c63fd59d9063102baf7036830e53d409ba8cf016a
+  metadata.gz: c54c2b2c4d0a0a72625d3abc3fbc1d9123de22ca13cb2e409e372a5a7e818ce13399adc64f2a65c88bf1351de6b82834753b6633e953f79c498ef2c07a9979ec
+  data.tar.gz: 9886473b35736ee93aa08adf71202e83b21be27cfa82defa98478dd01ee8fadd503d27064fe1ac1b916ef218c4b828dc01f457523e14d8b1b26c26856727e06c

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Change Log
 
+### 1.22.2
+
+* Log when next retry request to Knapsack Pro API happens before starting Fallback Mode
+
+    https://github.com/KnapsackPro/knapsack_pro-ruby/pull/114
+
+https://github.com/KnapsackPro/knapsack_pro-ruby/compare/v1.22.1...v1.22.2
+
 ### 1.22.1
 
 * Fix for an auto split of slow RSpec test files by test examples when using `KNAPSACK_PRO_RSPEC_SPLIT_BY_TEST_EXAMPLES=true` and `parallel_tests` gem. Save the JSON reports with unique file names with the CI node index in the name to avoid accidentally overriding the files on the same disk.

data/lib/knapsack_pro/client/connection.rb

@@ -121,7 +121,11 @@ module KnapsackPro
         if retries < MAX_RETRY.call
           wait = retries * REQUEST_RETRY_TIMEBOX
           logger.warn("Wait #{wait}s and retry request to Knapsack Pro API.")
-          Kernel.sleep(wait)
+          print_every = 2 # seconds
+          (wait / print_every).ceil.times do |i|
+            logger.warn("Next request in #{wait - i * print_every}s...")
+            Kernel.sleep(print_every)
+          end
           retry
         else
           response_body

data/lib/knapsack_pro/version.rb

@@ -1,3 +1,3 @@
 module KnapsackPro
-  VERSION = '1.22.1'
+  VERSION = '1.22.2'
 end

data/spec/knapsack_pro/client/connection_spec.rb

@@ -87,9 +87,20 @@ shared_examples 'when retry request' do
       expect(logger).to receive(:warn).exactly(3).with(server_error.inspect)
 
       expect(logger).to receive(:warn).with("Wait 8s and retry request to Knapsack Pro API.")
+      expect(logger).to receive(:warn).with("Next request in 8s...")
+      expect(logger).to receive(:warn).with("Next request in 6s...")
+      expect(logger).to receive(:warn).with("Next request in 4s...")
+      expect(logger).to receive(:warn).with("Next request in 2s...")
       expect(logger).to receive(:warn).with("Wait 16s and retry request to Knapsack Pro API.")
-      expect(Kernel).to receive(:sleep).with(8)
-      expect(Kernel).to receive(:sleep).with(16)
+      expect(logger).to receive(:warn).with("Next request in 16s...")
+      expect(logger).to receive(:warn).with("Next request in 14s...")
+      expect(logger).to receive(:warn).with("Next request in 12s...")
+      expect(logger).to receive(:warn).with("Next request in 10s...")
+      expect(logger).to receive(:warn).with("Next request in 8s...")
+      expect(logger).to receive(:warn).with("Next request in 6s...")
+      expect(logger).to receive(:warn).with("Next request in 4s...")
+      expect(logger).to receive(:warn).with("Next request in 2s...")
+      expect(Kernel).to receive(:sleep).exactly(12).with(2)
 
       expect(subject).to eq(parsed_response)
 
@@ -115,15 +126,37 @@ shared_examples 'when retry request' do
         expect(logger).to receive(:warn).exactly(6).with(server_error.inspect)
 
         expect(logger).to receive(:warn).with("Wait 8s and retry request to Knapsack Pro API.")
+        expect(logger).to receive(:warn).with("Next request in 8s...")
+        expect(logger).to receive(:warn).with("Next request in 6s...")
+        expect(logger).to receive(:warn).with("Next request in 4s...")
+        expect(logger).to receive(:warn).with("Next request in 2s...")
+
         expect(logger).to receive(:warn).with("Wait 16s and retry request to Knapsack Pro API.")
+        expect(logger).to receive(:warn).with("Next request in 16s...")
+        expect(logger).to receive(:warn).with("Next request in 14s...")
+        expect(logger).to receive(:warn).with("Next request in 12s...")
+        expect(logger).to receive(:warn).with("Next request in 10s...")
+        expect(logger).to receive(:warn).with("Next request in 8s...")
+        expect(logger).to receive(:warn).with("Next request in 6s...")
+        expect(logger).to receive(:warn).with("Next request in 4s...")
+        expect(logger).to receive(:warn).with("Next request in 2s...")
+
         expect(logger).to receive(:warn).with("Wait 24s and retry request to Knapsack Pro API.")
+        12.times do |i|
+          expect(logger).to receive(:warn).with("Next request in #{(i+1)*2}s...")
+        end
+
         expect(logger).to receive(:warn).with("Wait 32s and retry request to Knapsack Pro API.")
+        16.times do |i|
+          expect(logger).to receive(:warn).with("Next request in #{(i+1)*2}s...")
+        end
+
         expect(logger).to receive(:warn).with("Wait 40s and retry request to Knapsack Pro API.")
-        expect(Kernel).to receive(:sleep).with(8)
-        expect(Kernel).to receive(:sleep).with(16)
-        expect(Kernel).to receive(:sleep).with(24)
-        expect(Kernel).to receive(:sleep).with(32)
-        expect(Kernel).to receive(:sleep).with(40)
+        20.times do |i|
+          expect(logger).to receive(:warn).with("Next request in #{(i+1)*2}s...")
+        end
+
+        expect(Kernel).to receive(:sleep).exactly(60).with(2)
 
         expect(subject).to eq(parsed_response)
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knapsack_pro
 version: !ruby/object:Gem::Version
-  version: 1.22.1
+  version: 1.22.2
 platform: ruby
 authors:
 - ArturT
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-05-31 00:00:00.000000000 Z
+date: 2020-06-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake