Added that render :xml will try to call to_xml if it can [DHH]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@6574 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

2 files changed, 25 insertions, 10 deletions

diff --git a/actionpack/test/controller/new_render_test.rb b/actionpack/test/controller/new_render_test.rb
index 0765772e06..18a32f6e30 100644
--- a/actionpack/test/controller/new_render_test.rb
+++ b/actionpack/test/controller/new_render_test.rb
@@ -258,6 +258,20 @@ class NewRenderTestController < ActionController::Base
     head :forbidden, :x_custom_header => "something"
   end
 
+  def render_with_location
+    render :xml => "<hello/>", :location => "http://example.com", :status => 201
+  end
+
+  def render_with_to_xml
+    to_xmlable = Class.new do
+      def to_xml
+        "<i-am-xml/>"
+      end
+    end.new
+    
+    render :xml => to_xmlable
+  end
+
   helper NewRenderTestHelper
   helper do 
     def rjs_helper_method(value)
@@ -742,4 +756,14 @@ EOS
     assert_equal "something", @response.headers["X-Custom-Header"]
     assert_response :forbidden
   end
-end
+
+  def test_rendering_with_location_should_set_header
+    get :render_with_location
+    assert_equal "http://example.com", @response.headers["Location"]
+  end
+  
+  def test_rendering_xml_should_call_to_xml_if_possible
+    get :render_with_to_xml
+    assert_equal "<i-am-xml/>", @response.body
+  end
+end
\ No newline at end of file
diff --git a/actionpack/test/controller/render_test.rb b/actionpack/test/controller/render_test.rb
index eb23ed0c5b..d51af51454 100644
--- a/actionpack/test/controller/render_test.rb
+++ b/actionpack/test/controller/render_test.rb
@@ -73,10 +73,6 @@ class TestController < ActionController::Base
     head :ok
   end
 
-  def location
-    render :xml => "<hello/>", :location => "http://example.com", :status => 201
-  end
-
   def greeting
     # let's just rely on the template
   end
@@ -372,11 +368,6 @@ class RenderTest < Test::Unit::TestCase
     assert_equal '<test>passed formatted html erb</test>', @response.body
   end
 
-  def test_rendering_with_location_should_set_header
-    get :location
-    assert_equal "http://example.com", @response.headers["Location"]
-  end
-
 
   protected
     def assert_deprecated_render(&block)