Skip to content
Browse files

Yes we need this to pass the test. we are using config.log_level.

Fix broken test test_copying_migrations(RailtiesTest::EngineTest):
NoMethodError: undefined method `log_level' for #<Rails::Engine::Configuration:0xb87a98c>
  • Loading branch information...
1 parent df948f3 commit a840172c535ae1f8bf29a397185053b85d7b4398 @arunagw arunagw committed with tenderlove Aug 25, 2011
Showing with 2 additions and 2 deletions.
  1. +2 −2 railties/lib/rails/plugin.rb
View
4 railties/lib/rails/plugin.rb
@@ -74,8 +74,8 @@ def config
initializer :load_init_rb, :before => :load_config_initializers do |app|
init_rb = File.expand_path("init.rb", root)
if File.file?(init_rb)
- # FIXME: do we call this for side effects??
- app.config
+ # This double assignment is to prevent an "unused variable" warning on Ruby 1.9.3.
+ config = config = app.config
# TODO: think about evaling initrb in context of Engine (currently it's
# always evaled in context of Rails::Application)
eval(File.read(init_rb), binding, init_rb)

0 comments on commit a840172

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