Add missing requires (#2365)

ericproulx · web-flow · commit 8de048ed5681 · 2023-10-30T14:01:42.000-04:00
* Add require 'active_support/core_ext/enumerable'

* Add changelog

* Replace first, second by [0], [1]
require missing core_ext/object/deep_dup &amp;&amp; core_ext/string/exclude

* grape_entity require false

* Update CHANGELOG.md
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,6 +10,7 @@
   
 #### Fixes
 
+* [#2364](https://github.com/ruby-grape/grape/pull/2364): Add missing requires - [@ericproulx](https://github.com/ericproulx).
 * Your contribution here.
 
 ### 1.8.0 (2023/08/30)
diff --git a/Gemfile b/Gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/multi_json.gemfile b/gemfiles/multi_json.gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/multi_xml.gemfile b/gemfiles/multi_xml.gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/rack_1_0.gemfile b/gemfiles/rack_1_0.gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/rack_2_0.gemfile b/gemfiles/rack_2_0.gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/rack_3_0.gemfile b/gemfiles/rack_3_0.gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/rack_edge.gemfile b/gemfiles/rack_edge.gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/rails_5_2.gemfile b/gemfiles/rails_5_2.gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/rails_6_0.gemfile b/gemfiles/rails_6_0.gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/rails_6_1.gemfile b/gemfiles/rails_6_1.gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/rails_7_0.gemfile b/gemfiles/rails_7_0.gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/rails_7_1.gemfile b/gemfiles/rails_7_1.gemfile
@@ -27,7 +27,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/gemfiles/rails_edge.gemfile b/gemfiles/rails_edge.gemfile
@@ -26,7 +26,7 @@ end
 
 group :test do
   gem 'cookiejar'
-  gem 'grape-entity', '~> 0.6'
+  gem 'grape-entity', '~> 0.6', require: false
   gem 'mime-types'
   gem 'rack-jsonp', require: 'rack/jsonp'
   gem 'rack-test', '< 2.1'
diff --git a/lib/grape.rb b/lib/grape.rb
@@ -16,6 +16,7 @@
 require 'active_support/core_ext/array/conversions'
 require 'active_support/core_ext/array/extract_options'
 require 'active_support/core_ext/array/wrap'
+require 'active_support/core_ext/enumerable'
 require 'active_support/core_ext/hash/conversions'
 require 'active_support/core_ext/hash/deep_merge'
 require 'active_support/core_ext/hash/except'
@@ -24,7 +25,9 @@
 require 'active_support/core_ext/hash/reverse_merge'
 require 'active_support/core_ext/hash/slice'
 require 'active_support/core_ext/object/blank'
+require 'active_support/core_ext/object/deep_dup'
 require 'active_support/core_ext/object/duplicable'
+require 'active_support/core_ext/string/exclude'
 require 'active_support/dependencies/autoload'
 require 'active_support/deprecation'
 require 'active_support/inflector'
diff --git a/spec/grape/endpoint_spec.rb b/spec/grape/endpoint_spec.rb
@@ -201,8 +201,8 @@ def app
       get('/username', {}, 'HTTP_COOKIE' => 'username=user; sandbox=false')
       expect(last_response.body).to eq('user_test')
       cookies = Array(last_response.headers['Set-Cookie']).flat_map { |h| h.split("\n") }
-      expect(cookies.first).to match(/username=user_test/)
-      expect(cookies.second).to match(/sandbox=true/)
+      expect(cookies[0]).to match(/username=user_test/)
+      expect(cookies[1]).to match(/sandbox=true/)
     end
 
     it 'deletes cookie' do
