From 8f44a22ca3ffff4590dd357cf4aa92ac75c335c4 Mon Sep 17 00:00:00 2001
From: Andre Rodrigues <andrerod@microsoft.com>
Date: Thu, 5 Dec 2013 17:24:52 +0000
Subject: [PATCH] Implement labels CRUD

---
 Octokit/Clients/IIssuesLabelsClient.cs | 56 ++++++++++++++++
 Octokit/Clients/IssuesClient.cs        |  2 +
 Octokit/Clients/IssuesLabelsClient.cs  | 93 ++++++++++++++++++++++++++
 Octokit/Helpers/ApiUrls.cs             | 26 ++++++-
 Octokit/Models/Request/LabelUpdate.cs  | 22 ++++++
 Octokit/Models/Request/NewLabel.cs     | 24 +++++++
 Octokit/Models/Response/Label.cs       | 11 +++
 Octokit/Octokit-Mono.csproj            |  4 ++
 Octokit/Octokit-netcore45.csproj       |  4 ++
 Octokit/Octokit.csproj                 |  4 ++
 10 files changed, 244 insertions(+), 2 deletions(-)
 create mode 100644 Octokit/Clients/IIssuesLabelsClient.cs
 create mode 100644 Octokit/Clients/IssuesLabelsClient.cs
 create mode 100644 Octokit/Models/Request/LabelUpdate.cs
 create mode 100644 Octokit/Models/Request/NewLabel.cs

diff --git a/Octokit/Clients/IIssuesLabelsClient.cs b/Octokit/Clients/IIssuesLabelsClient.cs
new file mode 100644
index 00000000..c47fdea6
--- /dev/null
+++ b/Octokit/Clients/IIssuesLabelsClient.cs
@@ -0,0 +1,56 @@
+using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
+using System.Threading.Tasks;
+
+namespace Octokit
+{
+    public interface IIssuesLabelsClient
+    {
+        /// <summary>
+        /// Gets all  labels for the repository.
+        /// </summary>
+        /// <remarks>
+        /// http://developer.github.com/v3/repos/:owner/:repo/labels/:name
+        /// </remarks>
+        /// <returns></returns>
+        Task<IReadOnlyList<Label>> GetForRepository(string owner, string repo);
+
+        /// <summary>
+        /// Gets a single Label by name.
+        /// </summary>
+        /// <remarks>
+        /// http://developer.github.com/v3/repos/:owner/:repo/labels/:name
+        /// </remarks>
+        /// <returns></returns>
+        [SuppressMessage("Microsoft.Naming", "CA1716:IdentifiersShouldNotMatchKeywords", MessageId = "Get",
+        Justification = "Method makes a network request")]
+        Task<Label> Get(string owner, string repo, string name);
+
+        /// <summary>
+        /// Deletes a label.
+        /// </summary>
+        /// <remarks>
+        /// http://developer.github.com/v3/repos/:owner/:repo/labels/:label
+        /// </remarks>
+        /// <returns></returns>
+        Task Delete(string owner, string repo, string name);
+
+        /// <summary>
+        /// Deletes a label.
+        /// </summary>
+        /// <remarks>
+        /// http://developer.github.com/v3/repos/:owner/:repo/labels/:label
+        /// </remarks>
+        /// <returns></returns>
+        Task<Label> Create(string owner, string repo, NewLabel newLabel);
+
+        /// <summary>
+        /// Updates a label.
+        /// </summary>
+        /// <remarks>
+        /// http://developer.github.com/v3/repos/:owner/:repo/labels/:label
+        /// </remarks>
+        /// <returns></returns>
+        Task<Label> Create(string owner, string repo, string name, LabelUpdate labelUpdate);
+    }
+}
\ No newline at end of file
diff --git a/Octokit/Clients/IssuesClient.cs b/Octokit/Clients/IssuesClient.cs
index 9c2ddc05..85209fab 100644
--- a/Octokit/Clients/IssuesClient.cs
+++ b/Octokit/Clients/IssuesClient.cs
@@ -9,12 +9,14 @@ namespace Octokit
         {
             Assignee = new AssigneesClient(apiConnection);
             Events = new IssuesEventsClient(apiConnection);
+            Labels = new IssuesLabelsClient(apiConnection);
             Milestone = new MilestonesClient(apiConnection);
             Comment = new IssueCommentsClient(apiConnection);
         }
 
         public IAssigneesClient Assignee { get; private set; }
         public IIssuesEventsClient Events { get; private set; }
+        public IIssuesLabelsClient Labels { get; private set; }
         public IMilestonesClient Milestone { get; private set; }
         public IIssueCommentsClient Comment { get; private set; }
 
diff --git a/Octokit/Clients/IssuesLabelsClient.cs b/Octokit/Clients/IssuesLabelsClient.cs
new file mode 100644
index 00000000..6b05b730
--- /dev/null
+++ b/Octokit/Clients/IssuesLabelsClient.cs
@@ -0,0 +1,93 @@
+using System.Collections.Generic;
+using System.Threading.Tasks;
+
+namespace Octokit
+{
+    public class IssuesLabelsClient : ApiClient, IIssuesLabelsClient
+    {
+        public IssuesLabelsClient(IApiConnection apiConnection)
+            : base(apiConnection)
+        {
+        }
+
+        /// <summary>
+        /// Gets all  labels for the repository.
+        /// </summary>
+        /// <remarks>
+        /// http://developer.github.com/v3/repos/:owner/:repo/labels/:name
+        /// </remarks>
+        /// <returns></returns>
+        public Task<IReadOnlyList<Label>> GetForRepository(string owner, string repo)
+        {
+            Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
+            Ensure.ArgumentNotNullOrEmptyString(repo, "repo");
+
+            return ApiConnection.GetAll<Label>(ApiUrls.Labels(owner, repo));
+        }
+
+        /// <summary>
+        /// Gets a single Label by name.
+        /// </summary>
+        /// <remarks>
+        /// http://developer.github.com/v3/repos/:owner/:repo/labels/:name
+        /// </remarks>
+        /// <returns></returns>
+        public Task<Label> Get(string owner, string repo, string name)
+        {
+            Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
+            Ensure.ArgumentNotNullOrEmptyString(repo, "repo");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
+
+            return ApiConnection.Get<Label>(ApiUrls.Label(owner, repo, name));
+        }
+
+        /// <summary>
+        /// Creates a label.
+        /// </summary>
+        /// <remarks>
+        /// http://developer.github.com/v3/repos/:owner/:repo/labels/
+        /// </remarks>
+        /// <returns></returns>
+        public Task<Label> Create(string owner, string repo, NewLabel newLabel)
+        {
+            Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
+            Ensure.ArgumentNotNullOrEmptyString(repo, "repo");
+            Ensure.ArgumentNotNull(newLabel, "newLabel");
+
+            return ApiConnection.Post<Label>(ApiUrls.Labels(owner, repo), newLabel);
+        }
+
+        /// <summary>
+        /// Updates a label.
+        /// </summary>
+        /// <remarks>
+        /// http://developer.github.com/v3/repos/:owner/:repo/labels/:label
+        /// </remarks>
+        /// <returns></returns>
+        public Task<Label> Create(string owner, string repo, string name, LabelUpdate labelUpdate)
+        {
+            Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
+            Ensure.ArgumentNotNullOrEmptyString(repo, "repo");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
+            Ensure.ArgumentNotNull(labelUpdate, "labelUpdate");
+
+            return ApiConnection.Post<Label>(ApiUrls.Label(owner, repo, name), labelUpdate);
+        }
+
+        /// <summary>
+        /// Deletes a label.
+        /// </summary>
+        /// <remarks>
+        /// http://developer.github.com/v3/repos/:owner/:repo/labels/
+        /// </remarks>
+        /// <returns></returns>
+        public Task Delete(string owner, string repo, string name)
+        {
+            Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
+            Ensure.ArgumentNotNullOrEmptyString(repo, "repo");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
+
+            return ApiConnection.Delete(ApiUrls.Label(owner, repo, name));
+        }
+    }
+}
diff --git a/Octokit/Helpers/ApiUrls.cs b/Octokit/Helpers/ApiUrls.cs
index 78586961..398c0526 100644
--- a/Octokit/Helpers/ApiUrls.cs
+++ b/Octokit/Helpers/ApiUrls.cs
@@ -348,13 +348,12 @@ namespace Octokit
         /// </summary>
         /// <param name="owner">The owner of the repository</param>
         /// <param name="name">The name of the repository</param>
-        /// /// <param name="number">The milestone number</param>
+        /// <param name="number">The milestone number</param>
         /// <returns></returns>
         public static Uri Milestone(string owner, string name, int number)
         {
             return "repos/{0}/{1}/milestones/{2}".FormatUri(owner, name, number);
         }
-
         /// <summary>
         /// Returns the <see cref="Uri"/> that returns all of the milestones for the specified repository.
         /// </summary>
@@ -366,6 +365,29 @@ namespace Octokit
             return "repos/{0}/{1}/milestones".FormatUri(owner, name);
         }
 
+        /// <summary>
+        /// Returns the <see cref="Uri"/> that returns the specified label.
+        /// </summary>
+        /// <param name="owner">The owner of the repository</param>
+        /// <param name="repo">The name of the repository</param>
+        /// <param name="name">The milestone number</param>
+        /// <returns></returns>
+        public static Uri Label(string owner, string repo, string name)
+        {
+            return "repos/{0}/{1}/labels/{2}".FormatUri(owner, repo, name);
+        }
+
+        /// <summary>
+        /// Returns the <see cref="Uri"/> that returns all of the labels for the specified repository.
+        /// </summary>
+        /// <param name="owner">The owner of the repository</param>
+        /// <param name="repo">The name of the repository</param>
+        /// <returns></returns>
+        public static Uri Labels(string owner, string repo)
+        {
+            return "repos/{0}/{1}/labels/".FormatUri(owner, repo);
+        }
+
         /// <summary>
         /// Returns the <see cref="Uri"/> that lists the commit statuses for the specified reference.
         /// </summary>
diff --git a/Octokit/Models/Request/LabelUpdate.cs b/Octokit/Models/Request/LabelUpdate.cs
new file mode 100644
index 00000000..f8f140f8
--- /dev/null
+++ b/Octokit/Models/Request/LabelUpdate.cs
@@ -0,0 +1,22 @@
+namespace Octokit
+{
+    public class LabelUpdate
+    {
+        public LabelUpdate(string name, string color)
+        {
+            Name = name;
+            Color = color;
+        }
+
+
+        /// <summary>
+        /// Name of the label (required)
+        /// </summary>
+        public string Name { get; set; }
+
+        /// <summary>
+        /// Color of the label (required)
+        /// </summary>
+        public string Color { get; set; }
+    }
+}
diff --git a/Octokit/Models/Request/NewLabel.cs b/Octokit/Models/Request/NewLabel.cs
new file mode 100644
index 00000000..ac0dd2a2
--- /dev/null
+++ b/Octokit/Models/Request/NewLabel.cs
@@ -0,0 +1,24 @@
+namespace Octokit
+{
+    /// <summary>
+    /// Describes a new label to create via the <see cref="IIssuesLabelsClient.Create(string,string,NewLabel)"/> method.
+    /// </summary>
+    public class NewLabel
+    {
+        public NewLabel(string name, string color)
+        {
+            Name = name;
+            Color = color;
+        }
+
+        /// <summary>
+        /// Name of the label (required)
+        /// </summary>
+        public string Name { get; set; }
+
+        /// <summary>
+        /// Color of the label (required)
+        /// </summary>
+        public string Color { get; set; }
+    }
+}
diff --git a/Octokit/Models/Response/Label.cs b/Octokit/Models/Response/Label.cs
index 0e557228..e2119565 100644
--- a/Octokit/Models/Response/Label.cs
+++ b/Octokit/Models/Response/Label.cs
@@ -4,8 +4,19 @@ namespace Octokit
 {
     public class Label
     {
+        /// <summary>
+        /// Url of the label
+        /// </summary>
         public Uri Url { get; set; }
+
+        /// <summary>
+        /// Name of the label
+        /// </summary>
         public string Name { get; set; }
+
+        /// <summary>
+        /// Color of the label
+        /// </summary>
         public string Color { get; set; }
     }
 }
\ No newline at end of file
diff --git a/Octokit/Octokit-Mono.csproj b/Octokit/Octokit-Mono.csproj
index 80d77f4e..4805f639 100644
--- a/Octokit/Octokit-Mono.csproj
+++ b/Octokit/Octokit-Mono.csproj
@@ -57,11 +57,13 @@
     <Compile Include="Clients\IGitDatabaseClient.cs" />
     <Compile Include="Clients\IIssueCommentsClient.cs" />
     <Compile Include="Clients\IIssuesEventsClient.cs" />
+    <Compile Include="Clients\IIssuesLabelsClient.cs" />
     <Compile Include="Clients\IOrganizationMembersClient.cs" />
     <Compile Include="Clients\IReferencesClient.cs" />
     <Compile Include="Clients\IssueCommentsClient.cs" />
     <Compile Include="Clients\IssuesClient.cs" />
     <Compile Include="Clients\IssuesEventsClient.cs" />
+    <Compile Include="Clients\IssuesLabelsClient.cs" />
     <Compile Include="Clients\IStarredClient.cs" />
     <Compile Include="Clients\ITagsClient.cs" />
     <Compile Include="Clients\ITreesClient.cs" />
@@ -79,10 +81,12 @@
     <Compile Include="Clients\IMilestonesClient.cs" />
     <Compile Include="Helpers\ParameterAttribute.cs" />
     <Compile Include="Helpers\ReflectionExtensions.cs" />
+    <Compile Include="Models\Request\LabelUpdate.cs" />
     <Compile Include="Models\Request\MilestoneUpdate.cs" />
     <Compile Include="Models\Request\NewBlob.cs" />
     <Compile Include="Models\Request\NewCommit.cs" />
     <Compile Include="Models\Request\NewCommitStatus.cs" />
+    <Compile Include="Models\Request\NewLabel.cs" />
     <Compile Include="Models\Request\NewMilestone.cs" />
     <Compile Include="Models\Request\NewReference.cs" />
     <Compile Include="Models\Request\NewTag.cs" />
diff --git a/Octokit/Octokit-netcore45.csproj b/Octokit/Octokit-netcore45.csproj
index f621cbd4..a115d596 100644
--- a/Octokit/Octokit-netcore45.csproj
+++ b/Octokit/Octokit-netcore45.csproj
@@ -71,6 +71,7 @@
     <Compile Include="Clients\IIssueCommentsClient.cs" />
     <Compile Include="Clients\IIssuesClient.cs" />
     <Compile Include="Clients\IIssuesEventsClient.cs" />
+    <Compile Include="Clients\IIssuesLabelsClient.cs" />
     <Compile Include="Clients\IMilestonesClient.cs" />
     <Compile Include="Clients\IMiscellaneousClient.cs" />
     <Compile Include="Clients\INotificationsClient.cs" />
@@ -82,6 +83,7 @@
     <Compile Include="Clients\ISshKeysClient.cs" />
     <Compile Include="Clients\IssueCommentsClient.cs" />
     <Compile Include="Clients\IssuesClient.cs" />
+    <Compile Include="Clients\IssuesLabelsClient.cs" />
     <Compile Include="Clients\IStarredClient.cs" />
     <Compile Include="Clients\ITagsClient.cs" />
     <Compile Include="Clients\IssuesEventsClient.cs" />
@@ -155,6 +157,7 @@
     <Compile Include="Models\Request\AuthorizationUpdate.cs" />
     <Compile Include="Models\Request\IssueRequest.cs" />
     <Compile Include="Models\Request\IssueUpdate.cs" />
+    <Compile Include="Models\Request\LabelUpdate.cs" />
     <Compile Include="Models\Request\MilestoneRequest.cs" />
     <Compile Include="Models\Request\MilestoneUpdate.cs" />
     <Compile Include="Models\Request\NewAuthorization.cs" />
@@ -162,6 +165,7 @@
     <Compile Include="Models\Request\NewCommit.cs" />
     <Compile Include="Models\Request\NewCommitStatus.cs" />
     <Compile Include="Models\Request\NewIssue.cs" />
+    <Compile Include="Models\Request\NewLabel.cs" />
     <Compile Include="Models\Request\NewMilestone.cs" />
     <Compile Include="Models\Request\NewReference.cs" />
     <Compile Include="Models\Request\NewRepository.cs" />
diff --git a/Octokit/Octokit.csproj b/Octokit/Octokit.csproj
index e1c46319..8a0a4f69 100644
--- a/Octokit/Octokit.csproj
+++ b/Octokit/Octokit.csproj
@@ -57,9 +57,13 @@
     <Compile Include="Clients\GistsClient.cs" />
     <Compile Include="Clients\IGistCommentsClient.cs" />
     <Compile Include="Clients\IGistsClient.cs" />
+    <Compile Include="Clients\IIssuesLabelsClient.cs" />
     <Compile Include="Clients\IReferencesClient.cs" />
+    <Compile Include="Clients\IssuesLabelsClient.cs" />
     <Compile Include="Clients\ReferencesClient.cs" />
     <Compile Include="Models\Request\BodyWrapper.cs" />
+    <Compile Include="Models\Request\LabelUpdate.cs" />
+    <Compile Include="Models\Request\NewLabel.cs" />
     <Compile Include="Models\Request\NewReference.cs" />
     <Compile Include="Models\Request\ReferenceUpdate.cs" />
     <Compile Include="Models\Response\BlobReference.cs" />