Merge pull request #1212 from SamTheDev/PaginationCommitStatusClient

Add ApiOption overloads to methods on ICommitStatusClient
2026-06-04 03:16:11 +00:00 · 2016-04-03 15:45:12 -04:00
parent 9231286fed 8a559cca6f
commit c97917b9ae
9 changed files with 182 additions and 5 deletions
@@ -0,0 +1,85 @@
+using System.Reactive.Linq;
+using System.Threading.Tasks;
+using Octokit.Reactive;
+using Xunit;
+
+namespace Octokit.Tests.Integration.Reactive
+{
+    public class ObservableCommitStatusClientTests
+    {
+        public class TheGetAllMethod
+        {
+            readonly ObservableCommitStatusClient _commitStatusClient;
+            const string owner = "octokit";
+            const string name = "octokit.net";
+            const string reference = "1335f37";   
+
+            public TheGetAllMethod()
+            {
+                var github = Helper.GetAuthenticatedClient();
+                _commitStatusClient = new ObservableCommitStatusClient(github);
+            }
+
+            [IntegrationTest]
+            public async Task ReturnCommitStatus()
+            {
+                var commitStatus = await _commitStatusClient.GetAll(owner, name, reference).ToList();
+
+                Assert.NotEmpty(commitStatus);
+            }
+
+            [IntegrationTest]
+            public async Task ReturnsCorrectCountOfCommitStatusWithoutStart()
+            {
+                var options = new ApiOptions
+                {
+                    PageSize = 2,
+                    PageCount = 1
+                };
+
+                var commitStatus = await _commitStatusClient.GetAll(owner, name ,reference , options).ToList();
+
+                Assert.Equal(2, commitStatus.Count);
+            }
+
+            [IntegrationTest]
+            public async Task ReturnsCorrectCountOfCommitStatusWithStart()
+            {
+                var options = new ApiOptions
+                {
+                    PageSize = 2,
+                    PageCount = 1,
+                    StartPage = 1
+                };
+
+                var commitStatus = await _commitStatusClient.GetAll(owner, name, reference, options).ToList();
+
+                Assert.Equal(2, commitStatus.Count);
+            }
+
+            [IntegrationTest]
+            public async Task ReturnsDistinctResultsBasedOnStartPage()
+            {
+                var startOptions = new ApiOptions
+                {
+                    PageSize = 2,
+                    PageCount = 1
+                };
+
+                var firstPage = await _commitStatusClient.GetAll(owner, name, reference, startOptions).ToList();
+
+                var skipStartOptions = new ApiOptions
+                {
+                    PageSize = 2,
+                    PageCount = 1,
+                    StartPage = 2
+                };
+
+                var secondPage = await _commitStatusClient.GetAll(owner, name, reference,skipStartOptions).ToList();             
+
+                Assert.NotEqual(firstPage[0].Id, secondPage[0].Id);
+                Assert.NotEqual(firstPage[1].Id, secondPage[1].Id);                
+            }
+        }
+    }
+}