diff options
| author | José Valim <jose.valim@gmail.com> | 2011-06-30 11:47:14 -0300 |
|---|---|---|
| committer | José Valim <jose.valim@gmail.com> | 2011-06-30 11:47:36 -0300 |
| commit | 9c9ec2172e46710d1d5ac513a4ff52df35fef51f (patch) | |
| tree | 39c761c4bfcda6f87d79d2c6bb10beeac3623cea /actionpack/test | |
| parent | 35cdd256c00dd08b54c22d5b7f38dbcc897d876b (diff) | |
| download | rails-9c9ec2172e46710d1d5ac513a4ff52df35fef51f.tar.gz rails-9c9ec2172e46710d1d5ac513a4ff52df35fef51f.tar.bz2 rails-9c9ec2172e46710d1d5ac513a4ff52df35fef51f.zip | |
Make sure respond_with with :js tries to render a template in all cases
Diffstat (limited to 'actionpack/test')
| -rw-r--r-- | actionpack/test/controller/mime_responds_test.rb | 21 | ||||
| -rw-r--r-- | actionpack/test/fixtures/respond_with/using_resource.js.erb | 1 |
2 files changed, 16 insertions, 6 deletions
diff --git a/actionpack/test/controller/mime_responds_test.rb b/actionpack/test/controller/mime_responds_test.rb index 26270571cf..288682aa07 100644 --- a/actionpack/test/controller/mime_responds_test.rb +++ b/actionpack/test/controller/mime_responds_test.rb @@ -509,7 +509,7 @@ end class RespondWithController < ActionController::Base respond_to :html, :json respond_to :xml, :except => :using_resource_with_block - respond_to :js, :only => [ :using_resource_with_block, :using_resource ] + respond_to :js, :only => [ :using_resource_with_block, :using_resource, :using_hash_resource ] def using_resource respond_with(resource) @@ -575,11 +575,6 @@ protected def resource Customer.new("david", request.delete? ? nil : 13) end - - def _render_js(js, options) - self.content_type ||= Mime::JS - self.response_body = js.respond_to?(:to_js) ? js.to_js : js - end end class InheritedRespondWithController < RespondWithController @@ -638,6 +633,20 @@ class RespondWithControllerTest < ActionController::TestCase end end + def test_using_resource_with_js_simply_tries_to_render_the_template + @request.accept = "text/javascript" + get :using_resource + assert_equal "text/javascript", @response.content_type + assert_equal "alert(\"Hi\");", @response.body + end + + def test_using_hash_resource_with_js_raises_an_error_if_template_cant_be_found + @request.accept = "text/javascript" + assert_raise ActionView::MissingTemplate do + get :using_hash_resource + end + end + def test_using_hash_resource @request.accept = "application/xml" get :using_hash_resource diff --git a/actionpack/test/fixtures/respond_with/using_resource.js.erb b/actionpack/test/fixtures/respond_with/using_resource.js.erb new file mode 100644 index 0000000000..4417680bce --- /dev/null +++ b/actionpack/test/fixtures/respond_with/using_resource.js.erb @@ -0,0 +1 @@ +alert("Hi");
\ No newline at end of file |