gem_config 0.2.3 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 57e379d40fd9841d4cdfb62de3e01d7210f79a1d
-  data.tar.gz: 98f0ce64593e693a50b82003c2853529a5165e49
+  metadata.gz: e38c07a983ddbdf73d79f7b1f7c613849d552c60
+  data.tar.gz: 394380642649dbd31e48c5c91350dd0a74aab495
 SHA512:
-  metadata.gz: d94aaaca5660e5742b211cdd22015aa0661ee5d2d4dcd932014dda330271d7280f842d36c7437850ebd16611be8ba83fdec8a9c57d11fb09ba4fead4ad544767
-  data.tar.gz: 3fa452a4ed955290cb266d614abc838d937bd458f97add5bf91104be49f7659c6c01b306ff1449d3230e2cd93ae85fff96aef28b49d7be6f9dc763ae3904c78f
+  metadata.gz: b1fef53ce71a8d421011cfef1c1cd075795488b17fe32e1076dbf5d756f3512ef5e68e3ce66fce120ae07a8c1a6cbc0a1c87a0f986bac773c51d2e37748e5ced
+  data.tar.gz: 206c1aecd2f2aa37393b05fb69b7ff210c4bef027810cc2f6dbb844e5f07b58061f0f20b981ab56ff8b0786ae031db2b433570438adf9227ac8adee4103df4b7

data/gem_config.gemspec

@@ -21,6 +21,4 @@ Gem::Specification.new do |gem|
 
   gem.add_development_dependency 'rake', '>= 0.9.0'
   gem.add_development_dependency 'rspec', '~> 2.13.0'
-
-  gem.add_runtime_dependency 'activesupport', '>= 3.0.0'
 end

data/lib/gem_config/rules.rb

@@ -1,5 +1,3 @@
-require 'active_support/core_ext/hash/keys'
-
 module GemConfig
   class Rules < Hash
     InvalidKeyError = Class.new(StandardError)
@@ -24,7 +22,10 @@ module GemConfig
     private
 
     def check_attributes(attrs)
-      attrs.assert_valid_keys :classes, :values, :default
+      allowed_keys = [:classes, :values, :default]
+      attrs.keys.each do |k|
+        raise ArgumentError, %("#{k}" is not a valid attribute. Valid attributes are: #{allowed_keys.join(', ')}) unless allowed_keys.include?(k)
+      end
 
       if attrs.has_key?(:classes)
         other_than_class = Array(attrs[:classes]).any? do |value|

data/lib/gem_config/version.rb

@@ -1,3 +1,3 @@
 module GemConfig
-  VERSION = '0.2.3'
+  VERSION = '0.2.4'
 end

data/spec/lib/rules_spec.rb

@@ -33,78 +33,86 @@ describe GemConfig::Rules do
       end.to raise_error(ArgumentError)
     end
 
-    describe 'parameter :classes' do
-      it 'only allows a class or an array of classes as the value' do
+    describe 'parameters' do
+      it 'raises an exception when a not allowed parameter is passed' do
         expect do
-          subject.has :foo, classes: String
-        end.to_not raise_error(ArgumentError)
-
-        expect do
-          subject.has :foo, classes: [String, Numeric]
-        end.to_not raise_error(ArgumentError)
-
-        expect do
-          subject.has :foo, classes: 'foo'
+          subject.has :foo, bar: 'baz'
         end.to raise_error(ArgumentError)
       end
-    end
-
-    describe 'parameter :values' do
-      context 'when :classes are defined' do
-        it 'only allows values of one of the specified classes' do
-          expect do
-            subject.has :foo, classes: String, values: 'foo'
-          end.to_not raise_error(ArgumentError)
 
+      describe 'parameter :classes' do
+        it 'only allows a class or an array of classes as the value' do
           expect do
-            subject.has :foo, classes: Numeric, values: 1
+            subject.has :foo, classes: String
           end.to_not raise_error(ArgumentError)
 
           expect do
-            subject.has :foo, classes: [String, Numeric], values: ['foo', 1]
+            subject.has :foo, classes: [String, Numeric]
           end.to_not raise_error(ArgumentError)
 
           expect do
-            subject.has :foo, classes: String, values: ['foo', 1]
+            subject.has :foo, classes: 'foo'
           end.to raise_error(ArgumentError)
         end
       end
-    end
-
-    describe 'parameter :default' do
-      context 'when :classes are defined' do
-        it 'only allows a value of one of the specified classes' do
-          expect do
-            subject.has :foo, classes: String, default: 'foo'
-          end.to_not raise_error(ArgumentError)
-
-          expect do
-            subject.has :foo, classes: Numeric, default: 1
-          end.to_not raise_error(ArgumentError)
-
-          expect do
-            subject.has :foo, classes: [String, Numeric], default: 'foo'
-          end.to_not raise_error(ArgumentError)
 
-          expect do
-            subject.has :foo, classes: Numeric, default: 'foo'
-          end.to raise_error(ArgumentError)
+      describe 'parameter :values' do
+        context 'when :classes are defined' do
+          it 'only allows values of one of the specified classes' do
+            expect do
+              subject.has :foo, classes: String, values: 'foo'
+            end.to_not raise_error(ArgumentError)
+
+            expect do
+              subject.has :foo, classes: Numeric, values: 1
+            end.to_not raise_error(ArgumentError)
+
+            expect do
+              subject.has :foo, classes: [String, Numeric], values: ['foo', 1]
+            end.to_not raise_error(ArgumentError)
+
+            expect do
+              subject.has :foo, classes: String, values: ['foo', 1]
+            end.to raise_error(ArgumentError)
+          end
         end
       end
 
-      context 'when :values are defined' do
-        it 'only allows one of the specified values' do
-          expect do
-            subject.has :foo, values: 'foo', default: 'foo'
-          end.to_not raise_error(ArgumentError)
+      describe 'parameter :default' do
+        context 'when :classes are defined' do
+          it 'only allows a value of one of the specified classes' do
+            expect do
+              subject.has :foo, classes: String, default: 'foo'
+            end.to_not raise_error(ArgumentError)
+
+            expect do
+              subject.has :foo, classes: Numeric, default: 1
+            end.to_not raise_error(ArgumentError)
+
+            expect do
+              subject.has :foo, classes: [String, Numeric], default: 'foo'
+            end.to_not raise_error(ArgumentError)
+
+            expect do
+              subject.has :foo, classes: Numeric, default: 'foo'
+            end.to raise_error(ArgumentError)
+          end
+        end
 
-          expect do
-            subject.has :foo, values: ['foo', 1], default: 1
-          end.to_not raise_error(ArgumentError)
+        context 'when :values are defined' do
+          it 'only allows one of the specified values' do
+            expect do
+              subject.has :foo, values: 'foo', default: 'foo'
+            end.to_not raise_error(ArgumentError)
 
-          expect do
-            subject.has :foo, values: ['foo', 1], default: 'bar'
-          end.to raise_error(ArgumentError)
+            expect do
+              subject.has :foo, values: ['foo', 1], default: 1
+            end.to_not raise_error(ArgumentError)
+
+            expect do
+              subject.has :foo, values: ['foo', 1], default: 'bar'
+            end.to raise_error(ArgumentError)
+          end
         end
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gem_config
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.4
 platform: ruby
 authors:
 - Manuel Meurer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-05 00:00:00.000000000 Z
+date: 2013-06-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -38,20 +38,6 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 2.13.0
-- !ruby/object:Gem::Dependency
-  name: activesupport
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - '>='
-      - !ruby/object:Gem::Version
-        version: 3.0.0
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - '>='
-      - !ruby/object:Gem::Version
-        version: 3.0.0
 description: A nifty way to make your gem configurable.
 email: manuel.meurer@gmail.com
 executables: []