validations_spec.rb in grape-0.6.1

- old
+ new

@@ -1,231 +1,262 @@
 require 'spec_helper'
 
 describe Grape::Validations do
+
   subject { Class.new(Grape::API) }
-  def app; subject end
 
+  def app
+    subject
+  end
+
   describe 'params' do
     context 'optional' do
       it 'validates when params is present' do
-        subject.params { optional :a_number, :regexp => /^[0-9]+$/ }
-        subject.get '/optional' do 'optional works!'; end
+        subject.params do
+          optional :a_number, regexp: /^[0-9]+$/
+        end
+        subject.get '/optional' do
+          'optional works!'
+        end
 
-        get '/optional', { :a_number => 'string' }
+        get '/optional', a_number: 'string'
         last_response.status.should == 400
         last_response.body.should == 'a_number is invalid'
 
-        get '/optional', { :a_number => 45 }
+        get '/optional', a_number: 45
         last_response.status.should == 200
         last_response.body.should == 'optional works!'
       end
 
       it "doesn't validate when param not present" do
-        subject.params { optional :a_number, :regexp => /^[0-9]+$/ }
-        subject.get '/optional' do 'optional works!'; end
+        subject.params do
+          optional :a_number, regexp: /^[0-9]+$/
+        end
+        subject.get '/optional' do
+          'optional works!'
+        end
 
         get '/optional'
         last_response.status.should == 200
         last_response.body.should == 'optional works!'
       end
 
       it 'adds to declared parameters' do
-        subject.params { optional :some_param }
+        subject.params do
+          optional :some_param
+        end
         subject.settings[:declared_params].should == [:some_param]
       end
     end
 
     context 'required' do
       before do
-        subject.params { requires :key }
-        subject.get '/required' do 'required works'; end
+        subject.params do
+          requires :key
+        end
+        subject.get '/required' do
+          'required works'
+        end
       end
 
       it 'errors when param not present' do
         get '/required'
         last_response.status.should == 400
         last_response.body.should == 'key is missing'
       end
 
       it "doesn't throw a missing param when param is present" do
-        get '/required', { :key => 'cool' }
+        get '/required', key: 'cool'
         last_response.status.should == 200
         last_response.body.should == 'required works'
       end
 
       it 'adds to declared parameters' do
-        subject.params { requires :some_param }
+        subject.params do
+          requires :some_param
+        end
         subject.settings[:declared_params].should == [:some_param]
       end
     end
 
     context 'required with a block' do
       before do
-        subject.params {
+        subject.params do
           requires :items do
             requires :key
           end
-        }
-        subject.get '/required' do 'required works'; end
+        end
+        subject.get '/required' do
+          'required works'
+        end
       end
 
       it 'errors when param not present' do
         get '/required'
         last_response.status.should == 400
         last_response.body.should == 'items[key] is missing'
       end
 
       it "doesn't throw a missing param when param is present" do
-        get '/required', { :items => [:key => 'hello', :key => 'world'] }
+        get '/required', items: [key: 'hello', key: 'world']
         last_response.status.should == 200
         last_response.body.should == 'required works'
       end
 
       it "doesn't allow more than one parameter" do
         expect {
-          subject.params {
+          subject.params do
             requires(:items, desc: 'Foo') do
               requires :key
             end
-          }
+          end
         }.to raise_error ArgumentError
       end
 
       it 'adds to declared parameters' do
-        subject.params {
+        subject.params do
           requires :items do
             requires :key
           end
-        }
-        subject.settings[:declared_params].should == [:items => [:key]]
+        end
+        subject.settings[:declared_params].should == [items: [:key]]
       end
     end
 
     context 'group' do
       before do
-        subject.params {
+        subject.params do
           group :items do
             requires :key
           end
-        }
-        subject.get '/required' do 'required works'; end
+        end
+        subject.get '/required' do
+          'required works'
+        end
       end
 
       it 'errors when param not present' do
         get '/required'
         last_response.status.should == 400
         last_response.body.should == 'items[key] is missing'
       end
 
       it "doesn't throw a missing param when param is present" do
-        get '/required', { :items => [:key => 'hello', :key => 'world'] }
+        get '/required', items: [key: 'hello', key: 'world']
         last_response.status.should == 200
         last_response.body.should == 'required works'
       end
 
       it 'adds to declared parameters' do
-        subject.params {
+        subject.params do
           group :items do
             requires :key
           end
-        }
-        subject.settings[:declared_params].should == [:items => [:key]]
+        end
+        subject.settings[:declared_params].should == [items: [:key]]
       end
     end
 
     context 'optional with a block' do
       before do
-        subject.params {
+        subject.params do
           optional :items do
             requires :key
           end
-        }
-        subject.get '/optional_group' do 'optional group works'; end
+        end
+        subject.get '/optional_group' do
+          'optional group works'
+        end
       end
 
       it "doesn't throw a missing param when the group isn't present" do
         get '/optional_group'
         last_response.status.should == 200
         last_response.body.should == 'optional group works'
       end
 
       it "doesn't throw a missing param when both group and param are given" do
-        get '/optional_group', { :items => {:key => 'foo'} }
+        get '/optional_group', items: { key: 'foo' }
         last_response.status.should == 200
         last_response.body.should == 'optional group works'
       end
 
       it "errors when group is present, but required param is not" do
-        get '/optional_group', { :items => {:NOT_key => 'foo'} }
+        get '/optional_group', items: { not_key: 'foo' }
         last_response.status.should == 400
         last_response.body.should == 'items[key] is missing'
       end
 
       it 'adds to declared parameters' do
-        subject.params {
+        subject.params do
           optional :items do
             requires :key
           end
-        }
-        subject.settings[:declared_params].should == [:items => [:key]]
+        end
+        subject.settings[:declared_params].should == [items: [:key]]
       end
     end
 
     context 'nested optional blocks' do
       before do
-        subject.params {
+        subject.params do
           optional :items do
             requires :key
             optional(:optional_subitems) { requires :value }
             requires(:required_subitems) { requires :value }
           end
-        }
+        end
         subject.get('/nested_optional_group') { 'nested optional group works' }
       end
 
       it 'does no internal validations if the outer group is blank' do
         get '/nested_optional_group'
         last_response.status.should == 200
         last_response.body.should == 'nested optional group works'
       end
 
       it 'does internal validations if the outer group is present' do
-        get '/nested_optional_group', { :items => {:key => 'foo' }}
+        get '/nested_optional_group', items: { key: 'foo' }
         last_response.status.should == 400
         last_response.body.should == 'items[required_subitems][value] is missing'
 
-        get '/nested_optional_group', { :items => { :key => 'foo', :required_subitems => {:value => 'bar'}}}
+        get '/nested_optional_group', items: { key: 'foo', required_subitems: { value: 'bar' } }
         last_response.status.should == 200
         last_response.body.should == 'nested optional group works'
       end
 
       it 'handles deep nesting' do
-        get '/nested_optional_group', { :items => { :key => 'foo', :required_subitems => {:value => 'bar'}, :optional_subitems => {:NOT_value => 'baz'}}}
+        get '/nested_optional_group', items: { key: 'foo', required_subitems: { value: 'bar' }, optional_subitems: { not_value: 'baz' } }
         last_response.status.should == 400
         last_response.body.should == 'items[optional_subitems][value] is missing'
 
-        get '/nested_optional_group', { :items => { :key => 'foo', :required_subitems => {:value => 'bar'}, :optional_subitems => {:value => 'baz'}}}
+        get '/nested_optional_group', items: { key: 'foo', required_subitems: { value: 'bar' }, optional_subitems: { value: 'baz' } }
         last_response.status.should == 200
         last_response.body.should == 'nested optional group works'
       end
 
       it 'adds to declared parameters' do
-        subject.params {
+        subject.params do
           optional :items do
             requires :key
             optional(:optional_subitems) { requires :value }
             requires(:required_subitems) { requires :value }
           end
-        }
-        subject.settings[:declared_params].should == [:items => [:key, {:optional_subitems => [:value]}, {:required_subitems => [:value]}]]
+        end
+        subject.settings[:declared_params].should == [items: [:key, { optional_subitems: [:value] }, { required_subitems: [:value] }]]
       end
     end
 
     context 'multiple validation errors' do
       before do
-        subject.params { requires :yolo; requires :swag }
-        subject.get '/two_required' do 'two required works'; end
+        subject.params do
+          requires :yolo
+          requires :swag
+        end
+        subject.get '/two_required' do
+          'two required works'
+        end
       end
 
       it 'throws the validation errors' do
         get '/two_required'
         last_response.status.should == 400
@@ -237,28 +268,32 @@
     context 'custom validation' do
       module CustomValidations
         class Customvalidator < Grape::Validations::Validator
           def validate_param!(attr_name, params)
             unless params[attr_name] == 'im custom'
-              raise Grape::Exceptions::Validation, :param => @scope.full_name(attr_name), :message => "is not custom!"
+              raise Grape::Exceptions::Validation, param: @scope.full_name(attr_name), message: "is not custom!"
             end
           end
         end
       end
 
       context 'when using optional with a custom validator' do
         before do
-          subject.params { optional :custom, :customvalidator => true }
-          subject.get '/optional_custom' do 'optional with custom works!'; end
+          subject.params do
+            optional :custom, customvalidator: true
+          end
+          subject.get '/optional_custom' do
+            'optional with custom works!'
+          end
         end
 
         it 'validates when param is present' do
-          get '/optional_custom', { :custom => 'im custom' }
+          get '/optional_custom', { custom: 'im custom' }
           last_response.status.should == 200
           last_response.body.should == 'optional with custom works!'
 
-          get '/optional_custom', { :custom => 'im wrong' }
+          get '/optional_custom', { custom: 'im wrong' }
           last_response.status.should == 400
           last_response.body.should == 'custom is not custom!'
         end
 
         it "skips validation when parameter isn't present" do
@@ -266,30 +301,36 @@
           last_response.status.should == 200
           last_response.body.should == 'optional with custom works!'
         end
 
         it 'validates with custom validator when param present and incorrect type' do
-          subject.params { optional :custom, :type => String, :customvalidator => true }
+          subject.params do
+            optional :custom, type: String, customvalidator: true
+          end
 
-          get '/optional_custom', { :custom => 123 }
+          get '/optional_custom', custom: 123
           last_response.status.should == 400
           last_response.body.should == 'custom is not custom!'
         end
       end
 
       context 'when using requires with a custom validator' do
         before do
-          subject.params { requires :custom, :customvalidator => true }
-          subject.get '/required_custom' do 'required with custom works!'; end
+          subject.params do
+            requires :custom, customvalidator: true
+          end
+          subject.get '/required_custom' do
+            'required with custom works!'
+          end
         end
 
         it 'validates when param is present' do
-          get '/required_custom', { :custom => 'im wrong, validate me' }
+          get '/required_custom', custom: 'im wrong, validate me'
           last_response.status.should == 400
           last_response.body.should == 'custom is not custom!'
 
-          get '/required_custom', { :custom => 'im custom' }
+          get '/required_custom', custom: 'im custom'
           last_response.status.should == 200
           last_response.body.should == 'required with custom works!'
         end
 
         it 'validates when param is not present' do
@@ -298,53 +339,69 @@
           last_response.body.should == 'custom is missing, custom is not custom!'
         end
 
         context 'nested namespaces' do
           before do
-            subject.params { requires :custom, :customvalidator => true }
+            subject.params do
+              requires :custom, customvalidator: true
+            end
             subject.namespace 'nested' do
-              get 'one' do 'validation failed' end
+              get 'one' do
+                'validation failed'
+              end
               namespace 'nested' do
-                get 'two' do 'validation failed' end
+                get 'two' do
+                  'validation failed'
+                end
               end
             end
             subject.namespace 'peer' do
-              get 'one' do 'no validation required' end
+              get 'one' do
+                'no validation required'
+              end
               namespace 'nested' do
-                get 'two' do 'no validation required' end
+                get 'two' do
+                  'no validation required'
+                end
               end
             end
 
             subject.namespace 'unrelated' do
-              params{ requires :name }
-              get 'one' do 'validation required'; end
+              params do
+                requires :name
+              end
+              get 'one' do
+                'validation required'
+              end
 
               namespace 'double' do
-                get 'two' do 'no validation required' end
+                get 'two' do
+                  'no validation required'
+                end
               end
             end
           end
 
           specify 'the parent namespace uses the validator' do
-            get '/nested/one', { :custom => 'im wrong, validate me'}
+            get '/nested/one', custom: 'im wrong, validate me'
             last_response.status.should == 400
             last_response.body.should == 'custom is not custom!'
           end
 
           specify 'the nested namesapce inherits the custom validator' do
-            get '/nested/nested/two', { :custom => 'im wrong, validate me'}
+            get '/nested/nested/two', custom: 'im wrong, validate me'
             last_response.status.should == 400
             last_response.body.should == 'custom is not custom!'
           end
 
           specify 'peer namesapces does not have the validator' do
-            get '/peer/one', { :custom => 'im not validated' }
+            get '/peer/one', custom: 'im not validated'
             last_response.status.should == 200
             last_response.body.should == 'no validation required'
           end
 
           specify 'namespaces nested in peers should also not have the validator' do
-            get '/peer/nested/two', { :custom => 'im not validated' }
+            get '/peer/nested/two', custom: 'im not validated'
             last_response.status.should == 200
             last_response.body.should == 'no validation required'
           end
 
           specify 'when nested, specifying a route should clear out the validations for deeper nested params' do