From 32b8be95331928990a61a43023a343a0583b53c7 Mon Sep 17 00:00:00 2001
From: David Chelimsky <dchelimsky@gmail.com>
Date: Wed, 23 Jun 2010 10:19:13 -0500
Subject: Expose view via the view() method in AV::TestCase::Behavior
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- was exposed as _view, which suggested it was private
- left _view as an alias of view as not to break any extensions that are
  relying on _view

[#4932 state:resolved]

Signed-off-by: José Valim <jose.valim@gmail.com>
---
 actionpack/test/template/test_case_test.rb | 28 ++++++++++++++++------------
 1 file changed, 16 insertions(+), 12 deletions(-)

(limited to 'actionpack/test')

diff --git a/actionpack/test/template/test_case_test.rb b/actionpack/test/template/test_case_test.rb
index 4773eae039..a747f46527 100644
--- a/actionpack/test/template/test_case_test.rb
+++ b/actionpack/test/template/test_case_test.rb
@@ -37,8 +37,12 @@ module ActionView
     include SharedTests
     test_case = self
 
-    test "memoizes the _view" do
-      assert_same _view, _view
+    test "memoizes the view" do
+      assert_same view, view
+    end
+
+    test "exposes view as _view for backwards compatibility" do
+      assert_same _view, view
     end
 
     test "works without testing a helper module" do
@@ -61,13 +65,13 @@ module ActionView
     end
 
     test "delegates notice to request.flash" do
-      _view.request.flash.expects(:notice).with("this message")
-      _view.notice("this message")
+      view.request.flash.expects(:notice).with("this message")
+      view.notice("this message")
     end
 
     test "delegates alert to request.flash" do
-      _view.request.flash.expects(:alert).with("this message")
-      _view.alert("this message")
+      view.request.flash.expects(:alert).with("this message")
+      view.alert("this message")
     end
   end
 
@@ -136,7 +140,7 @@ module ActionView
     helper HelperThatInvokesProtectAgainstForgery
 
     test "protect_from_forgery? in any helpers returns false" do
-      assert !_view.help_me
+      assert !view.help_me
     end
 
   end
@@ -200,10 +204,10 @@ module ActionView
       assert_match /Hello: EloyHello: Manfred/, render(:file => 'test/list')
     end
 
-    test "is able to render partials from templates and also use instance variables after _view has been referenced" do
+    test "is able to render partials from templates and also use instance variables after view has been referenced" do
       @controller.controller_path = "test"
 
-      _view
+      view
 
       @customers = [stub(:name => 'Eloy'), stub(:name => 'Manfred')]
       assert_match /Hello: EloyHello: Manfred/, render(:file => 'test/list')
@@ -229,19 +233,19 @@ module ActionView
   end
 
   class RenderTemplateTest < ActionView::TestCase
-    test "render template supports specifying partials" do
+    test "supports specifying partials" do
       controller.controller_path = "test"
       render(:template => "test/calling_partial_with_layout")
       assert_template :partial => "_partial_for_use_in_layout"
     end
 
-    test "render template supports specifying locals (passing)" do
+    test "supports specifying locals (passing)" do
       controller.controller_path = "test"
       render(:template => "test/calling_partial_with_layout")
       assert_template :partial => "_partial_for_use_in_layout", :locals => { :name => "David" }
     end
 
-    test "render template supports specifying locals (failing)" do
+    test "supports specifying locals (failing)" do
       controller.controller_path = "test"
       render(:template => "test/calling_partial_with_layout")
       assert_raise Test::Unit::AssertionFailedError, /Somebody else.*David/m do
-- 
cgit v1.2.3