berater 0.6.1 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5c5cc50ba02c18bf9ef9f6b9c1dd6ccb8f8af0297df95757ac13406dc4539c40
-  data.tar.gz: f80341dccff30ca3f344f3bdf21de44b1ef1715bfaf4140933a91252ddb7231e
+  metadata.gz: dd61012052d49f83a59057ed5ce4cc46b9c25a2d2bf4b5e494f02e590c2e2249
+  data.tar.gz: 3add1b481830566088eb7a9dce3540716b55dda9bb658a7b26b93a261b403f3a
 SHA512:
-  metadata.gz: ada3fffc841e71498f652dc990eed482bccc942b86280fc5ee736d49fdf99d8ab2f921b5b52d96f3588c268c34343669c00e0be8822c462a3e6f9dcf954a4005
-  data.tar.gz: 0363abef69cfdad8140d89373b5088cfa8a6259f16716011c82ceb1e2e9e948cdb4d44ddd36aaf1d05b121dc7d275d0ee34571a0a3ee9b964f4286375ec356cf
+  metadata.gz: 476cf552a5fb56498f81a737dba79358d016bedbd76c3c2cc8069c420e82a221c54280781821ef701d25da2551ddc25910818dc400dc7c046cecbe8f6db0ddd2
+  data.tar.gz: a92e64f027425ecdc4a39ee5f88a792f4f2625ec4bcf65fac70e34d591fb136e7544caa80138474ae5f5e6158fc19ec8894892585cf858966bfe5fe13a3d011b

data/lib/berater/limiter.rb

@@ -52,7 +52,7 @@ module Berater
 
     def self.new(*)
       # can only call via subclass
-      raise NotImplementedError if self == Berater::Limiter
+      raise NoMethodError if self == Berater::Limiter
 
       super
     end

data/lib/berater/test_mode.rb

@@ -23,7 +23,7 @@ module Berater
         e = self.class.constants.map do |name|
           self.class.const_get(name)
         end.find do |const|
-          const < Berater::Overloaded
+          const.is_a?(Class) && const < Berater::Overloaded
         end || Berater::Overloaded
 
         raise e

data/lib/berater/version.rb

@@ -1,3 +1,3 @@
 module Berater
-  VERSION = '0.6.1'
+  VERSION = '0.6.2'
 end

data/spec/limiter_spec.rb

@@ -1,6 +1,6 @@
 describe Berater::Limiter do
   it 'can not be initialized' do
-    expect { described_class.new }.to raise_error(NotImplementedError)
+    expect { described_class.new }.to raise_error(NoMethodError)
   end
 
   describe 'abstract methods' do
@@ -12,7 +12,43 @@ describe Berater::Limiter do
     end
   end
 
-  describe '==' do
+  describe '#limit' do
+    subject { Berater::Unlimiter.new }
+
+    context 'with a capacity parameter' do
+      it 'overrides the stored value' do
+        is_expected.to receive(:acquire_lock).with(3, anything)
+
+        subject.limit(capacity: 3)
+      end
+
+      it 'validates the type' do
+        expect {
+          subject.limit(capacity: 'abc')
+        }.to raise_error(ArgumentError)
+      end
+    end
+
+    context 'with a cost parameter' do
+      it 'overrides the stored value' do
+        is_expected.to receive(:acquire_lock).with(anything, 2)
+
+        subject.limit(cost: 2)
+      end
+
+      it 'validates' do
+        expect {
+          subject.limit(cost: 'abc')
+        }.to raise_error(ArgumentError)
+
+        expect {
+          subject.limit(cost: -1)
+        }.to raise_error(ArgumentError)
+      end
+    end
+  end
+
+  describe '#==' do
     let(:limiter)  { Berater::RateLimiter.new(:key, 1, :second) }
 
     it 'equals itself' do

data/spec/test_mode_spec.rb

@@ -18,7 +18,7 @@ describe Berater::TestMode do
     end
 
     it 'preserves the original functionality via super' do
-      expect { Berater::Limiter.new }.to raise_error(NotImplementedError)
+      expect { Berater::Limiter.new }.to raise_error(NoMethodError)
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: berater
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.6.2
 platform: ruby
 authors:
 - Daniel Pepper
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-09 00:00:00.000000000 Z
+date: 2021-03-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis