From 6899cf86eeb043bff790d1420536724b4bd7f09a Mon Sep 17 00:00:00 2001
From: Prashant Khandelwal <prashant@midnightprogrammer.net>
Date: Thu, 4 Dec 2014 20:10:07 +0530
Subject: [PATCH 1/2] Added album.getTopTags API method to get top tags

---
 src/IF.Lastfm.Core/Api/AlbumApi.cs            | 10 ++-
 .../Commands/AlbumApi/AlbumTopTagsCommand.cs  | 75 +++++++++++++++++++
 src/IF.Lastfm.Core/IF.Lastfm.Core.csproj      |  1 +
 3 files changed, 84 insertions(+), 2 deletions(-)
 create mode 100644 src/IF.Lastfm.Core/Api/Commands/AlbumApi/AlbumTopTagsCommand.cs

diff --git a/src/IF.Lastfm.Core/Api/AlbumApi.cs b/src/IF.Lastfm.Core/Api/AlbumApi.cs
index d508a1f..9a2a1af 100644
--- a/src/IF.Lastfm.Core/Api/AlbumApi.cs
+++ b/src/IF.Lastfm.Core/Api/AlbumApi.cs
@@ -46,9 +46,15 @@ public Task<PageResponse<LastTag>> GetUserTagsForAlbumAsync(string artist, strin
             throw new NotImplementedException();
         }
 
-        public Task<PageResponse<LastTag>> GetTopTagsForAlbumAsync(string artist, string album, bool autocorrect = false)
+        public async Task<PageResponse<LastTag>> GetTopTagsForAlbumAsync(string artist, string album, bool autocorrect = false)
         {
-            throw new NotImplementedException();
+            var command = new AlbumTopTagsCommand(Auth)
+            {
+                ArtistName = artist,
+                AlbumName = album
+            };
+
+            return await command.ExecuteAsync();
         }
 
         public async Task<PageResponse<LastAlbum>> SearchForAlbumAsync(string albumname, int page = 1, int itemsPerPage = LastFm.DefaultPageLength)
diff --git a/src/IF.Lastfm.Core/Api/Commands/AlbumApi/AlbumTopTagsCommand.cs b/src/IF.Lastfm.Core/Api/Commands/AlbumApi/AlbumTopTagsCommand.cs
new file mode 100644
index 0000000..69226b9
--- /dev/null
+++ b/src/IF.Lastfm.Core/Api/Commands/AlbumApi/AlbumTopTagsCommand.cs
@@ -0,0 +1,75 @@
+using IF.Lastfm.Core.Api.Enums;
+using IF.Lastfm.Core.Api.Helpers;
+using IF.Lastfm.Core.Objects;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Linq;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Net.Http;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace IF.Lastfm.Core.Api.Commands.AlbumApi
+{
+    internal class AlbumTopTagsCommand : GetAsyncCommandBase<PageResponse<LastTag>>
+    {
+        public string AlbumMbid { get; set; }
+
+        public string ArtistName { get; set; }
+
+        public string AlbumName { get; set; }
+
+        public bool Autocorrect { get; set; }
+
+        public AlbumTopTagsCommand(ILastAuth auth)
+            : base(auth)
+        {
+            Method = "album.gettoptags";
+        }
+
+        public AlbumTopTagsCommand(ILastAuth auth, string album, string artist)
+            : this(auth)
+        {
+            AlbumName = album;
+            ArtistName = artist;
+        }
+
+        public override void SetParameters()
+        {
+            if (AlbumMbid != null)
+            {
+                Parameters.Add("mbid", AlbumMbid);
+            }
+            else
+            {
+                Parameters.Add("artist", ArtistName);
+                Parameters.Add("album", AlbumName);
+            }
+
+            Parameters.Add("autocorrect", Convert.ToInt32(Autocorrect).ToString());
+
+            AddPagingParameters();
+            DisableCaching();
+        }
+
+        public async override Task<PageResponse<LastTag>> HandleResponse(HttpResponseMessage response)
+        {
+            var json = await response.Content.ReadAsStringAsync();
+
+            LastFmApiError error;
+            if (LastFm.IsResponseValid(json, out error) && response.IsSuccessStatusCode)
+            {
+                var jtoken = JsonConvert.DeserializeObject<JToken>(json);
+                var resultsToken = jtoken.SelectToken("toptags");
+                var itemsToken = resultsToken.SelectToken("tag");
+
+                return PageResponse<LastTag>.CreateSuccessResponse(itemsToken, resultsToken, LastTag.ParseJToken, false);
+            }
+            else
+            {
+                return LastResponse.CreateErrorResponse<PageResponse<LastTag>>(error);
+            }
+        }
+    }
+}
diff --git a/src/IF.Lastfm.Core/IF.Lastfm.Core.csproj b/src/IF.Lastfm.Core/IF.Lastfm.Core.csproj
index 6de4b91..553c2e6 100644
--- a/src/IF.Lastfm.Core/IF.Lastfm.Core.csproj
+++ b/src/IF.Lastfm.Core/IF.Lastfm.Core.csproj
@@ -40,6 +40,7 @@
   <ItemGroup>
     <Compile Include="Api\AlbumApi.cs" />
     <Compile Include="Api\ArtistApi.cs" />
+    <Compile Include="Api\Commands\AlbumApi\AlbumTopTagsCommand.cs" />
     <Compile Include="Api\Commands\UserApi\GetRecommendedArtistsCommand.cs" />
     <Compile Include="Api\LastAuth.cs" />
     <Compile Include="Api\ChartApi.cs" />

From 68470c84d8458b91260e5ef6bf04a50f3d58ccc0 Mon Sep 17 00:00:00 2001
From: Prashant Khandelwal <prashant@midnightprogrammer.net>
Date: Thu, 4 Dec 2014 20:58:38 +0530
Subject: [PATCH 2/2] Test cases for GetAlbumTopTagsCommand

---
 .../AlbumApi/GetAlbumTopTagsCommandTest.cs    | 80 +++++++++++++++++++
 .../IF.Lastfm.Core.Tests.csproj               | 19 ++++-
 .../Resources/AlbumApi/AlbumGetTopTags.json   |  1 +
 .../AlbumApi/AlbumGetTopTagsEmpty.json        |  8 ++
 .../AlbumApi/AlbumGetTopTagsError.json        |  1 +
 .../Resources/AlbumApiResponses.Designer.cs   | 32 +++++++-
 .../Resources/AlbumApiResponses.resx          |  9 +++
 src/IF.Lastfm.Core/Api/AlbumApi.cs            |  2 +-
 ...gsCommand.cs => GetAlbumTopTagsCommand.cs} |  6 +-
 src/IF.Lastfm.Core/IF.Lastfm.Core.csproj      |  2 +-
 10 files changed, 152 insertions(+), 8 deletions(-)
 create mode 100644 src/IF.Lastfm.Core.Tests/Api/Commands/AlbumApi/GetAlbumTopTagsCommandTest.cs
 create mode 100644 src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTags.json
 create mode 100644 src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTagsEmpty.json
 create mode 100644 src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTagsError.json
 rename src/IF.Lastfm.Core/Api/Commands/AlbumApi/{AlbumTopTagsCommand.cs => GetAlbumTopTagsCommand.cs} (90%)

diff --git a/src/IF.Lastfm.Core.Tests/Api/Commands/AlbumApi/GetAlbumTopTagsCommandTest.cs b/src/IF.Lastfm.Core.Tests/Api/Commands/AlbumApi/GetAlbumTopTagsCommandTest.cs
new file mode 100644
index 0000000..9fd4498
--- /dev/null
+++ b/src/IF.Lastfm.Core.Tests/Api/Commands/AlbumApi/GetAlbumTopTagsCommandTest.cs
@@ -0,0 +1,80 @@
+using IF.Lastfm.Core.Api.Commands.AlbumApi;
+using IF.Lastfm.Core.Api.Enums;
+using IF.Lastfm.Core.Tests.Resources;
+using Microsoft.VisualStudio.TestTools.UnitTesting;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace IF.Lastfm.Core.Tests.Api.Commands.AlbumApi
+{
+
+
+    [TestClass]
+    public class GetAlbumTopTagsCommandTest : CommandTestsBase
+    {
+        private GetAlbumTopTagsCommand _command;
+
+        public GetAlbumTopTagsCommandTest()
+        {
+            _command = new GetAlbumTopTagsCommand(MAuth.Object)
+            {
+                AlbumName = "Believe",
+                ArtistName = "Cher"
+            };
+
+            _command.SetParameters();
+        }
+
+        [TestMethod]
+        public void Constructor()
+        {
+            Assert.AreEqual(_command.Method, "album.gettoptags");
+            Assert.AreEqual(_command.Parameters["album"], "Believe");
+            Assert.AreEqual(_command.Parameters["artist"], "Cher");
+
+        }
+
+        [TestMethod]
+        public async Task HandleSuccessResponse()
+        {
+            var response = CreateResponseMessage(Encoding.UTF8.GetString(AlbumApiResponses.AlbumGetTopTags));
+
+            var parsed = await _command.HandleResponse(response);
+
+            Assert.IsTrue(parsed.Success);
+            Assert.IsNotNull(parsed.Content);
+
+        }
+
+
+        [TestMethod]
+        public async Task HandleEmptyResponse()
+        {
+            var response = CreateResponseMessage(Encoding.UTF8.GetString(AlbumApiResponses.AlbumGetTopTagsEmpty));
+
+            var parsed = await _command.HandleResponse(response);
+
+            Assert.IsTrue(parsed.Success);
+            Assert.IsNotNull(parsed.Content);
+            Assert.IsTrue(!parsed.Content.Any());
+        }
+
+        [TestMethod]
+        public async Task HandleErrorResponse()
+        {
+            var response = CreateResponseMessage(Encoding.UTF8.GetString(AlbumApiResponses.AlbumGetTopTagsError));
+
+            var parsed = await _command.HandleResponse(response);
+
+            Assert.IsFalse(parsed.Success);
+            Assert.IsTrue(parsed.Error == LastFmApiError.MissingParameters);
+            Assert.IsNotNull(parsed.Content);
+            Assert.IsTrue(!parsed.Content.Any());
+        }
+    }
+
+
+}
diff --git a/src/IF.Lastfm.Core.Tests/IF.Lastfm.Core.Tests.csproj b/src/IF.Lastfm.Core.Tests/IF.Lastfm.Core.Tests.csproj
index 6a54462..06c2327 100644
--- a/src/IF.Lastfm.Core.Tests/IF.Lastfm.Core.Tests.csproj
+++ b/src/IF.Lastfm.Core.Tests/IF.Lastfm.Core.Tests.csproj
@@ -57,6 +57,12 @@
     </Reference>
     <Reference Include="System.Net" />
     <Reference Include="System.Net.Http" />
+    <Reference Include="System.Net.Http.Extensions">
+      <HintPath>..\..\packages\Microsoft.Net.Http.2.2.28\lib\net45\System.Net.Http.Extensions.dll</HintPath>
+    </Reference>
+    <Reference Include="System.Net.Http.Primitives">
+      <HintPath>..\..\packages\Microsoft.Net.Http.2.2.28\lib\net45\System.Net.Http.Primitives.dll</HintPath>
+    </Reference>
     <Reference Include="System.Net.Http.WebRequest" />
   </ItemGroup>
   <Choose>
@@ -70,6 +76,7 @@
   <ItemGroup>
     <Compile Include="Api\Commands\AlbumApi\GetAlbumInfoCommandTests.cs" />
     <Compile Include="Api\Commands\AlbumApi\GetAlbumShoutsCommandTests.cs" />
+    <Compile Include="Api\Commands\AlbumApi\GetAlbumTopTagsCommandTest.cs" />
     <Compile Include="Api\Commands\AlbumApi\SearchAlbumsCommandTests.cs" />
     <Compile Include="Api\Commands\CommandTestsBase.cs" />
     <Compile Include="Api\Commands\TrackApi\GetTrackShoutsCommandTests.cs" />
@@ -80,9 +87,9 @@
     <Compile Include="LastFmTests.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
     <Compile Include="Resources\AlbumApiResponses.Designer.cs">
-      <DependentUpon>AlbumApiResponses.resx</DependentUpon>
-      <DesignTime>True</DesignTime>
       <AutoGen>True</AutoGen>
+      <DesignTime>True</DesignTime>
+      <DependentUpon>AlbumApiResponses.resx</DependentUpon>
     </Compile>
     <Compile Include="Resources\TrackApiResponses.Designer.cs">
       <DependentUpon>TrackApiResponses.resx</DependentUpon>
@@ -102,10 +109,13 @@
     <None Include="Resources\AlbumApi\AlbumGetShoutsEmpty.json" />
     <None Include="Resources\AlbumApi\AlbumGetShoutsMultiple.json" />
     <None Include="Resources\AlbumApi\AlbumGetShoutsSingle.json" />
+    <None Include="Resources\AlbumApi\AlbumGetTopTagsEmpty.json" />
+    <None Include="Resources\AlbumApi\AlbumGetTopTagsError.json" />
     <None Include="Resources\AlbumApi\AlbumSearch.json" />
     <None Include="Resources\AlbumApi\AlbumSearchEmpty.json" />
     <None Include="Resources\AlbumApi\AlbumSearchError.json" />
     <None Include="Resources\AlbumApi\AlbumSearchSingle.json" />
+    <None Include="Resources\AlbumApi\AlbumGetTopTags.json" />
     <None Include="Resources\TrackApi\TrackGetShouts.json" />
     <None Include="Resources\TrackApi\TrackGetShoutsEmpty.json" />
     <None Include="Resources\TrackApi\TrackGetShoutsError.json" />
@@ -149,6 +159,11 @@
   <Import Project="$(VSToolsPath)\TeamTest\Microsoft.TestTools.targets" Condition="Exists('$(VSToolsPath)\TeamTest\Microsoft.TestTools.targets')" />
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
   <Import Project="$(SolutionDir)\.nuget\NuGet.targets" Condition="Exists('$(SolutionDir)\.nuget\NuGet.targets')" />
+  <Import Project="..\..\packages\Microsoft.Bcl.Build.1.0.14\tools\Microsoft.Bcl.Build.targets" Condition="Exists('..\..\packages\Microsoft.Bcl.Build.1.0.14\tools\Microsoft.Bcl.Build.targets')" />
+  <Target Name="EnsureBclBuildImported" BeforeTargets="BeforeBuild" Condition="'$(BclBuildImported)' == ''">
+    <Error Condition="!Exists('..\..\packages\Microsoft.Bcl.Build.1.0.14\tools\Microsoft.Bcl.Build.targets')" Text="This project references NuGet package(s) that are missing on this computer. Enable NuGet Package Restore to download them.  For more information, see http://go.microsoft.com/fwlink/?LinkID=317567." HelpKeyword="BCLBUILD2001" />
+    <Error Condition="Exists('..\..\packages\Microsoft.Bcl.Build.1.0.14\tools\Microsoft.Bcl.Build.targets')" Text="The build restored NuGet packages. Build the project again to include these packages in the build. For more information, see http://go.microsoft.com/fwlink/?LinkID=317568." HelpKeyword="BCLBUILD2002" />
+  </Target>
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it. 
        Other similar extension points exist, see Microsoft.Common.targets.
   <Target Name="BeforeBuild">
diff --git a/src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTags.json b/src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTags.json
new file mode 100644
index 0000000..ac563f0
--- /dev/null
+++ b/src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTags.json
@@ -0,0 +1 @@
+{"toptags":{"tag":[{"name":"sourabh","count":"100","url":"http:\/\/www.last.fm\/tag\/sourabh"},{"name":"albums","count":"63","url":"http:\/\/www.last.fm\/tag\/albums"},{"name":"pop","count":"57","url":"http:\/\/www.last.fm\/tag\/pop"},{"name":"90s","count":"52","url":"http:\/\/www.last.fm\/tag\/90s"},{"name":"dance","count":"42","url":"http:\/\/www.last.fm\/tag\/dance"},{"name":"cher","count":"31","url":"http:\/\/www.last.fm\/tag\/cher"},{"name":"1998","count":"21","url":"http:\/\/www.last.fm\/tag\/1998"},{"name":"albums I own","count":"21","url":"http:\/\/www.last.fm\/tag\/albums%20i%20own"},{"name":"Believe","count":"10","url":"http:\/\/www.last.fm\/tag\/believe"},{"name":"sourabh album","count":"10","url":"http:\/\/www.last.fm\/tag\/sourabh%20album"},{"name":"Disco","count":"10","url":"http:\/\/www.last.fm\/tag\/disco"},{"name":"female vocalists","count":"10","url":"http:\/\/www.last.fm\/tag\/female%20vocalists"},{"name":"Awesome","count":"10","url":"http:\/\/www.last.fm\/tag\/awesome"},{"name":"gotanygoodmusic","count":"10","url":"http:\/\/www.last.fm\/tag\/gotanygoodmusic"},{"name":"test2","count":"10","url":"http:\/\/www.last.fm\/tag\/test2"},{"name":"test1","count":"10","url":"http:\/\/www.last.fm\/tag\/test1"},{"name":"favourite albums","count":"10","url":"http:\/\/www.last.fm\/tag\/favourite%20albums"},{"name":"favourite","count":"10","url":"http:\/\/www.last.fm\/tag\/favourite"},{"name":"hot fun in the summertime","count":"7","url":"http:\/\/www.last.fm\/tag\/hot%20fun%20in%20the%20summertime"},{"name":"MyFavorite","count":"7","url":"http:\/\/www.last.fm\/tag\/myfavorite"},{"name":"autotuned","count":"5","url":"http:\/\/www.last.fm\/tag\/autotuned"},{"name":"1999","count":"5","url":"http:\/\/www.last.fm\/tag\/1999"},{"name":"90s pop","count":"5","url":"http:\/\/www.last.fm\/tag\/90s%20pop"},{"name":"good","count":"5","url":"http:\/\/www.last.fm\/tag\/good"},{"name":"female vocalist","count":"5","url":"http:\/\/www.last.fm\/tag\/female%20vocalist"},{"name":"music I like","count":"5","url":"http:\/\/www.last.fm\/tag\/music%20i%20like"},{"name":"Aleyster Crowley Bernardo de Andrade","count":"2","url":"http:\/\/www.last.fm\/tag\/aleyster%20crowley%20bernardo%20de%20andrade"},{"name":"Os Putos Brothers","count":"2","url":"http:\/\/www.last.fm\/tag\/os%20putos%20brothers"},{"name":"Krica Morena Bela","count":"2","url":"http:\/\/www.last.fm\/tag\/krica%20morena%20bela"},{"name":"Amidio Junior","count":"2","url":"http:\/\/www.last.fm\/tag\/amidio%20junior"},{"name":"DonKalifa","count":"2","url":"http:\/\/www.last.fm\/tag\/donkalifa"},{"name":"AleysterCrowley","count":"2","url":"http:\/\/www.last.fm\/tag\/aleystercrowley"},{"name":"simplesdemais","count":"2","url":"http:\/\/www.last.fm\/tag\/simplesdemais"},{"name":"Argeu","count":"2","url":"http:\/\/www.last.fm\/tag\/argeu"},{"name":"Krica Morena","count":"2","url":"http:\/\/www.last.fm\/tag\/krica%20morena"},{"name":"Mosca Navarro","count":"2","url":"http:\/\/www.last.fm\/tag\/mosca%20navarro"},{"name":"Familia Santos","count":"2","url":"http:\/\/www.last.fm\/tag\/familia%20santos"},{"name":"KricaMorena","count":"2","url":"http:\/\/www.last.fm\/tag\/kricamorena"},{"name":"Aleyster","count":"2","url":"http:\/\/www.last.fm\/tag\/aleyster"},{"name":"BEto Juara","count":"2","url":"http:\/\/www.last.fm\/tag\/beto%20juara"},{"name":"Wilson Seixas","count":"2","url":"http:\/\/www.last.fm\/tag\/wilson%20seixas"},{"name":"Silvio Passos","count":"2","url":"http:\/\/www.last.fm\/tag\/silvio%20passos"},{"name":"Argeu california seixas","count":"2","url":"http:\/\/www.last.fm\/tag\/argeu%20california%20seixas"},{"name":"california seixas","count":"2","url":"http:\/\/www.last.fm\/tag\/california%20seixas"},{"name":"Anyzio Rocha","count":"2","url":"http:\/\/www.last.fm\/tag\/anyzio%20rocha"},{"name":"Aleyster Crowley","count":"2","url":"http:\/\/www.last.fm\/tag\/aleyster%20crowley"},{"name":"mucajai","count":"2","url":"http:\/\/www.last.fm\/tag\/mucajai"},{"name":"Don Kalifa","count":"2","url":"http:\/\/www.last.fm\/tag\/don%20kalifa"},{"name":"caracarai","count":"2","url":"http:\/\/www.last.fm\/tag\/caracarai"},{"name":"Marcelo Nova e A Envergadura Moral","count":"2","url":"http:\/\/www.last.fm\/tag\/marcelo%20nova%20e%20a%20envergadura%20moral"},{"name":"os Karetas","count":"2","url":"http:\/\/www.last.fm\/tag\/os%20karetas"},{"name":"Trepidants","count":"2","url":"http:\/\/www.last.fm\/tag\/trepidants"},{"name":"mi micica","count":"2","url":"http:\/\/www.last.fm\/tag\/mi%20micica"},{"name":"mamao com acucar","count":"2","url":"http:\/\/www.last.fm\/tag\/mamao%20com%20acucar"},{"name":"seligalex","count":"2","url":"http:\/\/www.last.fm\/tag\/seligalex"},{"name":"CHER Cool","count":"2","url":"http:\/\/www.last.fm\/tag\/cher%20cool"},{"name":"meuitunesradio","count":"2","url":"http:\/\/www.last.fm\/tag\/meuitunesradio"},{"name":"essential cher","count":"2","url":"http:\/\/www.last.fm\/tag\/essential%20cher"},{"name":"Enoque Gomes","count":"2","url":"http:\/\/www.last.fm\/tag\/enoque%20gomes"},{"name":"Encontro dos Famosos","count":"2","url":"http:\/\/www.last.fm\/tag\/encontro%20dos%20famosos"},{"name":"Rodrigo Otarola","count":"2","url":"http:\/\/www.last.fm\/tag\/rodrigo%20otarola"},{"name":"mindracer","count":"2","url":"http:\/\/www.last.fm\/tag\/mindracer"},{"name":"California Rock Clube","count":"2","url":"http:\/\/www.last.fm\/tag\/california%20rock%20clube"},{"name":"AleysterCrowleyPE","count":"2","url":"http:\/\/www.last.fm\/tag\/aleystercrowleype"},{"name":"obamma","count":"2","url":"http:\/\/www.last.fm\/tag\/obamma"},{"name":"Believe cher","count":"2","url":"http:\/\/www.last.fm\/tag\/believe%20cher"},{"name":"Banda Metamorphosis","count":"2","url":"http:\/\/www.last.fm\/tag\/banda%20metamorphosis"},{"name":"argeu bernardo de andrade","count":"2","url":"http:\/\/www.last.fm\/tag\/argeu%20bernardo%20de%20andrade"},{"name":"tema favorito","count":"2","url":"http:\/\/www.last.fm\/tag\/tema%20favorito"},{"name":"TheBestOfAleysterCrowley","count":"2","url":"http:\/\/www.last.fm\/tag\/thebestofaleystercrowley"},{"name":"negocios","count":"2","url":"http:\/\/www.last.fm\/tag\/negocios"},{"name":"party shuffle","count":"2","url":"http:\/\/www.last.fm\/tag\/party%20shuffle"},{"name":"pa los quince","count":"2","url":"http:\/\/www.last.fm\/tag\/pa%20los%20quince"},{"name":"Eventos","count":"2","url":"http:\/\/www.last.fm\/tag\/eventos"},{"name":"Silva Alhandra","count":"2","url":"http:\/\/www.last.fm\/tag\/silva%20alhandra"},{"name":"nice elevator music","count":"2","url":"http:\/\/www.last.fm\/tag\/nice%20elevator%20music"},{"name":"destiny","count":"2","url":"http:\/\/www.last.fm\/tag\/destiny"},{"name":"anthony","count":"2","url":"http:\/\/www.last.fm\/tag\/anthony"},{"name":"90's","count":"2","url":"http:\/\/www.last.fm\/tag\/90%27s"},{"name":"JAMES BLUNT","count":"2","url":"http:\/\/www.last.fm\/tag\/james%20blunt"},{"name":"musica","count":"2","url":"http:\/\/www.last.fm\/tag\/musica"},{"name":"90s dance","count":"2","url":"http:\/\/www.last.fm\/tag\/90s%20dance"},{"name":"Workout","count":"2","url":"http:\/\/www.last.fm\/tag\/workout"},{"name":"Israel","count":"2","url":"http:\/\/www.last.fm\/tag\/israel"},{"name":"album rock","count":"2","url":"http:\/\/www.last.fm\/tag\/album%20rock"},{"name":"favorite albums","count":"2","url":"http:\/\/www.last.fm\/tag\/favorite%20albums"},{"name":"romance","count":"2","url":"http:\/\/www.last.fm\/tag\/romance"},{"name":"top 40","count":"2","url":"http:\/\/www.last.fm\/tag\/top%2040"},{"name":"singer","count":"2","url":"http:\/\/www.last.fm\/tag\/singer"},{"name":"dance-pop","count":"2","url":"http:\/\/www.last.fm\/tag\/dance-pop"},{"name":"World Music","count":"2","url":"http:\/\/www.last.fm\/tag\/world%20music"},{"name":"eurobeat","count":"2","url":"http:\/\/www.last.fm\/tag\/eurobeat"},{"name":"eurodance","count":"2","url":"http:\/\/www.last.fm\/tag\/eurodance"},{"name":"California","count":"2","url":"http:\/\/www.last.fm\/tag\/california"},{"name":"Europop","count":"2","url":"http:\/\/www.last.fm\/tag\/europop"},{"name":"radiohead","count":"2","url":"http:\/\/www.last.fm\/tag\/radiohead"},{"name":"american","count":"2","url":"http:\/\/www.last.fm\/tag\/american"},{"name":"alternative","count":"2","url":"http:\/\/www.last.fm\/tag\/alternative"},{"name":"latin","count":"2","url":"http:\/\/www.last.fm\/tag\/latin"},{"name":"soul","count":"2","url":"http:\/\/www.last.fm\/tag\/soul"}],"@attr":{"artist":"Cher","album":"Believe"}}}
diff --git a/src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTagsEmpty.json b/src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTagsEmpty.json
new file mode 100644
index 0000000..3fa134b
--- /dev/null
+++ b/src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTagsEmpty.json
@@ -0,0 +1,8 @@
+{
+    "toptags":
+    {
+        "#text":"\n",
+        "artist":"Amit Sana",
+        "album":"Chal Diye"
+    }
+}
\ No newline at end of file
diff --git a/src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTagsError.json b/src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTagsError.json
new file mode 100644
index 0000000..b80ade4
--- /dev/null
+++ b/src/IF.Lastfm.Core.Tests/Resources/AlbumApi/AlbumGetTopTagsError.json
@@ -0,0 +1 @@
+{"error":6,"message":"You must supply either an album & artist name or an album mbid.","links":[]}
diff --git a/src/IF.Lastfm.Core.Tests/Resources/AlbumApiResponses.Designer.cs b/src/IF.Lastfm.Core.Tests/Resources/AlbumApiResponses.Designer.cs
index d444e3d..41229c8 100644
--- a/src/IF.Lastfm.Core.Tests/Resources/AlbumApiResponses.Designer.cs
+++ b/src/IF.Lastfm.Core.Tests/Resources/AlbumApiResponses.Designer.cs
@@ -1,7 +1,7 @@
 //------------------------------------------------------------------------------
 // <auto-generated>
 //     This code was generated by a tool.
-//     Runtime Version:4.0.30319.0
+//     Runtime Version:4.0.30319.34014
 //
 //     Changes to this file may cause incorrect behavior and will be lost if
 //     the code is regenerated.
@@ -120,6 +120,36 @@ internal static byte[] AlbumGetShoutsSingle {
             }
         }
         
+        /// <summary>
+        ///   Looks up a localized resource of type System.Byte[].
+        /// </summary>
+        internal static byte[] AlbumGetTopTags {
+            get {
+                object obj = ResourceManager.GetObject("AlbumGetTopTags", resourceCulture);
+                return ((byte[])(obj));
+            }
+        }
+        
+        /// <summary>
+        ///   Looks up a localized resource of type System.Byte[].
+        /// </summary>
+        internal static byte[] AlbumGetTopTagsEmpty {
+            get {
+                object obj = ResourceManager.GetObject("AlbumGetTopTagsEmpty", resourceCulture);
+                return ((byte[])(obj));
+            }
+        }
+        
+        /// <summary>
+        ///   Looks up a localized resource of type System.Byte[].
+        /// </summary>
+        internal static byte[] AlbumGetTopTagsError {
+            get {
+                object obj = ResourceManager.GetObject("AlbumGetTopTagsError", resourceCulture);
+                return ((byte[])(obj));
+            }
+        }
+        
         /// <summary>
         ///   Looks up a localized resource of type System.Byte[].
         /// </summary>
diff --git a/src/IF.Lastfm.Core.Tests/Resources/AlbumApiResponses.resx b/src/IF.Lastfm.Core.Tests/Resources/AlbumApiResponses.resx
index c177e49..f5b34bf 100644
--- a/src/IF.Lastfm.Core.Tests/Resources/AlbumApiResponses.resx
+++ b/src/IF.Lastfm.Core.Tests/Resources/AlbumApiResponses.resx
@@ -136,6 +136,15 @@
   <data name="AlbumGetShoutsSingle" type="System.Resources.ResXFileRef, System.Windows.Forms">
     <value>albumapi\albumgetshoutssingle.json;System.Byte[], mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
   </data>
+  <data name="AlbumGetTopTags" type="System.Resources.ResXFileRef, System.Windows.Forms">
+    <value>AlbumApi\AlbumGetTopTags.json;System.Byte[], mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
+  </data>
+  <data name="AlbumGetTopTagsEmpty" type="System.Resources.ResXFileRef, System.Windows.Forms">
+    <value>AlbumApi\AlbumGetTopTagsEmpty.json;System.Byte[], mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
+  </data>
+  <data name="AlbumGetTopTagsError" type="System.Resources.ResXFileRef, System.Windows.Forms">
+    <value>AlbumApi\AlbumGetTopTagsError.json;System.Byte[], mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
+  </data>
   <data name="AlbumSearch" type="System.Resources.ResXFileRef, System.Windows.Forms">
     <value>AlbumApi\AlbumSearch.json;System.Byte[], mscorlib, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>
   </data>
diff --git a/src/IF.Lastfm.Core/Api/AlbumApi.cs b/src/IF.Lastfm.Core/Api/AlbumApi.cs
index 9a2a1af..80a3cdb 100644
--- a/src/IF.Lastfm.Core/Api/AlbumApi.cs
+++ b/src/IF.Lastfm.Core/Api/AlbumApi.cs
@@ -48,7 +48,7 @@ public Task<PageResponse<LastTag>> GetUserTagsForAlbumAsync(string artist, strin
 
         public async Task<PageResponse<LastTag>> GetTopTagsForAlbumAsync(string artist, string album, bool autocorrect = false)
         {
-            var command = new AlbumTopTagsCommand(Auth)
+            var command = new GetAlbumTopTagsCommand(Auth)
             {
                 ArtistName = artist,
                 AlbumName = album
diff --git a/src/IF.Lastfm.Core/Api/Commands/AlbumApi/AlbumTopTagsCommand.cs b/src/IF.Lastfm.Core/Api/Commands/AlbumApi/GetAlbumTopTagsCommand.cs
similarity index 90%
rename from src/IF.Lastfm.Core/Api/Commands/AlbumApi/AlbumTopTagsCommand.cs
rename to src/IF.Lastfm.Core/Api/Commands/AlbumApi/GetAlbumTopTagsCommand.cs
index 69226b9..7871212 100644
--- a/src/IF.Lastfm.Core/Api/Commands/AlbumApi/AlbumTopTagsCommand.cs
+++ b/src/IF.Lastfm.Core/Api/Commands/AlbumApi/GetAlbumTopTagsCommand.cs
@@ -12,7 +12,7 @@
 
 namespace IF.Lastfm.Core.Api.Commands.AlbumApi
 {
-    internal class AlbumTopTagsCommand : GetAsyncCommandBase<PageResponse<LastTag>>
+    internal class GetAlbumTopTagsCommand : GetAsyncCommandBase<PageResponse<LastTag>>
     {
         public string AlbumMbid { get; set; }
 
@@ -22,13 +22,13 @@ internal class AlbumTopTagsCommand : GetAsyncCommandBase<PageResponse<LastTag>>
 
         public bool Autocorrect { get; set; }
 
-        public AlbumTopTagsCommand(ILastAuth auth)
+        public GetAlbumTopTagsCommand(ILastAuth auth)
             : base(auth)
         {
             Method = "album.gettoptags";
         }
 
-        public AlbumTopTagsCommand(ILastAuth auth, string album, string artist)
+        public GetAlbumTopTagsCommand(ILastAuth auth, string album, string artist)
             : this(auth)
         {
             AlbumName = album;
diff --git a/src/IF.Lastfm.Core/IF.Lastfm.Core.csproj b/src/IF.Lastfm.Core/IF.Lastfm.Core.csproj
index 553c2e6..a42842d 100644
--- a/src/IF.Lastfm.Core/IF.Lastfm.Core.csproj
+++ b/src/IF.Lastfm.Core/IF.Lastfm.Core.csproj
@@ -40,7 +40,7 @@
   <ItemGroup>
     <Compile Include="Api\AlbumApi.cs" />
     <Compile Include="Api\ArtistApi.cs" />
-    <Compile Include="Api\Commands\AlbumApi\AlbumTopTagsCommand.cs" />
+    <Compile Include="Api\Commands\AlbumApi\GetAlbumTopTagsCommand.cs" />
     <Compile Include="Api\Commands\UserApi\GetRecommendedArtistsCommand.cs" />
     <Compile Include="Api\LastAuth.cs" />
     <Compile Include="Api\ChartApi.cs" />