Merge pull request #360 from blackcandy-org/favorite-api-response

Return full song data in response of favorite playlist API
blackcandy-org · Mar 11, 2024 · 0627416 · 0627416
2 parents 6cb81d9 + 9ffc9b1
commit 0627416
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/app/views/api/v1/favorite_playlist/songs/create.json.jbuilder b/app/views/api/v1/favorite_playlist/songs/create.json.jbuilder
@@ -1 +1 @@
-json.call(@song, :id, :name)
+json.partial! "api/v1/songs/song", song: @song
diff --git a/app/views/api/v1/favorite_playlist/songs/destroy.json.jbuilder b/app/views/api/v1/favorite_playlist/songs/destroy.json.jbuilder
@@ -1 +1 @@
-json.call(@song, :id, :name)
+json.partial! "api/v1/songs/song", song: @song
diff --git a/test/controllers/api/v1/favorite_playlist/songs_controller_test.rb b/test/controllers/api/v1/favorite_playlist/songs_controller_test.rb
@@ -13,6 +13,7 @@ class Api::V1::FavoritePlaylist::SongsControllerTest < ActionDispatch::Integrati
 
     assert_response :success
     assert_equal 3, response["id"]
+    assert response["is_favorited"]
     assert_equal [3, 1, 2], @playlist.reload.song_ids
   end
 
@@ -22,6 +23,7 @@ class Api::V1::FavoritePlaylist::SongsControllerTest < ActionDispatch::Integrati
 
     assert_response :success
     assert_equal 1, response["id"]
+    assert_not response["is_favorited"]
     assert_equal [2], @playlist.reload.song_ids
 
     delete api_v1_favorite_playlist_song_url(id: 2), as: :json, headers: api_token_header(@user)