aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/template/log_subscriber_test.rb
diff options
context:
space:
mode:
authorPiotr Sarnacki <drogus@gmail.com>2012-06-14 12:04:49 +0200
committerPiotr Sarnacki <drogus@gmail.com>2012-08-28 10:51:04 +0200
commit731bb2fe6815ac6a1888d9fb54bb1782e4768c0b (patch)
tree06f4e63efb1c1a46b60e58d00d6d11310bf18763 /actionpack/test/template/log_subscriber_test.rb
parentba83aa7f03d2742dd242a45229e0f67785871515 (diff)
downloadrails-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/template/log_subscriber_test.rb')
-rw-r--r--actionpack/test/template/log_subscriber_test.rb7
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