[Approved] Add repository topics

Octokit.Reactive/Clients/IObservableRepositoriesClient.cs

@@ -1,6 +1,8 @@
 using System;
+using System.Collections.Generic;
 using System.Diagnostics.CodeAnalysis;
 using System.Reactive;
+using System.Threading.Tasks;
 
 namespace Octokit.Reactive
 {
@@ -556,5 +558,79 @@ public interface IObservableRepositoriesClient
         /// Refer to the API documentation for more information: https://developer.github.com/v3/repos/projects/
         /// </remarks>
         IObservableProjectsClient Project { get; }
+
+        /// <summary>
+        /// Gets all topics for the specified owner and repository name.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#get-all-repository-topics">API documentation</a> for more details
+        /// </remarks>
+        /// <param name="owner">The owner of the repository</param>
+        /// <param name="name">The name of the repository</param>
+        /// <param name="options">Options for changing the API response</param>
+        /// <returns>All topics associated with the repository.</returns>
+        IObservable<RepositoryTopics> GetAllTopics(string owner, string name, ApiOptions options);
+
+        /// <summary>
+        /// Gets all topics for the specified owner and repository name.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#get-all-repository-topics">API documentation</a> for more details
+        /// </remarks>
+        /// <param name="owner">The owner of the repository</param>
+        /// <param name="name">The name of the repository</param>
+        /// <returns>All topics associated with the repository.</returns>
+        IObservable<RepositoryTopics> GetAllTopics(string owner, string name);
+
+
+
+        /// <summary>
+        /// Gets all topics for the specified repository ID.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#get-all-repository-topics">API documentation</a> for more details
+        /// </remarks>
+        /// <param name="repositoryId">The ID of the repository</param>
+        /// <param name="options">Options for changing the API response</param>
+        /// <returns>All topics associated with the repository.</returns>
+        IObservable<RepositoryTopics> GetAllTopics(long repositoryId, ApiOptions options);
+
+        /// <summary>
+        /// Gets all topics for the specified repository ID.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#get-all-repository-topics">API documentation</a> for more details
+        /// </remarks>
+        /// <param name="repositoryId">The ID of the repository</param>
+        /// <returns>All topics associated with the repository.</returns>
+        IObservable<RepositoryTopics> GetAllTopics(long repositoryId);
+
+        /// <summary>
+        /// Replaces all topics for the specified repository.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#replace-all-repository-topics">API documentation</a> for more details
+        ///
+        /// This is a replacement operation; it is not additive. To clear repository topics, for example, you could specify an empty list of topics here.
+        /// </remarks>
+        /// <param name="repositoryId">The ID of the repository</param>
+        /// <param name="topics">The list of topics to associate with the repository</param>
+        /// <returns>All topics now associated with the repository.</returns>
+        IObservable<RepositoryTopics> ReplaceAllTopics(long repositoryId, RepositoryTopics topics);
+
+        /// <summary>
+        /// Replaces all topics for the specified repository.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#replace-all-repository-topics">API documentation</a> for more details
+        ///
+        /// This is a replacement operation; it is not additive. To clear repository topics, for example, you could specify an empty list of topics here.
+        /// </remarks>
+        /// <param name="owner">The owner of the repository</param>
+        /// <param name="name">The name of the repository</param>
+        /// <param name="topics">The list of topics to associate with the repository</param>
+        /// <returns>All topics now associated with the repository.</returns>
+        IObservable<RepositoryTopics> ReplaceAllTopics(string owner, string name, RepositoryTopics topics);
+
     }
 }

Octokit.Reactive/Clients/ObservableRepositoriesClient.cs

@@ -1,9 +1,11 @@
 using System;
 using System.Collections.Generic;
 using System.Diagnostics.CodeAnalysis;
+using System.Linq;
 using System.Reactive;
 using System.Reactive.Linq;
 using System.Reactive.Threading.Tasks;
+using System.Threading.Tasks;
 using Octokit.Reactive.Clients;
 using Octokit.Reactive.Internal;
 
@@ -821,5 +823,94 @@ public IObservable<CompareResult> Compare(string owner, string name, string @bas
         /// Refer to the API documentation for more information: https://developer.github.com/v3/repos/projects/
         /// </remarks>
         public IObservableProjectsClient Project { get; private set; }
+
+        /// <summary>
+        /// Gets all topics for the specified owner and repository name.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#get-all-repository-topics">API documentation</a> for more details
+        /// </remarks>
+        /// <param name="owner">The owner of the repository</param>
+        /// <param name="name">The name of the repository</param>
+        /// <param name="options">Options for changing the API response</param>
+        /// <returns>All topics associated with the repository.</returns>
+        public IObservable<RepositoryTopics> GetAllTopics(string owner, string name, ApiOptions options)
+        {
+            return _client.GetAllTopics(owner, name, options).ToObservable();
+        }
+
+        /// <summary>
+        /// Gets all topics for the specified owner and repository name.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#get-all-repository-topics">API documentation</a> for more details
+        /// </remarks>
+        /// <param name="owner">The owner of the repository</param>
+        /// <param name="name">The name of the repository</param>
+        /// <returns>All topics associated with the repository.</returns>
+        public IObservable<RepositoryTopics> GetAllTopics(string owner, string name)
+        {
+            return GetAllTopics(owner, name, ApiOptions.None);
+        }
+
+        /// <summary>
+        /// Gets all topics for the specified repository ID.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#get-all-repository-topics">API documentation</a> for more details
+        /// </remarks>
+        /// <param name="repositoryId">The ID of the repository</param>
+        /// <param name="options">Options for changing the API response</param>
+        /// <returns>All topics associated with the repository.</returns>
+        public IObservable<RepositoryTopics> GetAllTopics(long repositoryId, ApiOptions options)
+        {
+            return _client.GetAllTopics(repositoryId, options).ToObservable();
+        }
+
+        /// <summary>
+        /// Gets all topics for the specified repository ID.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#get-all-repository-topics">API documentation</a> for more details
+        /// </remarks>
+        /// <param name="repositoryId">The ID of the repository</param>
+        /// <returns>All topics associated with the repository.</returns>
+        public IObservable<RepositoryTopics> GetAllTopics(long repositoryId)
+        {
+            return GetAllTopics(repositoryId, ApiOptions.None);
+        }
+
+        /// <summary>
+        /// Replaces all topics for the specified repository.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#replace-all-repository-topics">API documentation</a> for more details
+        ///
+        /// This is a replacement operation; it is not additive. To clear repository topics, for example, you could specify an empty list of topics here.
+        /// </remarks>
+        /// <param name="repositoryId">The ID of the repository</param>
+        /// <param name="topics">The list of topics to associate with the repository</param>
+        /// <returns>All topics now associated with the repository.</returns>
+        public IObservable<RepositoryTopics> ReplaceAllTopics(long repositoryId, RepositoryTopics topics)
+        {
+            return _client.ReplaceAllTopics(repositoryId, topics).ToObservable();
+        }
+
+        /// <summary>
+        /// Replaces all topics for the specified repository.
+        /// </summary>
+        /// <remarks>
+        /// See the <a href="https://docs.github.com/en/rest/reference/repos#replace-all-repository-topics">API documentation</a> for more details
+        ///
+        /// This is a replacement operation; it is not additive. To clear repository topics, for example, you could specify an empty list of topics here.
+        /// </remarks>
+        /// <param name="owner">The owner of the repository</param>
+        /// <param name="name">The name of the repository</param>
+        /// <param name="topics">The list of topics to associate with the repository</param>
+        /// <returns>All topics now associated with the repository.</returns>
+        public IObservable<RepositoryTopics> ReplaceAllTopics(string owner, string name, RepositoryTopics topics)
+        {
+            return _client.ReplaceAllTopics(owner, name, topics).ToObservable();
+        }
     }
 }

Octokit.Tests.Integration/Clients/RepositoriesClientTests.cs

@@ -1371,6 +1371,93 @@ public async Task GetsEmptyLanguagesWhenNoneWithRepositoryId()
         }
     }
 
+    public class TheReplaceAllTopicsMethod : IDisposable
+    {
+        readonly IGitHubClient _github = Helper.GetAuthenticatedClient();
+        readonly RepositoryTopics _defaultTopics = new RepositoryTopics(new List<string> { "blog", "ruby", "jekyll" });
+        const string theRepoOwner = "SeanKilleen";
+        const string theRepository = "seankilleen.github.io";
+
+        [Fact]
+        public async Task ClearsTopicsWithAnEmptyList()
+        {
+            var result = await _github.Repository.ReplaceAllTopics(theRepoOwner, theRepository, new RepositoryTopics());
+            Assert.Empty(result.Names);
+
+            var doubleCheck = await _github.Repository.GetAllTopics(theRepoOwner, theRepository);
+            Assert.Empty((doubleCheck.Names));
+        }
+
+        [Fact]
+        public async Task ClearsTopicsWithAnEmptyListWhenUsingRepoId()
+        {
+            var repo = await _github.Repository.Get(theRepoOwner, theRepository);
+            var result = await _github.Repository.ReplaceAllTopics(repo.Id, new RepositoryTopics());
+            Assert.Empty(result.Names);
+
+            var doubleCheck = await _github.Repository.GetAllTopics(theRepoOwner, theRepository);
+            Assert.Empty((doubleCheck.Names));
+        }
+
+        [Fact]
+        public async Task ReplacesTopicsWithAList()
+        {
+            var defaultTopicsList = new RepositoryTopics(_defaultTopics.Names);
+            var result = await _github.Repository.ReplaceAllTopics(theRepoOwner, theRepository, defaultTopicsList);
+
+            Assert.NotEmpty(result.Names);
+            Assert.Contains(result.Names, item => _defaultTopics.Names.Contains(item, StringComparer.InvariantCultureIgnoreCase));
+
+            var doubleCheck = await _github.Repository.GetAllTopics(theRepoOwner, theRepository);
+            Assert.Contains(doubleCheck.Names, item => _defaultTopics.Names.Contains(item, StringComparer.InvariantCultureIgnoreCase));
+        }
+
+        [Fact]
+        public async Task ReplacesTopicsWithAListWhenUsingRepoId()
+        {
+            var defaultTopicsList = new RepositoryTopics(_defaultTopics.Names);
+            var repo = await _github.Repository.Get(theRepoOwner, theRepository);
+            var result = await _github.Repository.ReplaceAllTopics(repo.Id, defaultTopicsList);
+
+            Assert.NotEmpty(result.Names);
+            Assert.Contains(result.Names, item => _defaultTopics.Names.Contains(item, StringComparer.InvariantCultureIgnoreCase));
+
+            var doubleCheck = await _github.Repository.GetAllTopics(theRepoOwner, theRepository);
+            Assert.Contains(doubleCheck.Names, item => _defaultTopics.Names.Contains(item, StringComparer.InvariantCultureIgnoreCase));
+        }
+
+        public void Dispose()
+        {
+            _github.Repository.ReplaceAllTopics(theRepoOwner, theRepository, _defaultTopics).ConfigureAwait(false);
+        }
+    }
+    public class TheGetAllTopicsMethod
+    {
+        [Fact]
+        public async Task GetsTopicsByOwnerAndName()
+        {
+            var github = Helper.GetAnonymousClient();
+            var result = await github.Repository.GetAllTopics("SeanKilleen", "seankilleen.github.io");
+
+            Assert.Contains("blog", result.Names);
+            Assert.Contains("ruby", result.Names);
+            Assert.Contains("jekyll", result.Names);
+        }
+
+        [Fact]
+        public async Task GetsTopicsByRepoID()
+        {
+            var github = Helper.GetAnonymousClient();
+            var repo = await github.Repository.Get("SeanKilleen", "seankilleen.github.io");
+            var result = await github.Repository.GetAllTopics(repo.Id);
+
+            Assert.Contains("blog", result.Names);
+            Assert.Contains("ruby", result.Names);
+            Assert.Contains("jekyll", result.Names);
+        }
+
+    }
+
     public class TheGetAllTagsMethod
     {
         [IntegrationTest]