Skip to content
Browse files

Do not redirect cache logger to /dev/null in test

For some reason, redirecting cache's logger to '/dev/null' resulting in
a test failures and LoadError. I think it's because of Thread issue.
Instead of trying to make every logger threadsafe for Rails 3.2, I think
it's better to just don't set the logger for now.

(Note: resetting the logger back to original value in the teardown block
didn't fix the problem.)
  • Loading branch information...
1 parent 5693d44 commit 8598633cc1d5ea402f8b49b1f71cd92180cc1138 @sikachu sikachu committed Feb 22, 2013
Showing with 0 additions and 1 deletion.
  1. +0 −1 activesupport/test/caching_test.rb
View
1 activesupport/test/caching_test.rb
@@ -690,7 +690,6 @@ def setup
@data = @cache.instance_variable_get(:@data)
@cache.clear
@cache.silence!
- @cache.logger = Logger.new("/dev/null")
end
include CacheStoreBehavior

0 comments on commit 8598633

Please sign in to comment.
Something went wrong with that request. Please try again.