diff options
author | Guillermo Iguaran <guilleiguaran@gmail.com> | 2014-02-21 09:50:19 -0500 |
---|---|---|
committer | Guillermo Iguaran <guilleiguaran@gmail.com> | 2014-02-21 09:50:19 -0500 |
commit | b1867c85fa934caad1cd7410e1bd9bc0463a2f42 (patch) | |
tree | 5c325634a2a07d4e86cd3b6858fd29651c8f91a9 /railties/test/application | |
parent | c93c558aa81721653dd18cdc45f3f7299dfa8a72 (diff) | |
download | rails-b1867c85fa934caad1cd7410e1bd9bc0463a2f42.tar.gz rails-b1867c85fa934caad1cd7410e1bd9bc0463a2f42.tar.bz2 rails-b1867c85fa934caad1cd7410e1bd9bc0463a2f42.zip |
Revert "Only lookup `config.log_level` for stdlib `::Logger`. Closes #11665."
This reverts commit e0a521cfcd13e4d1f0ae8ab96004289e1c020f71.
Conflicts:
railties/CHANGELOG.md
We expect loggers to quack like stdlib logger. If log4r needs different
level= assignment, using a Logger-quacking wrapper is the way to do it.
Fixes #14114.
Diffstat (limited to 'railties/test/application')
-rw-r--r-- | railties/test/application/configuration_test.rb | 15 |
1 files changed, 1 insertions, 14 deletions
diff --git a/railties/test/application/configuration_test.rb b/railties/test/application/configuration_test.rb index b2d0e7e202..b39cd3747b 100644 --- a/railties/test/application/configuration_test.rb +++ b/railties/test/application/configuration_test.rb @@ -762,7 +762,7 @@ module ApplicationTests end end - test "lookup config.log_level with custom logger (stdlib Logger)" do + test "config.log_level with custom logger" do make_basic_app do |app| app.config.logger = Logger.new(STDOUT) app.config.log_level = :info @@ -770,19 +770,6 @@ module ApplicationTests assert_equal Logger::INFO, Rails.logger.level end - test "assign log_level as is with custom logger (third party logger)" do - logger_class = Class.new do - attr_accessor :level - end - logger_instance = logger_class.new - make_basic_app do |app| - app.config.logger = logger_instance - app.config.log_level = :info - end - assert_equal logger_instance, Rails.logger - assert_equal :info, Rails.logger.level - end - test "respond_to? accepts include_private" do make_basic_app |