kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

remove unnecessary require and extend

Browse source 
`rack/test` is require at the top of file.
https://github.com/rails/rails/blob/master/railties/test/railties/engine_test.rb#L3

In addtion, `Rack::Test::Methods` is include in class, extend is unnecessary.
https://github.com/rails/rails/blob/master/railties/test/railties/engine_test.rb#L9
This commit is contained in:
yuuji.yaginuma 2016-07-31 16:25:39 +09:00
parent 815b730b1b
commit e3fc35b5d6
1 changed files with 0 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
railties/test/railties/engine_test.rb
View file

@ -36,8 +36,6 @@ module RailtiesTest
add_to_env_config "development", "config.assets.digest = false"
boot_rails
require 'rack/test'
extend Rack::Test::Methods
get "/assets/engine.js"
assert_match "alert()", last_response.body
@ -321,8 +319,6 @@ module RailtiesTest
RUBY
boot_rails
require 'rack/test'
extend Rack::Test::Methods
get "/sprokkit"
assert_equal "I am a Sprokkit", last_response.body
@ -359,8 +355,6 @@ module RailtiesTest
RUBY
boot_rails
require 'rack/test'
extend Rack::Test::Methods
get '/foo'
assert_equal 'foo', last_response.body
@ -449,8 +443,6 @@ YAML
RUBY
boot_rails
require 'rack/test'
extend Rack::Test::Methods
get "/admin/foo/bar"
assert_equal 200, last_response.status