From 8022de2d54b90d6cb391aa62dbaac3fc8c9e29f5 Mon Sep 17 00:00:00 2001
From: george <george@benevolentcode.com>
Date: Tue, 30 Apr 2013 14:20:18 -0400
Subject: [PATCH] Grape::ActiveModelSerializers for models with compound names 
  should generate the proper 'root' node for individual objects   should
 generate the proper 'root' node for serialized arrays of objects

---
 .../formatter.rb                              |  4 ++--
 spec/db/schema.rb                             | 10 ++++++++-
 spec/grape_ams_spec.rb                        | 21 +++++++++++++++++++
 spec/spec_fakes.rb                            | 10 ++++++++-
 4 files changed, 41 insertions(+), 4 deletions(-)

diff --git a/lib/grape-active_model_serializers/formatter.rb b/lib/grape-active_model_serializers/formatter.rb
index 5b7ef8a..6704548 100644
--- a/lib/grape-active_model_serializers/formatter.rb
+++ b/lib/grape-active_model_serializers/formatter.rb
@@ -48,7 +48,7 @@ module Grape
 
             if options[:root] != false && serializer.root != false
               # the serializer for an Array is ActiveModel::ArraySerializer
-              options[:root] ||= serializer.root || resource.first.class.name.downcase.pluralize
+              options[:root] ||= serializer.root || resource.first.class.name.underscore.pluralize
             end
           end
 
@@ -61,4 +61,4 @@ module Grape
       end
     end
   end
-end
\ No newline at end of file
+end
diff --git a/spec/db/schema.rb b/spec/db/schema.rb
index 215c183..660fd9a 100644
--- a/spec/db/schema.rb
+++ b/spec/db/schema.rb
@@ -7,4 +7,12 @@ ActiveRecord::Schema.define(version: 20130403105356) do
     t.datetime "created_at"
     t.datetime "updated_at"
   end
-end
\ No newline at end of file
+
+  create_table "blog_posts", force: true do |t|
+    t.string   "title"
+    t.string   "body"
+
+    t.datetime "created_at"
+    t.datetime "updated_at"
+  end
+end
diff --git a/spec/grape_ams_spec.rb b/spec/grape_ams_spec.rb
index 7c03c96..7bc2c64 100644
--- a/spec/grape_ams_spec.rb
+++ b/spec/grape_ams_spec.rb
@@ -53,6 +53,27 @@ describe Grape::ActiveModelSerializers do
     last_response.body.should == "{\"users\":[{\"first_name\":\"JR\",\"last_name\":\"HE\"},{\"first_name\":\"JR\",\"last_name\":\"HE\"}]}"
   end
 
+  context "for models with compound names" do
+    it "should generate the proper 'root' node for individual objects" do
+      subject.get("/home") do
+        BlogPost.new({title: 'Grape AM::S Rocks!', body: 'Really, it does.'})
+      end
+
+      get "/home"
+      last_response.body.should == "{\"blog_post\":{\"title\":\"Grape AM::S Rocks!\",\"body\":\"Really, it does.\"}}"
+    end
+
+    it "should generate the proper 'root' node for serialized arrays of objects" do
+      subject.get("/home") do
+        blog_post = BlogPost.new({title: 'Grape AM::S Rocks!', body: 'Really, it does.'})
+        [blog_post, blog_post]
+      end
+
+      get "/home"
+      last_response.body.should == "{\"blog_posts\":[{\"title\":\"Grape AM::S Rocks!\",\"body\":\"Really, it does.\"},{\"title\":\"Grape AM::S Rocks!\",\"body\":\"Really, it does.\"}]}"
+    end
+  end
+
   # [User2Serializer, 'user2', :user2].each do |serializer|
   #   it "should render using serializer (#{serializer})" do
   #     subject.get("/home", serializer: serializer) do
diff --git a/spec/spec_fakes.rb b/spec/spec_fakes.rb
index 2661bed..1fdc393 100644
--- a/spec/spec_fakes.rb
+++ b/spec/spec_fakes.rb
@@ -6,4 +6,12 @@ end
 
 class UserSerializer < ActiveModel::Serializer
   attributes :first_name, :last_name
-end
\ No newline at end of file
+end
+
+class BlogPost < ActiveRecord::Base
+  attr_accessor :title, :body
+end
+
+class BlogPostSerializer < ActiveModel::Serializer
+  attributes :title, :body
+end