Merge pull request #717 from r-ideas/0-9-stable

fixed issue with rendering Hash which appears in rails 4.2.0.beta4
This commit is contained in:
Alexandre de Oliveira 2015-01-06 09:34:03 -02:00
commit 3baff109eb
3 changed files with 15 additions and 2 deletions

View File

@ -7,9 +7,13 @@ rvm:
- ruby-head
- jruby-19mode
- rbx-2
- ruby-head
sudo: false
install:
- bundle install --retry=3
env:
- "RAILS_VERSION=3.2.17"
- "RAILS_VERSION=4.0.3"

View File

@ -56,7 +56,7 @@ end
attr_reader :key_format
def serializer_for(resource, options = {})
if resource.respond_to?(:each)
if resource.respond_to?(:each) && !resource.is_a?(Hash)
if Object.constants.include?(:ArraySerializer)
::ArraySerializer
else

View File

@ -19,6 +19,10 @@ module ActionController
def render_comments
render json: [Comment.new(content: 'Comment 1')]
end
def render_hash
render json: {message: 'not found'}, status: 404
end
end
tests TestNamespace::MyController
@ -42,6 +46,11 @@ module ActionController
get :render_comments
assert_serializer CommentSerializer
end
def test_render_hash_regression
get :render_hash
assert_equal JSON.parse(response.body), {'message' => 'not found'}
end
end
class OptionNamespacedSerializationTest < ActionController::TestCase
@ -93,4 +102,4 @@ module ActionController
end
end
end
end