From 397b1f1c821759f10ddaf552c3fe5feb5dde7ac9 Mon Sep 17 00:00:00 2001 From: Prayank Mathur Date: Mon, 14 Mar 2016 23:22:18 +0530 Subject: [PATCH] Updated the enum ItemState with deprecating the ItemState.All and making the necessary changes --- .../Clients/IssuesClientTests.cs | 10 +++++----- .../Clients/IssuesEventsClientTests.cs | 10 +++++----- .../Clients/MilestonesClientTests.cs | 12 ++++++------ .../Clients/PullRequestsClientTests.cs | 10 +++++----- .../Clients/SearchClientTests.cs | 2 +- .../Reactive/ObservableMilestonesClientTests.cs | 2 +- Octokit.Tests.Integration/RedirectTests.cs | 2 +- Octokit.Tests/Clients/SearchClientTests.cs | 4 ++-- Octokit.Tests/Models/MilestoneRequestTests.cs | 2 +- Octokit.Tests/Models/PullRequestRequestTests.cs | 2 +- Octokit/Models/Request/IssueRequest.cs | 2 +- Octokit/Models/Request/IssueUpdate.cs | 2 +- Octokit/Models/Request/MilestoneRequest.cs | 4 ++-- Octokit/Models/Request/MilestoneUpdate.cs | 2 +- Octokit/Models/Request/NewMilestone.cs | 4 ++-- Octokit/Models/Request/PullRequestRequest.cs | 6 +++--- Octokit/Models/Request/PullRequestUpdate.cs | 4 ++-- Octokit/Models/Request/SearchIssuesRequest.cs | 2 +- Octokit/Models/Response/Issue.cs | 2 +- 19 files changed, 42 insertions(+), 42 deletions(-) diff --git a/Octokit.Tests.Integration/Clients/IssuesClientTests.cs b/Octokit.Tests.Integration/Clients/IssuesClientTests.cs index 733f7192..a75bdcf8 100644 --- a/Octokit.Tests.Integration/Clients/IssuesClientTests.cs +++ b/Octokit.Tests.Integration/Clients/IssuesClientTests.cs @@ -55,7 +55,7 @@ public class IssuesClientTests : IDisposable finally { var closed = _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue.Number, - new IssueUpdate { State = ItemStateFilter.Closed }) + new IssueUpdate { State = ItemState.Closed }) .Result; Assert.NotNull(closed); } @@ -75,7 +75,7 @@ public class IssuesClientTests : IDisposable await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue3); var closed = await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue4); await _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, closed.Number, - new IssueUpdate { State = ItemStateFilter.Closed }); + new IssueUpdate { State = ItemState.Closed }); var issues = await _issuesClient.GetAllForRepository(_context.RepositoryOwner, _context.RepositoryName); @@ -99,7 +99,7 @@ public class IssuesClientTests : IDisposable await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue3); var closed = await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue4); await _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, closed.Number, - new IssueUpdate { State = ItemStateFilter.Closed }); + new IssueUpdate { State = ItemState.Closed }); var issues = await _issuesClient.GetAllForRepository(_context.RepositoryOwner, _context.RepositoryName, new RepositoryIssueRequest { SortDirection = SortDirection.Ascending }); @@ -119,7 +119,7 @@ public class IssuesClientTests : IDisposable await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue2); var closed = await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue2); await _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, closed.Number, - new IssueUpdate { State = ItemStateFilter.Closed }); + new IssueUpdate { State = ItemState.Closed }); var issues = await _issuesClient.GetAllForRepository(_context.RepositoryOwner, _context.RepositoryName, new RepositoryIssueRequest { State = ItemStateFilter.Closed }); @@ -156,7 +156,7 @@ public class IssuesClientTests : IDisposable var issue3 = await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue3); var issue4 = await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue4); await _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue4.Number, - new IssueUpdate { State = ItemStateFilter.Closed }); + new IssueUpdate { State = ItemState.Closed }); var retrieved = await _issuesClient.GetAllForRepository(_context.RepositoryOwner, _context.RepositoryName, new RepositoryIssueRequest { }); diff --git a/Octokit.Tests.Integration/Clients/IssuesEventsClientTests.cs b/Octokit.Tests.Integration/Clients/IssuesEventsClientTests.cs index de362c0b..61f9c919 100644 --- a/Octokit.Tests.Integration/Clients/IssuesEventsClientTests.cs +++ b/Octokit.Tests.Integration/Clients/IssuesEventsClientTests.cs @@ -33,7 +33,7 @@ public class IssuesEventsClientTests : IDisposable var issueEventInfo = await _issuesEventsClient.GetAllForIssue(_context.RepositoryOwner, _context.RepositoryName, issue.Number); Assert.Empty(issueEventInfo); - var closed = _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue.Number, new IssueUpdate { State = ItemStateFilter.Closed }) + var closed = _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue.Number, new IssueUpdate { State = ItemState.Closed }) .Result; Assert.NotNull(closed); issueEventInfo = await _issuesEventsClient.GetAllForIssue(_context.RepositoryOwner, _context.RepositoryName, issue.Number); @@ -55,15 +55,15 @@ public class IssuesEventsClientTests : IDisposable Thread.Sleep(1000); // close and open issue1 - var closed1 = _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue1.Number, new IssueUpdate { State = ItemStateFilter.Closed }) + var closed1 = _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue1.Number, new IssueUpdate { State = ItemState.Closed }) .Result; Assert.NotNull(closed1); - var reopened1 = _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue1.Number, new IssueUpdate { State = ItemStateFilter.Open }) + var reopened1 = _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue1.Number, new IssueUpdate { State = ItemState.Open }) .Result; Assert.NotNull(reopened1); // close issue2 - var closed2 = _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue2.Number, new IssueUpdate { State = ItemStateFilter.Closed }) + var closed2 = _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue2.Number, new IssueUpdate { State = ItemState.Closed }) .Result; Assert.NotNull(closed2); @@ -78,7 +78,7 @@ public class IssuesEventsClientTests : IDisposable { var newIssue = new NewIssue("a test issue") { Body = "A new unassigned issue" }; var issue = await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue); - var closed = _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue.Number, new IssueUpdate { State = ItemStateFilter.Closed }) + var closed = _issuesClient.Update(_context.RepositoryOwner, _context.RepositoryName, issue.Number, new IssueUpdate { State = ItemState.Closed }) .Result; Assert.NotNull(closed); var issueEvents = await _issuesEventsClient.GetAllForRepository(_context.RepositoryOwner, _context.RepositoryName); diff --git a/Octokit.Tests.Integration/Clients/MilestonesClientTests.cs b/Octokit.Tests.Integration/Clients/MilestonesClientTests.cs index bd2eaf37..d3325025 100644 --- a/Octokit.Tests.Integration/Clients/MilestonesClientTests.cs +++ b/Octokit.Tests.Integration/Clients/MilestonesClientTests.cs @@ -46,7 +46,7 @@ public class MilestonesClientTests : IDisposable { var milestone1 = new NewMilestone("milestone 1") { DueOn = DateTime.Now }; var milestone2 = new NewMilestone("milestone 2") { DueOn = DateTime.Now.AddDays(1) }; - var milestone3 = new NewMilestone("milestone 3") { DueOn = DateTime.Now.AddDays(3), State = ItemStateFilter.Closed }; + var milestone3 = new NewMilestone("milestone 3") { DueOn = DateTime.Now.AddDays(3), State = ItemState.Closed }; await _milestonesClient.Create(_context.RepositoryOwner, _context.RepositoryName, milestone1); await _milestonesClient.Create(_context.RepositoryOwner, _context.RepositoryName, milestone2); await _milestonesClient.Create(_context.RepositoryOwner, _context.RepositoryName, milestone3); @@ -62,7 +62,7 @@ public class MilestonesClientTests : IDisposable { var milestone1 = new NewMilestone("milestone 1") { DueOn = DateTime.Now }; var milestone2 = new NewMilestone("milestone 2") { DueOn = DateTime.Now.AddDays(1) }; - var milestone3 = new NewMilestone("milestone 3") { DueOn = DateTime.Now.AddDays(3), State = ItemStateFilter.Closed }; + var milestone3 = new NewMilestone("milestone 3") { DueOn = DateTime.Now.AddDays(3), State = ItemState.Closed }; await _milestonesClient.Create(_context.RepositoryOwner, _context.RepositoryName, milestone1); await _milestonesClient.Create(_context.RepositoryOwner, _context.RepositoryName, milestone2); await _milestonesClient.Create(_context.RepositoryOwner, _context.RepositoryName, milestone3); @@ -79,13 +79,13 @@ public class MilestonesClientTests : IDisposable { var milestone1 = new NewMilestone("milestone 1") { DueOn = DateTime.Now }; var milestone2 = new NewMilestone("milestone 2") { DueOn = DateTime.Now.AddDays(1) }; - var milestone3 = new NewMilestone("milestone 3") { DueOn = DateTime.Now.AddDays(3), State = ItemStateFilter.Closed }; + var milestone3 = new NewMilestone("milestone 3") { DueOn = DateTime.Now.AddDays(3), State = ItemState.Closed }; await _milestonesClient.Create(_context.RepositoryOwner, _context.RepositoryName, milestone1); await _milestonesClient.Create(_context.RepositoryOwner, _context.RepositoryName, milestone2); await _milestonesClient.Create(_context.RepositoryOwner, _context.RepositoryName, milestone3); var milestones = await _milestonesClient.GetAllForRepository(_context.RepositoryOwner, _context.RepositoryName, - new MilestoneRequest { State = ItemState.Closed }); + new MilestoneRequest { State = ItemStateFilter.Closed }); Assert.Equal(1, milestones.Count); Assert.Equal("milestone 3", milestones[0].Title); @@ -98,9 +98,9 @@ public class MilestonesClientTests : IDisposable var issue1 = await _github.Issue.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue); var issue2 = await _github.Issue.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue); await _github.Issue.Update(_context.RepositoryOwner, _context.RepositoryName, issue1.Number, - new IssueUpdate { State = ItemStateFilter.Closed }); + new IssueUpdate { State = ItemState.Closed }); await _github.Issue.Update(_context.RepositoryOwner, _context.RepositoryName, issue2.Number, - new IssueUpdate { State = ItemStateFilter.Closed }); + new IssueUpdate { State = ItemState.Closed }); var retrieved = await _github.Issue.GetAllForRepository(_context.RepositoryOwner, _context.RepositoryName, new RepositoryIssueRequest { State = ItemStateFilter.Closed }); diff --git a/Octokit.Tests.Integration/Clients/PullRequestsClientTests.cs b/Octokit.Tests.Integration/Clients/PullRequestsClientTests.cs index 8543469f..c9e0a2eb 100644 --- a/Octokit.Tests.Integration/Clients/PullRequestsClientTests.cs +++ b/Octokit.Tests.Integration/Clients/PullRequestsClientTests.cs @@ -60,7 +60,7 @@ public class PullRequestsClientTests : IDisposable var newPullRequest = new NewPullRequest("a pull request", branchName, "master"); var result = await _fixture.Create(Helper.UserName, _context.RepositoryName, newPullRequest); - var openPullRequests = new PullRequestRequest { State = ItemState.Open }; + var openPullRequests = new PullRequestRequest { State = ItemStateFilter.Open }; var pullRequests = await _fixture.GetAllForRepository(Helper.UserName, _context.RepositoryName, openPullRequests); Assert.Equal(1, pullRequests.Count); @@ -75,7 +75,7 @@ public class PullRequestsClientTests : IDisposable var newPullRequest = new NewPullRequest("a pull request", branchName, "master"); await _fixture.Create(Helper.UserName, _context.RepositoryName, newPullRequest); - var openPullRequests = new PullRequestRequest { State = ItemState.Closed }; + var openPullRequests = new PullRequestRequest { State = ItemStateFilter.Closed }; var pullRequests = await _fixture.GetAllForRepository(Helper.UserName, _context.RepositoryName, openPullRequests); Assert.Empty(pullRequests); @@ -104,7 +104,7 @@ public class PullRequestsClientTests : IDisposable var newPullRequest = new NewPullRequest("a pull request", branchName, "master"); var pullRequest = await _fixture.Create(Helper.UserName, _context.RepositoryName, newPullRequest); - var updatePullRequest = new PullRequestUpdate { State = ItemStateFilter.Closed }; + var updatePullRequest = new PullRequestUpdate { State = ItemState.Closed }; var result = await _fixture.Update(Helper.UserName, _context.RepositoryName, pullRequest.Number, updatePullRequest); Assert.Equal(ItemState.Closed, result.State); @@ -120,10 +120,10 @@ public class PullRequestsClientTests : IDisposable var newPullRequest = new NewPullRequest("a pull request", branchName, "master"); var pullRequest = await _fixture.Create(Helper.UserName, _context.RepositoryName, newPullRequest); - var updatePullRequest = new PullRequestUpdate { State = ItemStateFilter.Closed }; + var updatePullRequest = new PullRequestUpdate { State = ItemState.Closed }; await _fixture.Update(Helper.UserName, _context.RepositoryName, pullRequest.Number, updatePullRequest); - var closedPullRequests = new PullRequestRequest { State = ItemState.Closed }; + var closedPullRequests = new PullRequestRequest { State = ItemStateFilter.Closed }; var pullRequests = await _fixture.GetAllForRepository(Helper.UserName, _context.RepositoryName, closedPullRequests); Assert.Equal(1, pullRequests.Count); diff --git a/Octokit.Tests.Integration/Clients/SearchClientTests.cs b/Octokit.Tests.Integration/Clients/SearchClientTests.cs index cae762dd..3292c390 100644 --- a/Octokit.Tests.Integration/Clients/SearchClientTests.cs +++ b/Octokit.Tests.Integration/Clients/SearchClientTests.cs @@ -79,7 +79,7 @@ public class SearchClientTests { var request = new SearchIssuesRequest("phone"); request.Repos.Add("caliburn-micro", "caliburn.micro"); - request.State = ItemStateFilter.Open; + request.State = ItemState.Open; var issues = await _gitHubClient.Search.SearchIssues(request); diff --git a/Octokit.Tests.Integration/Reactive/ObservableMilestonesClientTests.cs b/Octokit.Tests.Integration/Reactive/ObservableMilestonesClientTests.cs index 91e9acc9..e1560211 100644 --- a/Octokit.Tests.Integration/Reactive/ObservableMilestonesClientTests.cs +++ b/Octokit.Tests.Integration/Reactive/ObservableMilestonesClientTests.cs @@ -31,7 +31,7 @@ namespace Octokit.Tests.Integration var github = Helper.GetAuthenticatedClient(); var client = new ObservableMilestonesClient(github); - var milestones = await client.GetAllForRepository("libgit2", "libgit2sharp", new MilestoneRequest { State = ItemState.Closed }).ToList(); + var milestones = await client.GetAllForRepository("libgit2", "libgit2sharp", new MilestoneRequest { State = ItemStateFilter.Closed }).ToList(); Assert.NotEmpty(milestones); Assert.True(milestones.All(m => m.State == ItemState.Closed)); diff --git a/Octokit.Tests.Integration/RedirectTests.cs b/Octokit.Tests.Integration/RedirectTests.cs index a620c453..b61416c2 100644 --- a/Octokit.Tests.Integration/RedirectTests.cs +++ b/Octokit.Tests.Integration/RedirectTests.cs @@ -38,7 +38,7 @@ namespace Octokit.Tests.Integration Assert.NotNull(resolvedIssue); var update = resolvedIssue.ToUpdate(); - update.State = ItemStateFilter.Closed; + update.State = ItemState.Closed; await client.Issue.Update(owner, oldRepoName, issue.Number, update); } } diff --git a/Octokit.Tests/Clients/SearchClientTests.cs b/Octokit.Tests/Clients/SearchClientTests.cs index ba79d5ea..4d5125d5 100644 --- a/Octokit.Tests/Clients/SearchClientTests.cs +++ b/Octokit.Tests/Clients/SearchClientTests.cs @@ -902,7 +902,7 @@ namespace Octokit.Tests.Clients var connection = Substitute.For(); var client = new SearchClient(connection); var request = new SearchIssuesRequest("something"); - request.State = ItemStateFilter.Open; + request.State = ItemState.Open; client.SearchIssues(request); @@ -917,7 +917,7 @@ namespace Octokit.Tests.Clients var connection = Substitute.For(); var client = new SearchClient(connection); var request = new SearchIssuesRequest("something"); - request.State = ItemStateFilter.Closed; + request.State = ItemState.Closed; client.SearchIssues(request); diff --git a/Octokit.Tests/Models/MilestoneRequestTests.cs b/Octokit.Tests/Models/MilestoneRequestTests.cs index d489d6c9..6964e761 100644 --- a/Octokit.Tests/Models/MilestoneRequestTests.cs +++ b/Octokit.Tests/Models/MilestoneRequestTests.cs @@ -10,7 +10,7 @@ public class MilestoneRequestTests { var request = new MilestoneRequest { - State = ItemState.Closed, + State = ItemStateFilter.Closed, SortProperty = MilestoneSort.Completeness, SortDirection = SortDirection.Descending, }; diff --git a/Octokit.Tests/Models/PullRequestRequestTests.cs b/Octokit.Tests/Models/PullRequestRequestTests.cs index 51fb7714..1c1810e6 100644 --- a/Octokit.Tests/Models/PullRequestRequestTests.cs +++ b/Octokit.Tests/Models/PullRequestRequestTests.cs @@ -10,7 +10,7 @@ public class PullRequestRequestTests { var request = new PullRequestRequest { - State = ItemState.Closed, + State = ItemStateFilter.Closed, Head = "user:ref-name", Base = "fake_base_branch" }; diff --git a/Octokit/Models/Request/IssueRequest.cs b/Octokit/Models/Request/IssueRequest.cs index e5d312aa..977740b0 100644 --- a/Octokit/Models/Request/IssueRequest.cs +++ b/Octokit/Models/Request/IssueRequest.cs @@ -33,7 +33,7 @@ namespace Octokit public IssueFilter Filter { get; set; } /// - /// Gets or sets the for the issues to return. + /// Gets or sets the for the issues to return. /// /// /// The state. diff --git a/Octokit/Models/Request/IssueUpdate.cs b/Octokit/Models/Request/IssueUpdate.cs index 89d18621..8d6b44bc 100644 --- a/Octokit/Models/Request/IssueUpdate.cs +++ b/Octokit/Models/Request/IssueUpdate.cs @@ -52,7 +52,7 @@ namespace Octokit /// /// Whether the issue is open or closed. /// - public ItemStateFilter? State { get; set; } + public ItemState? State { get; set; } internal string DebuggerDisplay { diff --git a/Octokit/Models/Request/MilestoneRequest.cs b/Octokit/Models/Request/MilestoneRequest.cs index c39a014a..bc181509 100644 --- a/Octokit/Models/Request/MilestoneRequest.cs +++ b/Octokit/Models/Request/MilestoneRequest.cs @@ -13,12 +13,12 @@ namespace Octokit { public MilestoneRequest() { - State = ItemState.Open; + State = ItemStateFilter.Open; SortProperty = MilestoneSort.DueDate; SortDirection = SortDirection.Ascending; } - public ItemState State { get; set; } + public ItemStateFilter State { get; set; } [Parameter(Key = "sort")] public MilestoneSort SortProperty { get; set; } diff --git a/Octokit/Models/Request/MilestoneUpdate.cs b/Octokit/Models/Request/MilestoneUpdate.cs index 36002d59..80fac3b1 100644 --- a/Octokit/Models/Request/MilestoneUpdate.cs +++ b/Octokit/Models/Request/MilestoneUpdate.cs @@ -18,7 +18,7 @@ namespace Octokit /// /// Whether the milestone is open or closed. The default is . /// - public ItemStateFilter? State { get; set; } + public ItemState? State { get; set; } /// /// Optional description for the milestone. diff --git a/Octokit/Models/Request/NewMilestone.cs b/Octokit/Models/Request/NewMilestone.cs index 34128a3c..4817fa3a 100644 --- a/Octokit/Models/Request/NewMilestone.cs +++ b/Octokit/Models/Request/NewMilestone.cs @@ -19,7 +19,7 @@ namespace Octokit Ensure.ArgumentNotNull(title, "title"); Title = title; - State = ItemStateFilter.Open; + State = ItemState.Open; } /// @@ -30,7 +30,7 @@ namespace Octokit /// /// Whether the milestone is open or closed. The default is . /// - public ItemStateFilter State { get; set; } + public ItemState State { get; set; } /// /// Optional description for the milestone. diff --git a/Octokit/Models/Request/PullRequestRequest.cs b/Octokit/Models/Request/PullRequestRequest.cs index b5a28250..e8d2e2fb 100644 --- a/Octokit/Models/Request/PullRequestRequest.cs +++ b/Octokit/Models/Request/PullRequestRequest.cs @@ -13,15 +13,15 @@ namespace Octokit { public PullRequestRequest() { - State = ItemState.Open; + State = ItemStateFilter.Open; SortProperty = PullRequestSort.Created; SortDirection = SortDirection.Descending; } /// - /// "open" or "closed" to filter by state. Default is "open". + /// "open" , "closed" or "All" to filter by state. Default is "open". /// - public ItemState State { get; set; } + public ItemStateFilter State { get; set; } /// /// Filter pulls by head user and branch name in the format of "user:ref-name". diff --git a/Octokit/Models/Request/PullRequestUpdate.cs b/Octokit/Models/Request/PullRequestUpdate.cs index e28b6cc9..32400553 100644 --- a/Octokit/Models/Request/PullRequestUpdate.cs +++ b/Octokit/Models/Request/PullRequestUpdate.cs @@ -16,9 +16,9 @@ namespace Octokit public string Title { get; set; } /// - /// Whether the pull request is open or closed. The default is . + /// Whether the pull request is open or closed. The default is . /// - public ItemStateFilter? State { get; set; } + public ItemState? State { get; set; } /// /// The body for the pull request. Supports GFM. diff --git a/Octokit/Models/Request/SearchIssuesRequest.cs b/Octokit/Models/Request/SearchIssuesRequest.cs index f5f589d1..77057c2f 100644 --- a/Octokit/Models/Request/SearchIssuesRequest.cs +++ b/Octokit/Models/Request/SearchIssuesRequest.cs @@ -132,7 +132,7 @@ namespace Octokit /// /// https://help.github.com/articles/searching-issues#state /// - public ItemStateFilter? State { get; set; } + public ItemState? State { get; set; } private IEnumerable _labels; /// diff --git a/Octokit/Models/Response/Issue.cs b/Octokit/Models/Response/Issue.cs index 9905e8bd..55c5edd3 100644 --- a/Octokit/Models/Response/Issue.cs +++ b/Octokit/Models/Response/Issue.cs @@ -154,7 +154,7 @@ namespace Octokit Assignee = assignee, Body = Body, Milestone = milestoneId, - State = FilteredState, + State = State, Title = Title };