Merge branch 'api-json-root_revert'

Revert default rails behavior for JSON root in API responses, and tune
index views to play nicely with Ember.js DS.RESTAdapter defaults.

app/views/api/playlists/index.rabl

@@ -1,4 +1,3 @@
-collection @playlists
+collection @playlists, root: :playlists, object_root: false
 
-attribute :id
+extends 'api/playlists/show'
-attribute :name

app/views/api/tracks/index.rabl

@@ -1,6 +1,5 @@
-collection @tracks
+collection @tracks, root: :tracks, object_root: false
 
-attribute :id
+extends 'api/tracks/show'
-attribute :name
 
 node(:sound_url, if: ->(t) { t.sound? }) { |t| api_sound_url t.sound }

config/initializers/rabl_init.rb

@@ -1,16 +0,0 @@
-Rabl.configure do |config|
-  # Commented as these are defaults
-  # config.cache_sources = false
-  # config.json_engine = nil # Any multi\_json engines
-  # config.msgpack_engine = nil # Defaults to ::MessagePack
-  # config.bson_engine = nil # Defaults to ::BSON
-  # config.plist_engine = nil # Defaults to ::Plist::Emit
-  # config.include_json_root = true
-  config.include_json_root = false
-  # config.include_msgpack_root = true
-  # config.include_bson_root = true
-  # config.include_plist_root = true
-  # config.include_xml_root  = false
-  # config.enable_json_callbacks = false
-  # config.xml_options = { dasherize: true, skip_types: false }
-end

config/initializers/wrap_parameters.rb

@@ -9,6 +9,6 @@ ActiveSupport.on_load(:action_controller) do
 end
 
 # To enable root element in JSON for ActiveRecord objects.
-ActiveSupport.on_load(:active_record) do
+# ActiveSupport.on_load(:active_record) do
-  self.include_root_in_json = true
+#  self.include_root_in_json = true
-end
+# end

spec/integration/api/api_sign_in_spec.rb

@@ -13,7 +13,7 @@ describe 'API sign in' do
   it 'signs the user in with valid credentials' do
     do_create
 
-    expect(json).to eq(id: user.id)
+    expect(json).to eq(user: { id: user.id })
   end
 
   [:email, :password].each do |attr|

spec/integration/api/playlists_spec.rb

@@ -7,33 +7,35 @@ describe 'API playlists' do
     playlist = create :playlist
     get api_playlists_path, format: :json
 
-    expect(json).to match [
+    expect(json).to eq(
-      a_hash_including(
+      playlists: [{
-        id:   an_instance_of(Fixnum),
+        id:   playlist.id,
-        name: playlist[:name]
+        name: playlist.name
+      }]
     )
-    ]
   end
 
   it 'shows a playlist' do
     playlist = create :playlist
     get api_playlist_path playlist, format: :json
 
-    expect(json).to match(
+    expect(json).to eq(
+      playlist: {
         id:   playlist.id,
         name: playlist.name
+      }
     )
   end
 
   it 'creates playlist' do
     playlist = attributes_for :playlist
-    post api_playlists_path,
+    post api_playlists_path, format: :json, playlist: playlist
-      format: :json,
-      playlist: playlist
 
     expect(json).to match(
+      playlist: {
         id:   an_instance_of(Fixnum),
         name: playlist[:name]
+      }
     )
   end
 end

spec/integration/api/tracks_spec.rb

@@ -8,7 +8,8 @@ describe 'API tracks' do
     track_2 = create :track, name: 'Track 2'
     get api_tracks_path, format: :json
 
-    expect(json).to eq [
+    expect(json).to eq(
+      tracks: [
         {
           id:         track_1.id,
           name:       'Track 1',
@@ -19,15 +20,18 @@ describe 'API tracks' do
           name:       'Track 2'
         }
       ]
+    )
   end
 
   it 'shows a track' do
     track = create :track
     get api_track_path track, format: :json
 
-    expect(json).to match(
+    expect(json).to eq(
+      track: {
         id:   track.id,
         name: track.name
+      }
     )
   end
 end