Added functionality for retrieving repositories for the authenticated…

… user. Closes #25
baynezy · May 6, 2016 · 504feb8 · 504feb8
1 parent 7f75087
commit 504feb8
Show file tree

Hide file tree

Showing 8 changed files with 55 additions and 9 deletions.
diff --git a/src/GitHubLabelInitialiser/Extensions/ConvertToGitHubRepositoryExtension.cs b/src/GitHubLabelInitialiser/Extensions/ConvertToGitHubRepositoryExtension.cs
@@ -0,0 +1,16 @@
+using GitHubLabelInitialiser.Models;
+using Octokit;
+
+namespace GitHubLabelInitialiser.Extensions
+{
+	static class ConvertToGitHubRepositoryExtension
+	{
+		public static GitHubRepository ConvertToGitHubRepository(this Repository repository)
+		{
+			return new GitHubRepository
+				{
+					Name = repository.Name
+				};
+		}
+	}
+}
diff --git a/src/GitHubLabelInitialiser/GitHubApi.cs b/src/GitHubLabelInitialiser/GitHubApi.cs
@@ -48,9 +48,11 @@ public async Task<GitHubLabel> AddLabel(string username, string repositoryName,
 			return label;
 		}
 
-		public Task<IList<GitHubRepository>> GetAllForAuthenticatedUser()
+		public async Task<IList<GitHubRepository>> GetAllForAuthenticatedUser()
 		{
-			throw new System.NotImplementedException();
+			var repositories = await _client.Repository.GetAllForCurrent();
+
+			return repositories.Select(r => r.ConvertToGitHubRepository()).ToList();
 		}
 
 		private static NewLabel CreateLabel(GitHubLabel label)

diff --git a/src/GitHubLabelInitialiser/GitHubLabelInitialiser.csproj b/src/GitHubLabelInitialiser/GitHubLabelInitialiser.csproj
@@ -47,6 +47,7 @@
   </ItemGroup>
   <ItemGroup>
     <Compile Include="Extensions\ConvertToGitHubLabelExtension.cs" />
+    <Compile Include="Extensions\ConvertToGitHubRepositoryExtension.cs" />
     <Compile Include="GitHubApi.cs" />
     <Compile Include="IRepositoryManager.cs" />
     <Compile Include="Models\GitHubCredentials.cs" />

diff --git a/src/GitHubLabelInitialiser/IRepositoryManager.cs b/src/GitHubLabelInitialiser/IRepositoryManager.cs
@@ -1,7 +1,11 @@
-namespace GitHubLabelInitialiser
+using System.Collections.Generic;
+using System.Threading.Tasks;
+using GitHubLabelInitialiser.Models;
+
+namespace GitHubLabelInitialiser
 {
 	public interface IRepositoryManager
 	{
-		void GetAllForAuthenticatedUser();
+		Task<IList<GitHubRepository>> GetAllForAuthenticatedUser();
 	}
 }
diff --git a/src/GitHubLabelInitialiser/Models/GitHubRepository.cs b/src/GitHubLabelInitialiser/Models/GitHubRepository.cs
@@ -2,5 +2,6 @@
 {
 	public class GitHubRepository
 	{
+		public string Name { get; set; }
 	}
 }
diff --git a/src/GitHubLabelInitialiser/RepositoryManager.cs b/src/GitHubLabelInitialiser/RepositoryManager.cs
@@ -1,4 +1,8 @@
-namespace GitHubLabelInitialiser
+using System.Collections.Generic;
+using System.Threading.Tasks;
+using GitHubLabelInitialiser.Models;
+
+namespace GitHubLabelInitialiser
 {
 	public class RepositoryManager : IRepositoryManager
 	{
@@ -9,9 +13,9 @@ public RepositoryManager(IGitHubApi gitHubApi)
 			_gitHubApi = gitHubApi;
 		}
 
-		public void GetAllForAuthenticatedUser()
+		public Task<IList<GitHubRepository>> GetAllForAuthenticatedUser()
 		{
-			_gitHubApi.GetAllForAuthenticatedUser();
+			return _gitHubApi.GetAllForAuthenticatedUser();
 		}
 	}
 }
diff --git a/test/GitHubLabelInitialiser.Test/LabelManagerTest.cs b/test/GitHubLabelInitialiser.Test/LabelManagerTest.cs
@@ -111,8 +111,6 @@ public void IntialiseLabels_WhenCalledWithLabels_ThenAddNewLabels()
 			api.Verify(f => f.AddLabel(_username, _repositoryName, It.IsAny<GitHubLabel>()), Times.Exactly(2));
 		}
 
-
-
 		private static ILabelManager CreateManager(IGitHubApi gitHubApi = null)
 		{
 			return new LabelManager(gitHubApi ?? new Mock<IGitHubApi>().Object);

diff --git a/test/GitHubLabelInitialiser.Test/RepositoryManagerTest.cs b/test/GitHubLabelInitialiser.Test/RepositoryManagerTest.cs
@@ -1,4 +1,5 @@
 using System.Collections.Generic;
+using System.Threading.Tasks;
 using GitHubLabelInitialiser.Models;
 using Moq;
 using NUnit.Framework;
@@ -28,6 +29,25 @@ public void GetAllForAuthenticatedUser_WhenCalled_ThenShouldCallIGitHubApiGetAll
 			api.Verify(f => f.GetAllForAuthenticatedUser(), Times.Once);
 		}
 
+		[Test]
+		public async Task GetAllForAuthenticatedUser_WhenCalled_ThenShouldReturnRepositoriesFromIGitHubApi()
+		{
+			var api = new Mock<IGitHubApi>();
+			var expectedRepo = new GitHubRepository {Name = "some-name"};
+			api.Setup(m => m.GetAllForAuthenticatedUser()).ReturnsAsync(new List<GitHubRepository>
+				{
+					expectedRepo
+				});
+
+			var manager = CreateManager(api.Object);
+
+			var repositories = await manager.GetAllForAuthenticatedUser();
+
+			Assert.That(repositories.Count, Is.EqualTo(1));
+			var returnedRepo = repositories[0];
+			Assert.That(returnedRepo, Is.EqualTo(expectedRepo));
+		}
+
 		private static IRepositoryManager CreateManager(IGitHubApi api = null)
 		{
 			return new RepositoryManager(api ?? new Mock<IGitHubApi>().Object);