dhc 2.2.0 → 2.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dbcbbfbe466771c1ad389aebe29d997915137216b50b0aceff73f3cbc589f050
-  data.tar.gz: a659a0b022d803a700352d20fe96482731470b31234e8db9df3f665da5547ae7
+  metadata.gz: 0f70dd2da0d8ec740cf21499fc95ba815a73730c2e1c6a4b8cfef6ff18eef0ce
+  data.tar.gz: b0773620c07f30f040439d1c56c7a144ee2f144eb0d987d885aec2b940367b6a
 SHA512:
-  metadata.gz: 1851454238cb9552457a56c020edcb88b8b7e37eed6ebc0c6d1a8574c8250c520cfd48d8ca1e9f82ddcd334085daf1651fb0c31221ea638921aeb5873218846f
-  data.tar.gz: 4432965dcb69081d1d082cfc47101644a817f387d79213f57b8d2b80192901148e49b6a9ceb4ddb10c632e425385b30026de4c2bbbe69ffdaeefb5bf105c6bf5
+  metadata.gz: 3c652d89a8d1dcbeadf72472322881ce136692d1b51630c61f4cf0b058ff96d9ded1345c80238fa5d46599d494328ebaecc007a915283eae1971efe89de8cea6
+  data.tar.gz: 9f71766d909205e3771cd2cfd65130458ac6a73bf35119bd6edf83f1de1ecba418611b66544affd4572b4b275641875d6ae6507fd4a4d07ad1609ae263ff1694

data/lib/dhc/interceptors/throttle.rb

@@ -59,7 +59,7 @@ class DHC::Throttle < DHC::Interceptor
       remaining = tracker[:remaining] * 100
       limit = tracker[:limit]
       remaining_quota = 100 - options[:break].to_i
-      remaining < remaining_quota * limit
+      remaining <= remaining_quota * limit
     end
   end
 

data/lib/dhc/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DHC
-  VERSION ||= '2.2.0'
+  VERSION ||= '2.2.1'
 end

data/spec/interceptors/throttle/manually_spec.rb

@@ -33,11 +33,11 @@ describe DHC::Throttle do
   end
 
   context 'breaks' do
-    let(:quota_limit) { 10 }
+    let(:quota_limit) { 100 }
     let(:break_after) { '79%' }
 
     it 'hit the breaks if throttling quota is reached' do
-      8.times do
+      79.times do
         DHC.get('http://depay.fi', options)
       end
       expect { DHC.get('http://depay.fi', options) }.to raise_error(
@@ -50,19 +50,29 @@ describe DHC::Throttle do
       let(:break_after) { '80%' }
 
       it 'does not hit the breaks' do
-        9.times do
+        80.times do
           DHC.get('http://depay.fi', options)
         end
       end
+
+      it 'does hit the breaks surpassing quota' do
+        80.times do
+          DHC.get('http://depay.fi', options)
+        end
+        expect { DHC.get('http://depay.fi', options) }.to raise_error(
+          DHC::Throttle::OutOfQuota,
+          'Reached predefined quota for depay.fi'
+        )
+      end
     end
   end
 
   context 'expires' do
     let(:break_after) { '80%' }
-    let(:quota_limit) { 10 }
+    let(:quota_limit) { 100 }
 
     it 'attempts another request if the quota expired' do
-      9.times do
+      80.times do
         DHC.get('http://depay.fi', options)
       end
       expect { DHC.get('http://depay.fi', options) }.to raise_error(
@@ -70,7 +80,7 @@ describe DHC::Throttle do
         'Reached predefined quota for depay.fi'
       )
       Timecop.travel(Time.zone.now + 1.minute)
-      9.times do
+      80.times do
         DHC.get('http://depay.fi', options)
       end
       expect { DHC.get('http://depay.fi', options) }.to raise_error(

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dhc
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 2.2.1
 platform: ruby
 authors:
 - https://github.com/DePayFi/dhc/contributors