Fix the guard for max amount of serializers

sorentwo · Jul 4, 2016 · d9caeb8 · d9caeb8
1 parent 4aaa51a
commit d9caeb8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/readthis/serializers.rb b/lib/readthis/serializers.rb
@@ -41,7 +41,7 @@ def <<(serializer)
       case
       when serializers.frozen?
         raise SerializersFrozenError
-      when serializers.length > SERIALIZER_LIMIT
+      when serializers.length >= SERIALIZER_LIMIT
         raise SerializersLimitError
       else
         @serializers[serializer] = flags.max.succ

diff --git a/spec/readthis/serializers_spec.rb b/spec/readthis/serializers_spec.rb
@@ -22,9 +22,9 @@
 
     it 'prevents more than seven serializers' do
       serializers = Readthis::Serializers.new
-
+      serializers << Class.new until serializers.flags.length >= 7
       expect do
-        10.times { serializers << Class.new }
+        serializers << Class.new
       end.to raise_error(Readthis::SerializersLimitError)
     end
   end