Add ids key in body when saving/removing tracks - Fixes #193

thelinmichael · Apr 29, 2018 · f0b6b52 · f0b6b52
1 parent 803fe10
commit f0b6b52
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 7 deletions.
diff --git a/__tests__/spotify-web-api.js b/__tests__/spotify-web-api.js
@@ -2484,7 +2484,9 @@ describe('Spotify Web API', () => {
       callback
     ) {
       expect(method).toBe(superagent.del);
-      expect(JSON.parse(options.data)).toEqual(['3VNWq8rTnQG6fM1eldSpZ0']);
+      expect(JSON.parse(options.data)).toEqual({
+        ids: ['3VNWq8rTnQG6fM1eldSpZ0']
+      });
       expect(uri).toBe('https://api.spotify.com/v1/me/tracks');
       expect(options.query).toBeFalsy();
       callback();
@@ -3377,7 +3379,9 @@ describe('Spotify Web API', () => {
       callback
     ) {
       expect(method).toBe(superagent.put);
-      expect(JSON.parse(options.data)).toEqual(['3VNWq8rTnQG6fM1eldSpZ0']);
+      expect(JSON.parse(options.data)).toEqual({
+        ids: ['3VNWq8rTnQG6fM1eldSpZ0']
+      });
       expect(uri).toBe('https://api.spotify.com/v1/me/tracks');
       expect(options.query).toBeFalsy();
       expect(options.headers.Authorization).toBe('Bearer myAccessToken');
@@ -3409,7 +3413,9 @@ describe('Spotify Web API', () => {
       callback
     ) {
       expect(method).toBe(superagent.put);
-      expect(JSON.parse(options.data)).toEqual(['3VNWq8rTnQG6fM1eldSpZ0']);
+      expect(JSON.parse(options.data)).toEqual({
+        ids: ['3VNWq8rTnQG6fM1eldSpZ0']
+      });
       expect(uri).toBe('https://api.spotify.com/v1/me/tracks');
       expect(options.query).toBeFalsy();
       expect(options.headers.Authorization).toBe('Bearer myAccessToken');
@@ -3435,7 +3441,9 @@ describe('Spotify Web API', () => {
       callback
     ) {
       expect(method).toBe(superagent.put);
-      expect(JSON.parse(options.data)).toEqual(['3VNWq8rTnQG6fM1eldSpZ0']);
+      expect(JSON.parse(options.data)).toEqual({
+        ids: ['3VNWq8rTnQG6fM1eldSpZ0']
+      });
       expect(uri).toBe('https://api.spotify.com/v1/me/tracks');
       expect(options.query).toBeFalsy();
       expect(options.headers.Authorization).toBe(
@@ -3472,7 +3480,9 @@ describe('Spotify Web API', () => {
       callback
     ) {
       expect(method).toBe(superagent.put);
-      expect(options.data).toEqual(JSON.stringify(['3VNWq8rTnQG6fM1eldSpZ0']));
+      expect(options.data).toEqual(
+        JSON.stringify({ ids: ['3VNWq8rTnQG6fM1eldSpZ0'] })
+      );
       expect(uri).toBe('https://api.spotify.com/v1/me/tracks');
       expect(options.query).toBeFalsy();
       expect(options.headers.Authorization).toBe(

diff --git a/src/spotify-web-api.js b/src/spotify-web-api.js
@@ -974,7 +974,7 @@ SpotifyWebApi.prototype = {
     return WebApiRequest.builder(this.getAccessToken())
       .withPath('/v1/me/tracks')
       .withHeaders({ 'Content-Type': 'application/json' })
-      .withBodyParameters(trackIds)
+      .withBodyParameters({ ids: trackIds })
       .build()
       .execute(HttpManager.del, callback);
   },
@@ -989,7 +989,7 @@ SpotifyWebApi.prototype = {
     return WebApiRequest.builder(this.getAccessToken())
       .withPath('/v1/me/tracks')
       .withHeaders({ 'Content-Type': 'application/json' })
-      .withBodyParameters(trackIds)
+      .withBodyParameters({ ids: trackIds })
       .build()
       .execute(HttpManager.put, callback);
   },