Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Commit 363f32a

Browse files
StanleyGoldmanStanleyGoldman
committed
Adding methods to convert enumerations
1 parent 2603075 commit 363f32a

File tree

2 files changed

+114
-8
lines changed

2 files changed

+114
-8
lines changed

‎src/GitHub.App/Services/PullRequestService.cs‎

Lines changed: 78 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -106,15 +106,15 @@ public async Task<Page<PullRequestListItemModel>> ReadPullRequests(
106106
CheckRuns = suite.CheckRuns(null, null, null, null, null).AllPages(10)
107107
.Select(run => new CheckRunSummaryModel
108108
{
109-
Conclusion = (CheckConclusionState?)run.Conclusion,
110-
Status = (CheckStatusState)run.Status
109+
Conclusion = FromGraphQl(run.Conclusion),
110+
Status = FromGraphQl(run.Status)
111111
}).ToList()
112112
}).ToList(),
113113
Statuses = commit.Commit.Status
114114
.Select(context =>
115115
context.Contexts.Select(statusContext => new StatusSummaryModel
116116
{
117-
State = (StatusState)statusContext.State,
117+
State = FromGraphQl(statusContext.State),
118118
}).ToList()
119119
).SingleOrDefault()
120120
}).ToList().FirstOrDefault(),
@@ -130,7 +130,7 @@ public async Task<Page<PullRequestListItemModel>> ReadPullRequests(
130130
Body = review.Body,
131131
CommentCount = review.Comments(null, null, null, null).TotalCount,
132132
}).ToList(),
133-
State = (PullRequestStateEnum)pr.State,
133+
State = FromGraphQl(pr.State),
134134
Title = pr.Title,
135135
UpdatedAt = pr.UpdatedAt,
136136
}).ToList(),
@@ -934,6 +934,80 @@ static Tuple<string, int> ParseGHfVSConfigKeyValue(string value)
934934
return null;
935935
}
936936

937+
static PullRequestStateEnum FromGraphQl(PullRequestState value)
938+
{
939+
switch (value)
940+
{
941+
case PullRequestState.Open:
942+
return PullRequestStateEnum.Open;
943+
case PullRequestState.Closed:
944+
return PullRequestStateEnum.Closed;
945+
case PullRequestState.Merged:
946+
return PullRequestStateEnum.Merged;
947+
default:
948+
throw new ArgumentOutOfRangeException(nameof(value), value, null);
949+
}
950+
}
951+
952+
static StatusState FromGraphQl(Octokit.GraphQL.Model.StatusState value)
953+
{
954+
switch (value)
955+
{
956+
case Octokit.GraphQL.Model.StatusState.Expected:
957+
return StatusState.Expected;
958+
case Octokit.GraphQL.Model.StatusState.Error:
959+
return StatusState.Error;
960+
case Octokit.GraphQL.Model.StatusState.Failure:
961+
return StatusState.Failure;
962+
case Octokit.GraphQL.Model.StatusState.Pending:
963+
return StatusState.Pending;
964+
case Octokit.GraphQL.Model.StatusState.Success:
965+
return StatusState.Success;
966+
default:
967+
throw new ArgumentOutOfRangeException(nameof(value), value, null);
968+
}
969+
}
970+
971+
static CheckStatusState FromGraphQl(Octokit.GraphQL.Model.CheckStatusState value)
972+
{
973+
switch (value)
974+
{
975+
case Octokit.GraphQL.Model.CheckStatusState.Queued:
976+
return CheckStatusState.Queued;
977+
case Octokit.GraphQL.Model.CheckStatusState.InProgress:
978+
return CheckStatusState.InProgress;
979+
case Octokit.GraphQL.Model.CheckStatusState.Completed:
980+
return CheckStatusState.Completed;
981+
case Octokit.GraphQL.Model.CheckStatusState.Requested:
982+
return CheckStatusState.Requested;
983+
default:
984+
throw new ArgumentOutOfRangeException(nameof(value), value, null);
985+
}
986+
}
987+
988+
static CheckConclusionState? FromGraphQl(Octokit.GraphQL.Model.CheckConclusionState? value)
989+
{
990+
switch (value)
991+
{
992+
case null:
993+
return null;
994+
case Octokit.GraphQL.Model.CheckConclusionState.ActionRequired:
995+
return CheckConclusionState.ActionRequired;
996+
case Octokit.GraphQL.Model.CheckConclusionState.TimedOut:
997+
return CheckConclusionState.TimedOut;
998+
case Octokit.GraphQL.Model.CheckConclusionState.Cancelled:
999+
return CheckConclusionState.Cancelled;
1000+
case Octokit.GraphQL.Model.CheckConclusionState.Failure:
1001+
return CheckConclusionState.Failure;
1002+
case Octokit.GraphQL.Model.CheckConclusionState.Success:
1003+
return CheckConclusionState.Success;
1004+
case Octokit.GraphQL.Model.CheckConclusionState.Neutral:
1005+
return CheckConclusionState.Neutral;
1006+
default:
1007+
throw new ArgumentOutOfRangeException(nameof(value), value, null);
1008+
}
1009+
}
1010+
9371011
class ListItemAdapter : PullRequestListItemModel
9381012
{
9391013
public IList<ReviewAdapter> Reviews { get; set; }

‎src/GitHub.InlineReviews/Services/PullRequestSessionService.cs‎

Lines changed: 36 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
using LibGit2Sharp;
1818
using Microsoft.VisualStudio.Text;
1919
using Microsoft.VisualStudio.Text.Projection;
20+
using Octokit;
2021
using Octokit.GraphQL;
2122
using Octokit.GraphQL.Core;
2223
using Octokit.GraphQL.Model;
@@ -27,6 +28,10 @@
2728
using CheckAnnotationLevel = GitHub.Models.CheckAnnotationLevel;
2829
using CheckConclusionState = GitHub.Models.CheckConclusionState;
2930
using CheckStatusState = GitHub.Models.CheckStatusState;
31+
using DraftPullRequestReviewComment = Octokit.GraphQL.Model.DraftPullRequestReviewComment;
32+
using FileMode = System.IO.FileMode;
33+
using NotFoundException = LibGit2Sharp.NotFoundException;
34+
using PullRequestReviewState = Octokit.GraphQL.Model.PullRequestReviewState;
3035
using StatusState = GitHub.Models.StatusState;
3136

3237
// GraphQL DatabaseId field are marked as deprecated, but we need them for interop with REST.
@@ -295,14 +300,14 @@ public virtual async Task<PullRequestDetailModel> ReadPullRequestDetail(HostAddr
295300
HeadRefName = pr.HeadRefName,
296301
HeadRefSha = pr.HeadRefOid,
297302
HeadRepositoryOwner = pr.HeadRepositoryOwner != null ? pr.HeadRepositoryOwner.Login : null,
298-
State = (PullRequestStateEnum)pr.State,
303+
State = FromGraphQl(pr.State),
299304
UpdatedAt = pr.UpdatedAt,
300305
Reviews = pr.Reviews(null, null, null, null, null, null).AllPages().Select(review => new PullRequestReviewModel
301306
{
302307
Id = review.Id.Value,
303308
Body = review.Body,
304309
CommitId = review.Commit.Oid,
305-
State = (GitHub.Models.PullRequestReviewState)review.State,
310+
State = FromGraphQl(review.State),
306311
SubmittedAt = review.SubmittedAt,
307312
Author = new ActorModel
308313
{
@@ -853,9 +858,36 @@ static void BuildPullRequestThreads(PullRequestDetailModel model)
853858
model.Threads = threads;
854859
}
855860

856-
static GitHub.Models.PullRequestReviewState FromGraphQL(Octokit.GraphQL.Model.PullRequestReviewState s)
861+
static PullRequestStateEnum FromGraphQl(PullRequestState value)
857862
{
858-
return (GitHub.Models.PullRequestReviewState)s;
863+
switch (value) {
864+
case PullRequestState.Open:
865+
return PullRequestStateEnum.Open;
866+
case PullRequestState.Closed:
867+
return PullRequestStateEnum.Closed;
868+
case PullRequestState.Merged:
869+
return PullRequestStateEnum.Merged;
870+
default:
871+
throw new ArgumentOutOfRangeException(nameof(value), value, null);
872+
}
873+
}
874+
875+
static GitHub.Models.PullRequestReviewState FromGraphQl(PullRequestReviewState value)
876+
{
877+
switch (value) {
878+
case PullRequestReviewState.Pending:
879+
return GitHub.Models.PullRequestReviewState.Pending;
880+
case PullRequestReviewState.Commented:
881+
return GitHub.Models.PullRequestReviewState.Commented;
882+
case PullRequestReviewState.Approved:
883+
return GitHub.Models.PullRequestReviewState.Approved;
884+
case PullRequestReviewState.ChangesRequested:
885+
return GitHub.Models.PullRequestReviewState.ChangesRequested;
886+
case PullRequestReviewState.Dismissed:
887+
return GitHub.Models.PullRequestReviewState.Dismissed;
888+
default:
889+
throw new ArgumentOutOfRangeException(nameof(value), value, null);
890+
}
859891
}
860892

861893
static Octokit.GraphQL.Model.PullRequestReviewEvent ToGraphQl(Octokit.PullRequestReviewEvent e)

0 commit comments

Comments
 (0)