Fixed a bug where request parameters were not added, related to #461

Author: JohnnyCrazy

SpotifyAPI.Web.Tests/Clients/FollowClientTest.cs

@@ -0,0 +1,27 @@
+using System.Collections.Generic;
+using System.Threading.Tasks;
+using Moq;
+using NUnit.Framework;
+using SpotifyAPI.Web.Http;
+
+namespace SpotifyAPI.Web.Tests
+{
+  [TestFixture]
+  public class FollowClientTest
+  {
+    [Test]
+    public async Task OfCurrentUser_PassesQueryParams()
+    {
+      var api = new Mock<IAPIConnector>();
+      var client = new FollowClient(api.Object);
+
+      var request = new FollowOfCurrentUserRequest(FollowOfCurrentUserRequest.Type.Artist);
+      await client.OfCurrentUser(request);
+
+      api.Verify(a => a.Get<FollowedArtistsResponse>(
+        SpotifyUrls.CurrentUserFollower(),
+        It.Is<Dictionary<string, string>>(val => val.ContainsKey("type"))
+      ));
+    }
+  }
+}

SpotifyAPI.Web/Clients/FollowClient.cs

@@ -60,14 +60,14 @@ public Task<FollowedArtistsResponse> OfCurrentUser()
     {
       var request = new FollowOfCurrentUserRequest();
 
-      return API.Get<FollowedArtistsResponse>(URLs.CurrentUserFollower(), request.BuildQueryParams());
+      return OfCurrentUser(request);
     }
 
     public Task<FollowedArtistsResponse> OfCurrentUser(FollowOfCurrentUserRequest request)
     {
       Ensure.ArgumentNotNull(request, nameof(request));
 
-      return API.Get<FollowedArtistsResponse>(URLs.CurrentUserFollower());
+      return API.Get<FollowedArtistsResponse>(URLs.CurrentUserFollower(), request.BuildQueryParams());
     }
 
     public async Task<bool> Unfollow(UnfollowRequest request)

SpotifyAPI.Web/Clients/PlaylistsClient.cs

@@ -28,10 +28,9 @@ public Task<SnapshotResponse> AddItems(string playlistId, PlaylistAddItemsReques
 
     public Task<Paging<PlaylistTrack<IPlayableItem>>> GetItems(string playlistId)
     {
-      Ensure.ArgumentNotNullOrEmptyString(playlistId, nameof(playlistId));
       var request = new PlaylistGetItemsRequest();
 
-      return API.Get<Paging<PlaylistTrack<IPlayableItem>>>(URLs.PlaylistTracks(playlistId), request.BuildQueryParams());
+      return GetItems(playlistId, request);
     }
 
     public Task<Paging<PlaylistTrack<IPlayableItem>>> GetItems(string playlistId, PlaylistGetItemsRequest request)
@@ -83,10 +82,9 @@ public Task<Paging<SimplePlaylist>> GetUsers(string userId, PlaylistGetUsersRequ
 
     public Task<FullPlaylist> Get(string playlistId)
     {
-      Ensure.ArgumentNotNullOrEmptyString(playlistId, nameof(playlistId));
       var request = new PlaylistGetRequest(); // We need some defaults
 
-      return API.Get<FullPlaylist>(URLs.Playlist(playlistId), request.BuildQueryParams());
+      return Get(playlistId, request);
     }
 
     public Task<FullPlaylist> Get(string playlistId, PlaylistGetRequest request)