grape 0.12.0 → 0.14.0

data/spec/grape/validations_spec.rb

@@ -318,6 +318,35 @@ describe Grape::Validations do
       end
     end
 
+    context 'hash with a required param with validation' do
+      before do
+        subject.params do
+          requires :items, type: Hash do
+            requires :key, type: String, values: %w(a b)
+          end
+        end
+        subject.get '/required' do
+          'required works'
+        end
+      end
+
+      it 'errors when param is not a Hash' do
+        get '/required', items: 'not a hash'
+        expect(last_response.status).to eq(400)
+        expect(last_response.body).to eq('items is invalid, items[key] is missing, items[key] is invalid')
+
+        get '/required', items: [{ key: 'hash in array' }]
+        expect(last_response.status).to eq(400)
+        expect(last_response.body).to eq('items is invalid, items[0][key] does not have a valid value')
+      end
+
+      it 'works when all params match' do
+        get '/required', items: { key: 'a' }
+        expect(last_response.status).to eq(200)
+        expect(last_response.body).to eq('required works')
+      end
+    end
+
     context 'group' do
       before do
         subject.params do
@@ -374,10 +403,11 @@ describe Grape::Validations do
     end
 
     context 'custom validator for a Hash' do
-      module DateRangeValidations
-        class DateRangeValidator < Grape::Validations::Base
-          def validate_param!(attr_name, params)
-            unless params[attr_name][:from] <= params[attr_name][:to]
+      module ValuesSpec
+        module DateRangeValidations
+          class DateRangeValidator < Grape::Validations::Base
+            def validate_param!(attr_name, params)
+              return if params[attr_name][:from] <= params[attr_name][:to]
               fail Grape::Exceptions::Validation, params: [@scope.full_name(attr_name)], message: "'from' must be lower or equal to 'to'"
             end
           end
@@ -462,7 +492,7 @@ describe Grape::Validations do
         # NOTE: with body parameters in json or XML or similar this
         # should actually fail with: children[parents][name] is missing.
         expect(last_response.status).to eq(400)
-        expect(last_response.body).to eq('children[parents] is missing')
+        expect(last_response.body).to eq('children[0][parents] is missing')
       end
 
       it 'safely handles empty arrays and blank parameters' do
@@ -473,7 +503,7 @@ describe Grape::Validations do
         expect(last_response.body).to eq('children is missing')
         get '/within_array', children: [name: 'Jay']
         expect(last_response.status).to eq(400)
-        expect(last_response.body).to eq('children[parents] is missing')
+        expect(last_response.body).to eq('children[0][parents] is missing')
       end
 
       it 'errors when param is not an Array' do
@@ -488,7 +518,7 @@ describe Grape::Validations do
 
         get '/within_array', children: [name: 'Jay', parents: { name: 'Fred' }]
         expect(last_response.status).to eq(400)
-        expect(last_response.body).to eq('children[parents] is invalid')
+        expect(last_response.body).to eq('children[0][parents] is invalid')
       end
     end
 
@@ -614,7 +644,7 @@ describe Grape::Validations do
           { name: 'Job', parents: [{ name: 'Joy' }] }
         ]
         expect(last_response.status).to eq(400)
-        expect(last_response.body).to eq('children[parents][name] is missing')
+        expect(last_response.body).to eq('children[0][parents][0][name] is missing')
       end
 
       it 'safely handles empty arrays and blank parameters' do
@@ -622,7 +652,7 @@ describe Grape::Validations do
         expect(last_response.status).to eq(200)
         put_with_json '/within_array', children: [name: 'Jay']
         expect(last_response.status).to eq(400)
-        expect(last_response.body).to eq('children[parents] is missing')
+        expect(last_response.body).to eq('children[0][parents] is missing')
       end
     end
 
@@ -653,7 +683,7 @@ describe Grape::Validations do
       it 'errors when group is present, but required param is not' do
         get '/optional_group', items: [{ not_key: 'foo' }]
         expect(last_response.status).to eq(400)
-        expect(last_response.body).to eq('items[key] is missing')
+        expect(last_response.body).to eq('items[0][key] is missing')
       end
 
       it "errors when param is present but isn't an Array" do
@@ -697,7 +727,7 @@ describe Grape::Validations do
       it 'does internal validations if the outer group is present' do
         get '/nested_optional_group', items: [{ key: 'foo' }]
         expect(last_response.status).to eq(400)
-        expect(last_response.body).to eq('items[required_subitems] is missing')
+        expect(last_response.body).to eq('items[0][required_subitems] is missing')
 
         get '/nested_optional_group', items: [{ key: 'foo', required_subitems: [{ value: 'bar' }] }]
         expect(last_response.status).to eq(200)
@@ -707,7 +737,7 @@ describe Grape::Validations do
       it 'handles deep nesting' do
         get '/nested_optional_group', items: [{ key: 'foo', required_subitems: [{ value: 'bar' }], optional_subitems: [{ not_value: 'baz' }] }]
         expect(last_response.status).to eq(400)
-        expect(last_response.body).to eq('items[optional_subitems][value] is missing')
+        expect(last_response.body).to eq('items[0][optional_subitems][0][value] is missing')
 
         get '/nested_optional_group', items: [{ key: 'foo', required_subitems: [{ value: 'bar' }], optional_subitems: [{ value: 'baz' }] }]
         expect(last_response.status).to eq(200)
@@ -717,7 +747,7 @@ describe Grape::Validations do
       it 'handles validation within arrays' do
         get '/nested_optional_group', items: [{ key: 'foo' }]
         expect(last_response.status).to eq(400)
-        expect(last_response.body).to eq('items[required_subitems] is missing')
+        expect(last_response.body).to eq('items[0][required_subitems] is missing')
 
         get '/nested_optional_group', items: [{ key: 'foo', required_subitems: [{ value: 'bar' }] }]
         expect(last_response.status).to eq(200)
@@ -725,7 +755,7 @@ describe Grape::Validations do
 
         get '/nested_optional_group', items: [{ key: 'foo', required_subitems: [{ value: 'bar' }], optional_subitems: [{ not_value: 'baz' }] }]
         expect(last_response.status).to eq(400)
-        expect(last_response.body).to eq('items[optional_subitems][value] is missing')
+        expect(last_response.body).to eq('items[0][optional_subitems][0][value] is missing')
       end
 
       it 'adds to declared parameters' do
@@ -763,9 +793,8 @@ describe Grape::Validations do
       module CustomValidations
         class Customvalidator < Grape::Validations::Base
           def validate_param!(attr_name, params)
-            unless params[attr_name] == 'im custom'
-              fail Grape::Exceptions::Validation, params: [@scope.full_name(attr_name)], message: 'is not custom!'
-            end
+            return if params[attr_name] == 'im custom'
+            fail Grape::Exceptions::Validation, params: [@scope.full_name(attr_name)], message: 'is not custom!'
           end
         end
       end
@@ -912,9 +941,8 @@ describe Grape::Validations do
         module CustomValidations
           class CustomvalidatorWithOptions < Grape::Validations::Base
             def validate_param!(attr_name, params)
-              unless params[attr_name] == @option[:text]
-                fail Grape::Exceptions::Validation, params: [@scope.full_name(attr_name)], message: @option[:error_message]
-              end
+              return if params[attr_name] == @option[:text]
+              fail Grape::Exceptions::Validation, params: [@scope.full_name(attr_name)], message: @option[:error_message]
             end
           end
         end
@@ -950,25 +978,26 @@ describe Grape::Validations do
         end
 
         it 'in helper module which kind of Grape::DSL::Helpers::BaseHelper' do
-          module SharedParams
+          shared_params = Module.new do
             extend Grape::DSL::Helpers::BaseHelper
             params :pagination do
             end
           end
-          subject.helpers SharedParams
+          subject.helpers shared_params
         end
       end
 
       context 'can be included in usual params' do
         before do
-          module SharedParams
+          shared_params = Module.new do
             extend Grape::DSL::Helpers::BaseHelper
             params :period do
               optional :start_date
               optional :end_date
             end
           end
-          subject.helpers SharedParams
+
+          subject.helpers shared_params
 
           subject.helpers do
             params :pagination do

data/spec/shared/versioning_examples.rb

@@ -3,7 +3,7 @@ shared_examples_for 'versioning' do
     subject.format :txt
     subject.version 'v1', macro_options
     subject.get :hello do
-      "Version: #{request.env['api.version'] }"
+      "Version: #{request.env['api.version']}"
     end
     versioned_get '/hello', 'v1', macro_options
     expect(last_response.body).to eql 'Version: v1'
@@ -14,7 +14,7 @@ shared_examples_for 'versioning' do
     subject.prefix 'api'
     subject.version 'v1', macro_options
     subject.get :hello do
-      "Version: #{request.env['api.version'] }"
+      "Version: #{request.env['api.version']}"
     end
     versioned_get '/hello', 'v1', macro_options.merge(prefix: 'api')
     expect(last_response.body).to eql 'Version: v1'

data/spec/spec_helper.rb

@@ -12,7 +12,6 @@ require 'json'
 require 'rack/test'
 require 'base64'
 require 'cookiejar'
-require 'json'
 require 'mime/types'
 
 Dir["#{File.dirname(__FILE__)}/support/*.rb"].each do |file|

data/spec/support/versioned_helpers.rb

@@ -26,13 +26,13 @@ def versioned_headers(options)
   when :header
     {
       'HTTP_ACCEPT' => [
-        "application/vnd.#{options[:vendor] }-#{options[:version] }",
+        "application/vnd.#{options[:vendor]}-#{options[:version]}",
         options[:format]
       ].compact.join('+')
     }
   when :accept_version_header
     {
-      'HTTP_ACCEPT_VERSION' => "#{options[:version] }"
+      'HTTP_ACCEPT_VERSION' => "#{options[:version]}"
     }
   else
     fail ArgumentError.new("unknown versioning strategy: #{options[:using]}")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: grape
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 0.14.0
 platform: ruby
 authors:
 - Michael Bleigh
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-06-18 00:00:00.000000000 Z
+date: 2015-12-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -264,6 +264,20 @@ dependencies:
         version: '0'
 - !ruby/object:Gem::Dependency
   name: mime-types
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - <
+      - !ruby/object:Gem::Version
+        version: '3.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - <
+      - !ruby/object:Gem::Version
+        version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: appraisal
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
@@ -277,7 +291,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: appraisal
+  name: benchmark-ips
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
@@ -290,6 +304,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.35.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.35.1
 description: A Ruby framework for rapid API development with great conventions.
 email:
 - michael@intridea.com
@@ -297,28 +325,23 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rspec
-- .rubocop.yml
-- .rubocop_todo.yml
-- .travis.yml
-- .yardopts
 - Appraisals
 - CHANGELOG.md
 - CONTRIBUTING.md
 - Gemfile
+- Gemfile.lock
 - Guardfile
 - LICENSE
 - README.md
 - RELEASING.md
 - Rakefile
 - UPGRADING.md
+- benchmark/simple.rb
+- gemfiles/rack_1.5.2.gemfile
 - gemfiles/rails_3.gemfile
 - gemfiles/rails_4.gemfile
 - grape.gemspec
 - grape.png
-- lib/backports/active_support/deep_dup.rb
-- lib/backports/active_support/duplicable.rb
 - lib/grape.rb
 - lib/grape/api.rb
 - lib/grape/api/helpers.rb
@@ -344,6 +367,7 @@ files:
 - lib/grape/exceptions/invalid_accept_header.rb
 - lib/grape/exceptions/invalid_formatter.rb
 - lib/grape/exceptions/invalid_message_body.rb
+- lib/grape/exceptions/invalid_version_header.rb
 - lib/grape/exceptions/invalid_versioner_option.rb
 - lib/grape/exceptions/invalid_with_option_for_represent.rb
 - lib/grape/exceptions/missing_group_type.rb
@@ -351,6 +375,7 @@ files:
 - lib/grape/exceptions/missing_option.rb
 - lib/grape/exceptions/missing_vendor_option.rb
 - lib/grape/exceptions/unknown_options.rb
+- lib/grape/exceptions/unknown_parameter.rb
 - lib/grape/exceptions/unknown_validator.rb
 - lib/grape/exceptions/unsupported_group_type.rb
 - lib/grape/exceptions/validation.rb
@@ -361,7 +386,6 @@ files:
 - lib/grape/formatter/txt.rb
 - lib/grape/formatter/xml.rb
 - lib/grape/http/headers.rb
-- lib/grape/http/request.rb
 - lib/grape/locale/en.yml
 - lib/grape/middleware/auth/base.rb
 - lib/grape/middleware/auth/dsl.rb
@@ -376,6 +400,7 @@ files:
 - lib/grape/middleware/versioner/accept_version_header.rb
 - lib/grape/middleware/versioner/header.rb
 - lib/grape/middleware/versioner/param.rb
+- lib/grape/middleware/versioner/parse_media_type_patch.rb
 - lib/grape/middleware/versioner/path.rb
 - lib/grape/namespace.rb
 - lib/grape/parser/base.rb
@@ -383,8 +408,11 @@ files:
 - lib/grape/parser/xml.rb
 - lib/grape/path.rb
 - lib/grape/presenters/presenter.rb
+- lib/grape/request.rb
 - lib/grape/route.rb
 - lib/grape/util/content_types.rb
+- lib/grape/util/env.rb
+- lib/grape/util/file_response.rb
 - lib/grape/util/inheritable_setting.rb
 - lib/grape/util/inheritable_values.rb
 - lib/grape/util/stackable_values.rb
@@ -392,6 +420,14 @@ files:
 - lib/grape/validations.rb
 - lib/grape/validations/attributes_iterator.rb
 - lib/grape/validations/params_scope.rb
+- lib/grape/validations/types.rb
+- lib/grape/validations/types/build_coercer.rb
+- lib/grape/validations/types/custom_type_coercer.rb
+- lib/grape/validations/types/file.rb
+- lib/grape/validations/types/json.rb
+- lib/grape/validations/types/multiple_type_coercer.rb
+- lib/grape/validations/types/variant_collection_coercer.rb
+- lib/grape/validations/types/virtus_collection_patch.rb
 - lib/grape/validations/validators/all_or_none.rb
 - lib/grape/validations/validators/allow_blank.rb
 - lib/grape/validations/validators/at_least_one_of.rb
@@ -405,6 +441,7 @@ files:
 - lib/grape/validations/validators/regexp.rb
 - lib/grape/validations/validators/values.rb
 - lib/grape/version.rb
+- pkg/grape-0.13.0.gem
 - spec/grape/api/custom_validations_spec.rb
 - spec/grape/api/deeply_included_options_spec.rb
 - spec/grape/api/nested_helpers_spec.rb
@@ -448,12 +485,14 @@ files:
 - spec/grape/middleware/versioner_spec.rb
 - spec/grape/path_spec.rb
 - spec/grape/presenters/presenter_spec.rb
+- spec/grape/request_spec.rb
 - spec/grape/util/inheritable_setting_spec.rb
 - spec/grape/util/inheritable_values_spec.rb
 - spec/grape/util/stackable_values_spec.rb
 - spec/grape/util/strict_hash_configuration_spec.rb
 - spec/grape/validations/attributes_iterator_spec.rb
 - spec/grape/validations/params_scope_spec.rb
+- spec/grape/validations/types_spec.rb
 - spec/grape/validations/validators/all_or_none_spec.rb
 - spec/grape/validations/validators/allow_blank_spec.rb
 - spec/grape/validations/validators/at_least_one_of_spec.rb
@@ -473,7 +512,7 @@ files:
 - spec/support/endpoint_faker.rb
 - spec/support/file_streamer.rb
 - spec/support/versioned_helpers.rb
-homepage: https://github.com/intridea/grape
+homepage: https://github.com/ruby-grape/grape
 licenses:
 - MIT
 metadata: {}
@@ -492,7 +531,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: grape
+rubyforge_project: 
 rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
@@ -541,12 +580,14 @@ test_files:
 - spec/grape/middleware/versioner_spec.rb
 - spec/grape/path_spec.rb
 - spec/grape/presenters/presenter_spec.rb
+- spec/grape/request_spec.rb
 - spec/grape/util/inheritable_setting_spec.rb
 - spec/grape/util/inheritable_values_spec.rb
 - spec/grape/util/stackable_values_spec.rb
 - spec/grape/util/strict_hash_configuration_spec.rb
 - spec/grape/validations/attributes_iterator_spec.rb
 - spec/grape/validations/params_scope_spec.rb
+- spec/grape/validations/types_spec.rb
 - spec/grape/validations/validators/all_or_none_spec.rb
 - spec/grape/validations/validators/allow_blank_spec.rb
 - spec/grape/validations/validators/at_least_one_of_spec.rb

data/.gitignore

@@ -1,46 +0,0 @@
-## MAC OS
-.DS_Store
-.com.apple.timemachine.supported
-
-## TEXTMATE
-*.tmproj
-tmtags
-
-## EMACS
-*~
-\#*
-.\#*
-
-## REDCAR
-.redcar
-
-## VIM
-*.swp
-*.swo
-
-## RUBYMINE
-.idea
-
-## PROJECT::GENERAL
-coverage
-doc
-pkg
-.rvmrc
-.bundle
-.yardoc/*
-dist
-Gemfile.lock
-gemfiles/*.lock
-tmp
-
-## Rubinius
-.rbx
-
-## Bundler binstubs
-bin
-
-## ripper-tags and gem-ctags
-tags
-
-## PROJECT::SPECIFIC
-.project

data/.rspec

@@ -1,2 +0,0 @@
---color
---format=documentation

data/.rubocop.yml

@@ -1,7 +0,0 @@
-AllCops:
-  Exclude:
-    - vendor/**/*
-    - bin/**/*
-    - gemfiles/**/*
-
-inherit_from: .rubocop_todo.yml

data/.rubocop_todo.yml

@@ -1,84 +0,0 @@
-# This configuration was generated by `rubocop --auto-gen-config`
-# on 2015-06-04 09:15:17 -0400 using RuboCop version 0.31.0.
-# The point is for the user to remove these configuration records
-# one by one as the offenses are removed from the code base.
-# Note that changes in the inspected code, or installation of new
-# versions of RuboCop, may require this file to be generated again.
-
-# Offense count: 37
-Metrics/AbcSize:
-  Max: 48
-
-# Offense count: 2
-Metrics/BlockNesting:
-  Max: 4
-
-# Offense count: 4
-# Configuration parameters: CountComments.
-Metrics/ClassLength:
-  Max: 246
-
-# Offense count: 23
-Metrics/CyclomaticComplexity:
-  Max: 20
-
-# Offense count: 675
-# Configuration parameters: AllowURI, URISchemes.
-Metrics/LineLength:
-  Max: 198
-
-# Offense count: 44
-# Configuration parameters: CountComments.
-Metrics/MethodLength:
-  Max: 35
-
-# Offense count: 8
-# Configuration parameters: CountComments.
-Metrics/ModuleLength:
-  Max: 243
-
-# Offense count: 17
-Metrics/PerceivedComplexity:
-  Max: 22
-
-# Offense count: 26
-# Cop supports --auto-correct.
-# Configuration parameters: EnforcedStyle, SupportedStyles, ProceduralMethods, FunctionalMethods, IgnoredMethods.
-Style/BlockDelimiters:
-  Enabled: false
-
-# Offense count: 174
-Style/Documentation:
-  Enabled: false
-
-# Offense count: 7
-Style/DoubleNegation:
-  Enabled: false
-
-# Offense count: 5
-Style/EachWithObject:
-  Enabled: false
-
-# Offense count: 15
-# Configuration parameters: MinBodyLength.
-Style/GuardClause:
-  Enabled: false
-
-# Offense count: 4
-# Cop supports --auto-correct.
-Style/Lambda:
-  Enabled: false
-
-# Offense count: 3
-Style/MultilineTernaryOperator:
-  Enabled: false
-
-# Offense count: 3
-# Configuration parameters: NamePrefix, NamePrefixBlacklist.
-Style/PredicateName:
-  Enabled: false
-
-# Offense count: 13
-# Configuration parameters: EnforcedStyle, SupportedStyles.
-Style/RaiseArgs:
-  Enabled: false

data/.travis.yml

@@ -1,20 +0,0 @@
-language: ruby
-
-rvm:
-  - 2.2
-  - 2.1
-  - 2.0.0
-  - rbx-2.2.10
-  - jruby-19mode
-  - ruby-head
-  - jruby-head
-
-matrix:
-  allow_failures:
-    - rvm: ruby-head
-    - rvm: jruby-head
-
-gemfile:
-  - Gemfile
-  - gemfiles/rails_3.gemfile
-  - gemfiles/rails_4.gemfile

data/.yardopts

@@ -1,2 +0,0 @@
---markup-provider=redcarpet
---markup=markdown

data/lib/backports/active_support/deep_dup.rb

@@ -1,49 +0,0 @@
-# Backport from Rails 4.x
-# https://github.com/rails/rails/blob/4-0-stable/activesupport/lib/active_support/core_ext/object/deep_dup.rb
-
-require_relative 'duplicable'
-
-class Object
-  # Returns a deep copy of object if it's duplicable. If it's
-  # not duplicable, returns +self+.
-  #
-  #   object = Object.new
-  #   dup    = object.deep_dup
-  #   dup.instance_variable_set(:@a, 1)
-  #
-  #   object.instance_variable_defined?(:@a) #=> false
-  #   dup.instance_variable_defined?(:@a)    #=> true
-  def deep_dup
-    duplicable? ? dup : self
-  end
-end
-
-class Array
-  # Returns a deep copy of array.
-  #
-  #   array = [1, [2, 3]]
-  #   dup   = array.deep_dup
-  #   dup[1][2] = 4
-  #
-  #   array[1][2] #=> nil
-  #   dup[1][2]   #=> 4
-  def deep_dup
-    map(&:deep_dup)
-  end
-end
-
-class Hash
-  # Returns a deep copy of hash.
-  #
-  #   hash = { a: { b: 'b' } }
-  #   dup  = hash.deep_dup
-  #   dup[:a][:c] = 'c'
-  #
-  #   hash[:a][:c] #=> nil
-  #   dup[:a][:c]  #=> "c"
-  def deep_dup
-    each_with_object(dup) do |(key, value), hash|
-      hash[key.deep_dup] = value.deep_dup
-    end
-  end
-end