Merge pull request #978 from u2/rubocop0.29.1

Upgraded RuboCop to 0.29.1

Author: dm1try

Gemfile

@@ -3,7 +3,7 @@ source 'https://rubygems.org'
 gemspec
 
 group :development, :test do
-  gem 'rubocop', '~> 0.28.0'
+  gem 'rubocop', '~> 0.29.1'
   gem 'guard'
   gem 'guard-rspec'
   gem 'guard-rubocop'

lib/grape/dsl/configuration.rb

@@ -48,13 +48,13 @@ def stacked_hash_to_hash(settings)
       def desc_container
         Module.new do
           include Grape::Util::StrictHashConfiguration.module(
-                      :description,
-                      :detail,
-                      :params,
-                      :entity,
-                      :http_codes,
-                      :named,
-                      :headers
+            :description,
+            :detail,
+            :params,
+            :entity,
+            :http_codes,
+            :named,
+            :headers
           )
 
           def config_context.success(*args)

lib/grape/dsl/routing.rb

@@ -89,11 +89,11 @@ def mount(mounts)
             end
 
             endpoints << Grape::Endpoint.new(
-                in_setting,
-                method: :any,
-                path: path,
-                app: app,
-                for: self
+              in_setting,
+              method: :any,
+              path: path,
+              app: app,
+              for: self
             )
           end
         end

lib/grape/endpoint.rb

@@ -151,14 +151,14 @@ def prepare_routes
           request_method = (method.to_s.upcase unless method == :any)
 
           Route.new(options[:route_options].clone.merge(
-            prefix: namespace_inheritable(:root_prefix),
-            version: namespace_inheritable(:version) ? namespace_inheritable(:version).join('|') : nil,
-            namespace: namespace,
-            method: request_method,
-            path: prepared_path,
-            params: prepare_routes_path_params(path),
-            compiled: path,
-            settings: inheritable_setting.route.except(:saved_declared_params, :saved_validations)
+                      prefix: namespace_inheritable(:root_prefix),
+                      version: namespace_inheritable(:version) ? namespace_inheritable(:version).join('|') : nil,
+                      namespace: namespace,
+                      method: request_method,
+                      path: prepared_path,
+                      params: prepare_routes_path_params(path),
+                      compiled: path,
+                      settings: inheritable_setting.route.except(:saved_declared_params, :saved_validations)
           ))
         end
       end.flatten

spec/grape/api_spec.rb

@@ -419,7 +419,7 @@ def subject.enable_root_route!
             send verb, '/', MultiJson.dump(object), 'CONTENT_TYPE' => 'application/json'
             expect(last_response.status).to eq(verb == :post ? 201 : 200)
             expect(last_response.body).to eql MultiJson.dump(object)
-            expect(last_request.params).to eql Hash.new
+            expect(last_request.params).to eql({})
           end
           it 'stores input in api.request.input' do
             subject.format :json

spec/grape/endpoint_spec.rb

@@ -116,8 +116,8 @@ def app
     it 'includes request headers' do
       get '/headers'
       expect(JSON.parse(last_response.body)).to eq(
-          'Host' => 'example.org',
-          'Cookie' => ''
+        'Host' => 'example.org',
+        'Cookie' => ''
       )
     end
     it 'includes additional request headers' do

spec/grape/entity_spec.rb

@@ -114,7 +114,7 @@ def first
 
     it 'does not run autodetection for Entity when explicitely provided' do
       entity = Class.new(Grape::Entity)
-      some_array = Array.new
+      some_array = []
 
       subject.get '/example' do
         present some_array, with: entity