Merge pull request #1543 from ruby-grape/support-ruby-2.4

Support ruby 2.4

Author: dblock

.travis.yml

@@ -4,9 +4,19 @@ sudo: false
 
 matrix:
   include:
-    - rvm: 2.3.3
+    - rvm: 2.4.0
       script:
         - bundle exec danger
+    - rvm: 2.4.0
+      gemfile: Gemfile
+    - rvm: 2.4.0
+      gemfile: gemfiles/rack_edge.gemfile
+    - rvm: 2.4.0
+      gemfile: gemfiles/rack_1.5.2.gemfile
+    - rvm: 2.4.0
+      gemfile: gemfiles/rails_edge.gemfile
+    - rvm: 2.4.0
+      gemfile: gemfiles/rails_5.gemfile
     - rvm: 2.3.3
       gemfile: Gemfile
     - rvm: 2.3.3

CHANGELOG.md

@@ -1,6 +1,13 @@
-#### 0.19.1 (Next)
+### 0.19.1 (Next)
+
+#### Features
 
 * [#1536](https://github.com/ruby-grape/grape/pull/1536): Updates `invalid_versioner_option` translation - [@Lavode](https://github.com/Lavode).
+* [#1543](https://github.com/ruby-grape/grape/pull/1543): Support ruby 2.4 - [@LeFnord](https://github.com/LeFnord), [@namusyaka](https://github.com/namusyaka).
+* Your contribution here.
+
+#### Fixes
+
 * Your contribution here.
 
 ### 0.19.0 (12/18/2016)

README.md

@@ -32,6 +32,7 @@
   - [Include Missing](#include-missing)
 - [Parameter Validation and Coercion](#parameter-validation-and-coercion)
   - [Supported Parameter Types](#supported-parameter-types)
+  - [Integer/Fixnum and Coercions](#integerfixnum-and-coercions)
   - [Custom Types and Coercions](#custom-types-and-coercions)
   - [Multipart File Parameters](#multipart-file-parameters)
   - [First-Class `JSON` Types](#first-class-json-types)
@@ -799,6 +800,28 @@ The following are all valid types, supported out of the box by Grape:
 * Rack::Multipart::UploadedFile (alias `File`)
 * JSON
 
+### Integer/Fixnum and Coercions
+
+Please be aware that the behavior differs between Ruby 2.4 and earlier versions.
+In Ruby 2.4, values consisting of numbers are converted to Integer, but in earlier versions it will be treated as Fixnum.
+
+```ruby
+params do
+  requires :integers, type: Hash do
+    requires :int, coerce: Integer
+  end
+end
+get '/int' do
+  params[:integers][:int].class
+end
+
+...
+
+get '/int' integers: { int: '45' }
+  #=> Integer in ruby 2.4
+  #=> Fixnum in earlier ruby versions
+```
+
 ### Custom Types and Coercions
 
 Aside from the default set of supported types listed above, any class can be
@@ -2878,7 +2901,7 @@ end
 The behaviour is then:
 
 ```bash
-GET /123        # 'Fixnum'
+GET /123        # 'Integer'
 GET /foo        # 400 error - 'blah is invalid'
 ```
 

lib/grape/dsl/inside_route.rb

@@ -122,7 +122,7 @@ def status(status = nil)
         when Symbol
           raise ArgumentError, "Status code :#{status} is invalid." unless Rack::Utils::SYMBOL_TO_STATUS_CODE.keys.include?(status)
           @status = Rack::Utils.status_code(status)
-        when Fixnum
+        when Integer
           @status = status
         when nil
           return @status if @status
@@ -135,7 +135,7 @@ def status(status = nil)
             200
           end
         else
-          raise ArgumentError, 'Status code must be Fixnum or Symbol.'
+          raise ArgumentError, 'Status code must be Integer or Symbol.'
         end
       end
 

spec/grape/api_spec.rb

@@ -925,7 +925,7 @@ class DummyFormatClass
       end
 
       get '/', id: '32'
-      expect(last_response.body).to eql 'first 32:Fixnum second'
+      expect(last_response.body).to eql "first 32:#{integer_class_name} second"
     end
 
     it 'adds a after filter' do
@@ -2506,7 +2506,7 @@ def static
       end
     end
     describe 'status' do
-      it 'can be set to arbitrary Fixnum value' do
+      it 'can be set to arbitrary Integer value' do
         subject.get '/foo' do
           status 210
         end

spec/grape/dsl/inside_route_spec.rb

@@ -126,13 +126,13 @@ def initialize
         .to raise_error(ArgumentError, 'Status code :foo_bar is invalid.')
     end
 
-    it 'accepts unknown Fixnum status codes' do
+    it 'accepts unknown Integer status codes' do
       expect { subject.status 210 }.to_not raise_error
     end
 
-    it 'raises error if status is not a fixnum or symbol' do
+    it 'raises error if status is not a integer or symbol' do
       expect { subject.status Object.new }
-        .to raise_error(ArgumentError, 'Status code must be Fixnum or Symbol.')
+        .to raise_error(ArgumentError, 'Status code must be Integer or Symbol.')
     end
   end
 

spec/grape/dsl/request_response_spec.rb

@@ -131,7 +131,7 @@ def self.imbue(key, value)
           end
 
           it 'abort if :with option value is not Symbol, String or Proc' do
-            expect { subject.rescue_from :all, with: 1234 }.to raise_error(ArgumentError, 'with: Fixnum, expected Symbol, String or Proc')
+            expect { subject.rescue_from :all, with: 1234 }.to raise_error(ArgumentError, "with: #{integer_class_name}, expected Symbol, String or Proc")
           end
 
           it 'abort if both :with option and block are passed' do

spec/grape/validations/validators/allow_blank_spec.rb

@@ -57,9 +57,9 @@ class API < Grape::API
         get '/allow_float_blank'
 
         params do
-          requires :val, type: Fixnum, allow_blank: true
+          requires :val, type: Integer, allow_blank: true
         end
-        get '/allow_fixnum_blank'
+        get '/allow_integer_blank'
 
         params do
           requires :val, type: Symbol, allow_blank: true
@@ -173,9 +173,9 @@ class API < Grape::API
           get '/allow_float_blank'
 
           params do
-            requires :val, type: Fixnum, allow_blank: true
+            requires :val, type: Integer, allow_blank: true
           end
-          get '/allow_fixnum_blank'
+          get '/allow_integer_blank'
 
           params do
             requires :val, type: Symbol, allow_blank: true
@@ -345,8 +345,8 @@ def app
           expect(last_response.status).to eq(200)
         end
 
-        it 'accepts empty when fixnum allow_blank' do
-          get '/custom_message/allow_fixnum_blank', val: ''
+        it 'accepts empty when integer allow_blank' do
+          get '/custom_message/allow_integer_blank', val: ''
           expect(last_response.status).to eq(200)
         end
       end
@@ -483,8 +483,8 @@ def app
         expect(last_response.status).to eq(200)
       end
 
-      it 'accepts empty when fixnum allow_blank' do
-        get '/allow_fixnum_blank', val: ''
+      it 'accepts empty when integer allow_blank' do
+        get '/allow_integer_blank', val: ''
         expect(last_response.status).to eq(200)
       end
     end

spec/grape/validations/validators/coerce_spec.rb

@@ -175,7 +175,7 @@ class User
 
         get '/int', int: '45'
         expect(last_response.status).to eq(200)
-        expect(last_response.body).to eq('Fixnum')
+        expect(last_response.body).to eq(integer_class_name)
       end
 
       context 'Array' do
@@ -189,7 +189,7 @@ class User
 
           get '/array', arry: %w(1 2 3)
           expect(last_response.status).to eq(200)
-          expect(last_response.body).to eq('Fixnum')
+          expect(last_response.body).to eq(integer_class_name)
         end
 
         it 'Array of Bools' do
@@ -238,7 +238,7 @@ class User
 
           get '/set', set: Set.new([1, 2, 3, 4]).to_a
           expect(last_response.status).to eq(200)
-          expect(last_response.body).to eq('Fixnum')
+          expect(last_response.body).to eq(integer_class_name)
         end
 
         it 'Set of Bools' do
@@ -326,7 +326,7 @@ class User
 
         get '/int', integers: { int: '45' }
         expect(last_response.status).to eq(200)
-        expect(last_response.body).to eq('Fixnum')
+        expect(last_response.body).to eq(integer_class_name)
       end
     end
 
@@ -525,11 +525,11 @@ class User
 
         get '/', splines: '{"x":"1","y":"woof"}'
         expect(last_response.status).to eq(200)
-        expect(last_response.body).to eq('Fixnum.String')
+        expect(last_response.body).to eq("#{integer_class_name}.String")
 
         get '/', splines: '[{"x":1,"y":2},{"x":1,"y":"quack"}]'
         expect(last_response.status).to eq(200)
-        expect(last_response.body).to eq('Fixnum.Fixnum')
+        expect(last_response.body).to eq("#{integer_class_name}.#{integer_class_name}")
 
         get '/', splines: '{"x":"4","y":"woof"}'
         expect(last_response.status).to eq(400)
@@ -576,7 +576,7 @@ class User
 
         get '/', a: '5'
         expect(last_response.status).to eq(200)
-        expect(last_response.body).to eq('Fixnum')
+        expect(last_response.body).to eq(integer_class_name)
 
         get '/', a: 'anything else'
         expect(last_response.status).to eq(200)

spec/spec_helper.rb

@@ -16,6 +16,7 @@
 
 RSpec.configure do |config|
   config.include Rack::Test::Methods
+  config.include Spec::Support::Helpers
   config.raise_errors_for_deprecations!
 
   config.before(:each) { Grape::Util::InheritableSetting.reset_global! }