Skip to content

Commit

Permalink
Merge pull request rails#5995 from kennyj/fix_5847-3
Browse files Browse the repository at this point in the history
Fix rails#5847 and rails#4045. Load AR::Base before loading an application model.
  • Loading branch information
jonleighton committed Apr 26, 2012
2 parents 2f0c8c5 + 627f89b commit 8cd14c0
Show file tree
Hide file tree
Showing 2 changed files with 27 additions and 0 deletions.
5 changes: 5 additions & 0 deletions activerecord/lib/active_record/railtie.rb
Expand Up @@ -128,5 +128,10 @@ class Railtie < Rails::Railtie
end

end

config.after_initialize do
# We should load ActiveRecord::Base class before loading an application model.
require "active_record/base"
end
end
end
22 changes: 22 additions & 0 deletions railties/test/application/rake_test.rb
Expand Up @@ -167,5 +167,27 @@ def test_rake_clear_schema_cache
end
assert !File.exists?(File.join(app_path, 'db', 'schema_cache.dump'))
end

def test_load_activerecord_base_when_we_use_observers
Dir.chdir(app_path) do
`bundle exec rails g model user;
bundle exec rake db:migrate;
bundle exec rails g observer user;`

add_to_config "config.active_record.observers = :user_observer"

assert_equal "0", `bundle exec rails r "puts User.count"`.strip

app_file "lib/tasks/count_user.rake", <<-RUBY
namespace :user do
task :count => :environment do
puts User.count
end
end
RUBY

assert_equal "0", `bundle exec rake user:count`.strip
end
end
end
end

0 comments on commit 8cd14c0

Please sign in to comment.