From fd66ea54724e0f0084d9aaf0645d2004a3a49384 Mon Sep 17 00:00:00 2001
From: Henrik Andersson <henrik.a.andersson@outlook.com>
Date: Sat, 14 Jun 2014 23:57:07 +1000
Subject: [PATCH] Implement repo deploy keys client

Update comments
Add ApiUrls methods
---
 .../Clients/IRepositoryDeployKeysClient.cs    | 15 +++++++
 Octokit/Clients/RepositoryDeployKeysClient.cs | 45 +++++++++++++++++--
 Octokit/Helpers/ApiUrls.cs                    | 23 ++++++++++
 3 files changed, 79 insertions(+), 4 deletions(-)

diff --git a/Octokit/Clients/IRepositoryDeployKeysClient.cs b/Octokit/Clients/IRepositoryDeployKeysClient.cs
index d5e5eea7..cf4af6bd 100644
--- a/Octokit/Clients/IRepositoryDeployKeysClient.cs
+++ b/Octokit/Clients/IRepositoryDeployKeysClient.cs
@@ -11,6 +11,9 @@ namespace Octokit
         /// <summary>
         /// Get a single deploy key by number for a repository.
         /// </summary>
+        /// <remarks>
+        /// See the <a href="https://developer.github.com/v3/repos/keys/#get"> API documentation</a> for more information.
+        /// </remarks>
         /// <param name="owner">The owner of the repository.</param>
         /// <param name="name">The name of the repository.</param>
         /// <param name="number">The id of the deploy key.</param>
@@ -20,6 +23,9 @@ namespace Octokit
         /// <summary>
         /// Get all deploy keys for a repository.
         /// </summary>
+        /// <remarks>
+        /// See the <a href="https://developer.github.com/v3/repos/keys/#list"> API documentation</a> for more information.
+        /// </remarks>
         /// <param name="owner">The owner of the repository.</param>
         /// <param name="name">The name of the repository.</param>
         Task<IReadOnlyList<DeployKey>> GetForRepository(string owner, string name);
@@ -27,6 +33,9 @@ namespace Octokit
         /// <summary>
         /// Creates a new deploy key for a repository.
         /// </summary>
+        /// <remarks>
+        /// See the <a href="https://developer.github.com/v3/repos/keys/#create"> API documentation</a> for more information.
+        /// </remarks>
         /// <param name="owner">The owner of the repository.</param>
         /// <param name="name">The name of the repository.</param>
         /// <param name="newDeployKey">The deploy key to create for the repository.</param>
@@ -36,6 +45,9 @@ namespace Octokit
         /// <summary>
         /// Deploy keys are immutable. If you need to update a key, remove the key and create a new one instead.
         /// </summary>
+        /// <remarks>
+        /// See the <a href="https://developer.github.com/v3/repos/keys/#edit"> API documentation</a> for more information.
+        /// </remarks>
         /// <param name="owner"></param>
         /// <param name="name"></param>
         /// <param name="number"></param>
@@ -46,6 +58,9 @@ namespace Octokit
         /// <summary>
         /// Deletes a deploy key from a repository.
         /// </summary>
+        /// <remarks>
+        /// See the <a href="https://developer.github.com/v3/repos/keys/#delete"> API documentation</a> for more information.
+        /// </remarks>
         /// <param name="owner">The owner of the repository.</param>
         /// <param name="name">The name of the repository.</param>
         /// <param name="number">The id of the deploy key to delete.</param>
diff --git a/Octokit/Clients/RepositoryDeployKeysClient.cs b/Octokit/Clients/RepositoryDeployKeysClient.cs
index a9f6006e..fcb5a287 100644
--- a/Octokit/Clients/RepositoryDeployKeysClient.cs
+++ b/Octokit/Clients/RepositoryDeployKeysClient.cs
@@ -26,55 +26,92 @@ namespace Octokit
         /// <summary>
         /// Get a single deploy key by number for a repository.
         /// </summary>
+        /// <remarks>
+        /// See the <a href="https://developer.github.com/v3/repos/keys/#get"> API documentation</a> for more information.
+        /// </remarks>
         /// <param name="owner">The owner of the repository.</param>
         /// <param name="name">The name of the repository.</param>
         /// <param name="number">The id of the deploy key.</param>
         public Task<DeployKey> Get(string owner, string name, int number)
         {
-            throw new NotImplementedException();
+            Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
+            Ensure.ArgumentNotNull(number, "number");
+
+            return ApiConnection.Get<DeployKey>(ApiUrls.RepositoryDeployKey(owner, name, number));
         }
 
         /// <summary>
         /// Get all deploy keys for a repository.
         /// </summary>
+        /// <remarks>
+        /// See the <a href="https://developer.github.com/v3/repos/keys/#list"> API documentation</a> for more information.
+        /// </remarks>
         /// <param name="owner">The owner of the repository.</param>
         /// <param name="name">The name of the repository.</param>
         public Task<IReadOnlyList<DeployKey>> GetForRepository(string owner, string name)
         {
-            throw new NotImplementedException();
+            Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
+
+            return ApiConnection.Get<IReadOnlyList<DeployKey>>(ApiUrls.RepositoryDeployKeys(owner, name));
         }
 
         /// <summary>
         /// Creates a new deploy key for a repository.
         /// </summary>
+        /// <remarks>
+        /// See the <a href="https://developer.github.com/v3/repos/keys/#create"> API documentation</a> for more information.
+        /// </remarks>
         /// <param name="owner">The owner of the repository.</param>
         /// <param name="name">The name of the repository.</param>
         /// <param name="newDeployKey">The deploy key to create for the repository.</param>
         /// <returns></returns>
         public Task<DeployKey> Create(string owner, string name, NewDeployKey newDeployKey)
         {
-            throw new NotImplementedException();
+            Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
+            Ensure.ArgumentNotNull(newDeployKey, "newDeployKey");
+
+            if (string.IsNullOrWhiteSpace(newDeployKey.Title))
+                throw new ArgumentException("The new deploy key's title must not be null.");
+
+            if (string.IsNullOrWhiteSpace(newDeployKey.Key))
+                throw new ArgumentException("The new deploy key's key must not be null.");
+
+            return ApiConnection.Post<DeployKey>(ApiUrls.RepositoryDeployKeys(owner, name), newDeployKey);
         }
 
         /// <summary>
         /// Deploy keys are immutable. If you need to update a key, remove the key and create a new one instead.
         /// </summary>
+        /// <remarks>
+        /// https://developer.github.com/v3/repos/keys/#edit
+        /// </remarks>
         /// <param name="owner"></param>
         /// <param name="name"></param>
         /// <param name="number"></param>
         /// <param name="newDeployKey"></param>
         /// <returns></returns>
         /// Task<DeployKey> Update(string owner, string name, int number, NewDeployKey newDeployKey);
+        
         /// <summary>
         /// Deletes a deploy key from a repository.
         /// </summary>
+        /// <remarks>
+        /// See the <a href="https://developer.github.com/v3/repos/keys/#delete"> API documentation</a> for more information.
+        /// </remarks>
         /// <param name="owner">The owner of the repository.</param>
         /// <param name="name">The name of the repository.</param>
         /// <param name="number">The id of the deploy key to delete.</param>
         /// <returns></returns>
         public Task Delete(string owner, string name, int number)
         {
-            throw new NotImplementedException();
+            Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
+            Ensure.ArgumentNotNull(number, "number");
+
+            return ApiConnection.Delete(ApiUrls.RepositoryDeployKey(owner, name, number));
         }
     }
 }
diff --git a/Octokit/Helpers/ApiUrls.cs b/Octokit/Helpers/ApiUrls.cs
index 984f159c..b5c03f0c 100644
--- a/Octokit/Helpers/ApiUrls.cs
+++ b/Octokit/Helpers/ApiUrls.cs
@@ -1112,6 +1112,29 @@ namespace Octokit
             return "repos/{0}/{1}".FormatUri(owner, name);
         }
 
+        /// <summary>
+        /// Returns the <see cref="Uri"/> for a deploy key for a repository
+        /// </summary>
+        /// <param name="owner">The owner of the repository</param>
+        /// <param name="name">The name of the repository</param>
+        /// <param name="number">The id of the deploy key of the repository</param>
+        /// <returns></returns>
+        public static Uri RepositoryDeployKey(string owner, string name, int number)
+        {
+            return "repos/{0}/{1}/keys/{1}".FormatUri(owner, name, number);
+        }
+
+        /// <summary>
+        /// Returns the <see cref="Uri"/> for deploy keys for a repository.
+        /// </summary>
+        /// <param name="owner">The owner of the repository</param>
+        /// <param name="name">The name of the repository</param>
+        /// <returns></returns>
+        public static Uri RepositoryDeployKeys(string owner, string name)
+        {
+            return "repos/{0}/{1}/keys".FormatUri(owner, name);
+        }
+
         /// <summary>
         /// Returns the <see cref="System.Uri"/> for the Deployments API for the given repository.
         /// </summary>