diff options
author | Piotr Sarnacki <drogus@gmail.com> | 2012-06-14 12:04:49 +0200 |
---|---|---|
committer | Piotr Sarnacki <drogus@gmail.com> | 2012-08-28 10:51:04 +0200 |
commit | 731bb2fe6815ac6a1888d9fb54bb1782e4768c0b (patch) | |
tree | 06f4e63efb1c1a46b60e58d00d6d11310bf18763 /actionpack/test | |
parent | ba83aa7f03d2742dd242a45229e0f67785871515 (diff) | |
download | rails-731bb2fe6815ac6a1888d9fb54bb1782e4768c0b.tar.gz rails-731bb2fe6815ac6a1888d9fb54bb1782e4768c0b.tar.bz2 rails-731bb2fe6815ac6a1888d9fb54bb1782e4768c0b.zip |
Remove stubs from LogSubscriber tests
We can use another way to instantiate ActionView::Base, by passing
renderer as a first option. Thanks to that we can just pass prefixes to
LookupContext instead stubbing them on the controller. This is also good,
because that kind of API is used in Rails code.
Diffstat (limited to 'actionpack/test')
-rw-r--r-- | actionpack/test/template/log_subscriber_test.rb | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/actionpack/test/template/log_subscriber_test.rb b/actionpack/test/template/log_subscriber_test.rb index 1713ce935c..7f4c84929f 100644 --- a/actionpack/test/template/log_subscriber_test.rb +++ b/actionpack/test/template/log_subscriber_test.rb @@ -8,9 +8,10 @@ class AVLogSubscriberTest < ActiveSupport::TestCase def setup super - @controller = Object.new - @controller.stubs(:_prefixes).returns(%w(test)) - @view = ActionView::Base.new(ActionController::Base.view_paths, {}, @controller) + view_paths = ActionController::Base.view_paths + lookup_context = ActionView::LookupContext.new(view_paths, {}, ["test"]) + renderer = ActionView::Renderer.new(lookup_context) + @view = ActionView::Base.new(renderer, {}) Rails.stubs(:root).returns(File.expand_path(FIXTURE_LOAD_PATH)) ActionView::LogSubscriber.attach_to :action_view end |