diff options
author | yuuji.yaginuma <yuuji.yaginuma@gmail.com> | 2017-10-07 11:52:22 +0900 |
---|---|---|
committer | yuuji.yaginuma <yuuji.yaginuma@gmail.com> | 2017-10-07 11:56:43 +0900 |
commit | d342a35eaa59717155da0a646f9a44435f15c052 (patch) | |
tree | 55250f1fc319b690b88ef8ba78ac2bfcc78c8eb6 /railties/test/rails_info_test.rb | |
parent | 8c9a93090bff5f7350f2b22de5d37d863feffa53 (diff) | |
download | rails-d342a35eaa59717155da0a646f9a44435f15c052.tar.gz rails-d342a35eaa59717155da0a646f9a44435f15c052.tar.bz2 rails-d342a35eaa59717155da0a646f9a44435f15c052.zip |
Remove needless setup from `InfoTest`
The `abstract_unit` loads `rails/all`. Therefore, `Rails` is definitely defined.
Also, since `Info` is autoloaded, do not need to explicitly load the file.
https://github.com/rails/rails/blob/3ede539357acc91e377611cea9dd5f30678e7b2a/railties/lib/rails.rb#L29
Diffstat (limited to 'railties/test/rails_info_test.rb')
-rw-r--r-- | railties/test/rails_info_test.rb | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/railties/test/rails_info_test.rb b/railties/test/rails_info_test.rb index 227a739b71..43b60b9144 100644 --- a/railties/test/rails_info_test.rb +++ b/railties/test/rails_info_test.rb @@ -2,20 +2,7 @@ require "abstract_unit" -unless defined?(Rails) && defined?(Rails::Info) - module Rails - class Info; end - end -end - -require "active_support/core_ext/kernel/reporting" - class InfoTest < ActiveSupport::TestCase - def setup - Rails.send :remove_const, :Info - silence_warnings { load "rails/info.rb" } - end - def test_property_with_block_swallows_exceptions_and_ignores_property assert_nothing_raised do Rails::Info.module_eval do |