Pull Requests w/ Statuses

src/GitHub.App/GitHub.App.csproj

@@ -257,6 +257,7 @@
     <Compile Include="ViewModels\GitHubPane\LoggedOutViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\NavigationViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\GitHubPaneViewModel.cs" />
+    <Compile Include="SampleData\PullRequestCheckViewModelDesigner.cs" />
     <Compile Include="ViewModels\GitHubPane\PullRequestFilesViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\PullRequestListItemViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\PullRequestListViewModel.cs" />
@@ -267,6 +268,7 @@
     <Compile Include="ViewModels\GitHubPane\NotAGitRepositoryViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\PullRequestReviewAuthoringViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\PullRequestReviewCommentViewModel.cs" />
+    <Compile Include="ViewModels\GitHubPane\PullRequestCheckViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\PullRequestReviewSummaryViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\PullRequestReviewViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\PullRequestUserReviewsViewModel.cs" />

src/GitHub.App/SampleData/PullRequestCheckViewModelDesigner.cs

@@ -0,0 +1,25 @@
+using System;
+using System.Windows.Media.Imaging;
+using GitHub.ViewModels;
+using GitHub.ViewModels.GitHubPane;
+using ReactiveUI;
+
+namespace GitHub.SampleData
+{
+    public sealed class PullRequestCheckViewModelDesigner : ViewModelBase, IPullRequestCheckViewModel
+    {
+        public string Title { get; set; } = "continuous-integration/appveyor/pr";
+
+        public string Description { get; set; } = "AppVeyor build failed";
+
+        public PullRequestCheckStatus Status { get; set; } = PullRequestCheckStatus.Failure;
+
+        public Uri DetailsUrl { get; set; } = new Uri("http://github.com");
+
+        public string AvatarUrl { get; set; } = "https://avatars1.githubusercontent.com/u/417571?s=88&v=4";
+
+        public BitmapImage Avatar { get; set; } = null;
+
+        public ReactiveCommand<object> OpenDetailsUrl { get; set; } = null;
+    }
+}

src/GitHub.App/SampleData/PullRequestDetailViewModelDesigner.cs

@@ -1,14 +1,14 @@
-using System;
-using System.Collections.Generic;
-using System.Diagnostics.CodeAnalysis;
-using System.Reactive;
-using System.Text;
-using System.Threading.Tasks;
 using GitHub.Models;
 using GitHub.Services;
 using GitHub.ViewModels;
 using GitHub.ViewModels.GitHubPane;
 using ReactiveUI;
+using System;
+using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
+using System.Reactive;
+using System.Threading.Tasks;
+using GitHub.SampleData;
 
 namespace GitHub.SampleData
 {
@@ -95,6 +95,8 @@ public PullRequestDetailViewModelDesigner()
             };
 
             Files = new PullRequestFilesViewModelDesigner();
+
+            Checks = new PullRequestCheckViewModelDesigner[0];
         }
 
         public PullRequestDetailModel Model { get; }
@@ -123,6 +125,8 @@ public PullRequestDetailViewModelDesigner()
         public ReactiveCommand<object> OpenOnGitHub { get; }
         public ReactiveCommand<object> ShowReview { get; }
 
+        public IReadOnlyList<IPullRequestCheckViewModel> Checks { get; }
+
         public Task InitializeAsync(ILocalRepositoryModel localRepository, IConnection connection, string owner, string repo, int number) => Task.CompletedTask;
 
         public string GetLocalFilePath(IPullRequestFileNode file)

src/GitHub.App/SampleData/PullRequestListItemViewModelDesigner.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.Diagnostics.CodeAnalysis;
+using GitHub.Models;
 using GitHub.ViewModels;
 using GitHub.ViewModels.GitHubPane;
 
@@ -16,5 +17,6 @@ public class PullRequestListItemViewModelDesigner : ViewModelBase, IPullRequestL
         public int Number { get; set; }
         public string Title { get; set; }
         public DateTimeOffset UpdatedAt { get; set; }
+        public PullRequestChecksState Checks { get; set; }
     }
 }

src/GitHub.App/Services/PullRequestService.cs

@@ -22,6 +22,7 @@
 using Rothko;
 using static System.FormattableString;
 using static Octokit.GraphQL.Variable;
+using StatusState = GitHub.Models.StatusState;
 
 namespace GitHub.Services
 {
@@ -93,6 +94,17 @@ public async Task<Page<PullRequestListItemModel>> ReadPullRequests(
                         Items = page.Nodes.Select(pr => new ListItemAdapter
                         {
                             Id = pr.Id.Value,
+                            LastCommit = pr.Commits(null, null, 1, null).Nodes.Select(commit =>
+                                new LastCommitSummaryModel
+                                {
+                                    Statuses = commit.Commit.Status
+                                            .Select(context =>
+                                                context.Contexts.Select(statusContext => new StatusSummaryModel
+                                                {
+                                                    State = (StatusState)statusContext.State,
+                                                }).ToList()
+                                            ).SingleOrDefault()
+                                }).ToList().FirstOrDefault(),
                             Author = new ActorModel
                             {
                                 Login = pr.Author.Login,
@@ -123,10 +135,46 @@ public async Task<Page<PullRequestListItemModel>> ReadPullRequests(
 
             var result = await graphql.Run(readPullRequests, vars);
 
-            foreach (ListItemAdapter item in result.Items)
+            foreach (var item in result.Items.Cast<ListItemAdapter>())
             {
                 item.CommentCount += item.Reviews.Sum(x => x.Count);
                 item.Reviews = null;
+
+                var hasStatuses = item.LastCommit.Statuses != null 
+                                  && item.LastCommit.Statuses.Any();
+
+                if (!hasStatuses)
+                {
+                    item.Checks = PullRequestChecksState.None;
+                }
+                else
+                {
+                    var statusHasFailure = item.LastCommit
+                        .Statuses
+                        .Any(status => status.State == StatusState.Failure);
+
+                    var statusHasCompleteSuccess = true;
+                    if (!statusHasFailure)
+                    {
+                        statusHasCompleteSuccess =
+                            item.LastCommit.Statuses.All(status => status.State == StatusState.Success);
+                    }
+
+                    if (statusHasFailure)
+                    {
+                        item.Checks = PullRequestChecksState.Failure;
+                    }
+                    else if (statusHasCompleteSuccess)
+                    {
+                        item.Checks = PullRequestChecksState.Success;
+                    }
+                    else
+                    {
+                        item.Checks = PullRequestChecksState.Pending;
+                    }
+                }
+
+                item.LastCommit = null;
             }
 
             return result;
@@ -840,6 +888,8 @@ static Tuple<string, int> ParseGHfVSConfigKeyValue(string value)
         class ListItemAdapter : PullRequestListItemModel
         {
             public IList<ReviewAdapter> Reviews { get; set; }
+
+            public LastCommitSummaryModel LastCommit { get; set; }
         }
 
         class ReviewAdapter
@@ -848,5 +898,15 @@ class ReviewAdapter
             public int CommentCount { get; set; }
             public int Count => CommentCount + (!string.IsNullOrWhiteSpace(Body) ? 1 : 0);
         }
+
+        class StatusSummaryModel
+        {
+            public StatusState State { get; set; }
+        }
+
+        class LastCommitSummaryModel
+        {
+            public List<StatusSummaryModel> Statuses { get; set; }
+        }
     }
 }

src/GitHub.App/ViewModels/GitHubPane/PullRequestCheckViewModel.cs

@@ -0,0 +1,86 @@
+using System;
+using System.Collections.Generic;
+using System.ComponentModel.Composition;
+using System.Linq;
+using System.Reactive;
+using System.Reactive.Linq;
+using System.Windows.Media.Imaging;
+using GitHub.Extensions;
+using GitHub.Factories;
+using GitHub.Models;
+using GitHub.Services;
+using ReactiveUI;
+
+namespace GitHub.ViewModels.GitHubPane
+{
+    [Export(typeof(IPullRequestCheckViewModel))]
+    [PartCreationPolicy(CreationPolicy.NonShared)]
+    public class PullRequestCheckViewModel: ViewModelBase, IPullRequestCheckViewModel
+    {
+        private readonly IUsageTracker usageTracker;
+        const string DefaultAvatar = "pack://application:,,,/GitHub.App;component/Images/default_user_avatar.png";
+
+        public static IEnumerable<IPullRequestCheckViewModel> Build(IViewViewModelFactory viewViewModelFactory, PullRequestDetailModel pullRequest)
+        {
+            return pullRequest.Statuses?.Select(model =>
+            {
+                PullRequestCheckStatus checkStatus;
+                switch (model.State)
+                {
+                    case StatusState.Expected:
+                    case StatusState.Error:
+                    case StatusState.Failure:
+                        checkStatus = PullRequestCheckStatus.Failure;
+                        break;
+                    case StatusState.Pending:
+                        checkStatus = PullRequestCheckStatus.Pending;
+                        break;
+                    case StatusState.Success:
+                        checkStatus = PullRequestCheckStatus.Success;
+                        break;
+                    default:
+                        throw new InvalidOperationException("Unkown PullRequestCheckStatusEnum");
+                }
+
+                var pullRequestCheckViewModel = (PullRequestCheckViewModel) viewViewModelFactory.CreateViewModel<IPullRequestCheckViewModel>();
+                pullRequestCheckViewModel.Title = model.Context;
+                pullRequestCheckViewModel.Description = model.Description;
+                pullRequestCheckViewModel.Status = checkStatus;
+                pullRequestCheckViewModel.DetailsUrl = new Uri(model.TargetUrl);
+                pullRequestCheckViewModel.AvatarUrl = model.AvatarUrl ?? DefaultAvatar;
+                pullRequestCheckViewModel.Avatar = model.AvatarUrl != null
+                    ? new BitmapImage(new Uri(model.AvatarUrl))
+                    : AvatarProvider.CreateBitmapImage(DefaultAvatar);
+
+                return pullRequestCheckViewModel;
+
+            }) ?? new PullRequestCheckViewModel[0];
+        }
+
+        [ImportingConstructor]
+        public PullRequestCheckViewModel(IUsageTracker usageTracker)
+        {
+            this.usageTracker = usageTracker;
+            OpenDetailsUrl = ReactiveCommand.Create().OnExecuteCompleted(DoOpenDetailsUrl);
+        }
+
+        private void DoOpenDetailsUrl(object obj)
+        {
+            usageTracker.IncrementCounter(x => x.NumberOfPRCheckStatusesOpenInGitHub).Forget();
+        }
+
+        public string Title { get; private set; }
+
+        public string Description { get; private set; }
+
+        public PullRequestCheckStatus Status{ get; private set; }
+
+        public Uri DetailsUrl { get; private set; }
+
+        public string AvatarUrl { get; private set; }
+
+        public BitmapImage Avatar { get; private set; }
+
+        public ReactiveCommand<object> OpenDetailsUrl { get; }
+    }
+}

src/GitHub.App/ViewModels/GitHubPane/PullRequestDetailViewModel.cs

@@ -38,6 +38,7 @@ public sealed class PullRequestDetailViewModel : PanePageViewModelBase, IPullReq
         readonly IUsageTracker usageTracker;
         readonly ITeamExplorerContext teamExplorerContext;
         readonly ISyncSubmodulesCommand syncSubmodulesCommand;
+        readonly IViewViewModelFactory viewViewModelFactory;
         IModelService modelService;
         PullRequestDetailModel model;
         IActorViewModel author;
@@ -55,6 +56,7 @@ public sealed class PullRequestDetailViewModel : PanePageViewModelBase, IPullReq
         bool refreshOnActivate;
         Uri webUrl;
         IDisposable sessionSubscription;
+        IReadOnlyList<IPullRequestCheckViewModel> checks;
 
         /// <summary>
         /// Initializes a new instance of the <see cref="PullRequestDetailViewModel"/> class.
@@ -73,21 +75,24 @@ public PullRequestDetailViewModel(
             IUsageTracker usageTracker,
             ITeamExplorerContext teamExplorerContext,
             IPullRequestFilesViewModel files,
-            ISyncSubmodulesCommand syncSubmodulesCommand)
+            ISyncSubmodulesCommand syncSubmodulesCommand,
+            IViewViewModelFactory viewViewModelFactory)
         {
             Guard.ArgumentNotNull(pullRequestsService, nameof(pullRequestsService));
             Guard.ArgumentNotNull(sessionManager, nameof(sessionManager));
             Guard.ArgumentNotNull(modelServiceFactory, nameof(modelServiceFactory));
             Guard.ArgumentNotNull(usageTracker, nameof(usageTracker));
             Guard.ArgumentNotNull(teamExplorerContext, nameof(teamExplorerContext));
             Guard.ArgumentNotNull(syncSubmodulesCommand, nameof(syncSubmodulesCommand));
+            Guard.ArgumentNotNull(viewViewModelFactory, nameof(viewViewModelFactory));
 
             this.pullRequestsService = pullRequestsService;
             this.sessionManager = sessionManager;
             this.modelServiceFactory = modelServiceFactory;
             this.usageTracker = usageTracker;
             this.teamExplorerContext = teamExplorerContext;
             this.syncSubmodulesCommand = syncSubmodulesCommand;
+            this.viewViewModelFactory = viewViewModelFactory;
             Files = files;
 
             Checkout = ReactiveCommand.CreateAsyncObservable(
@@ -302,6 +307,12 @@ public Uri WebUrl
         /// </summary>
         public ReactiveCommand<object> ShowReview { get; }
 
+        public IReadOnlyList<IPullRequestCheckViewModel> Checks
+        {
+            get { return checks; }
+            private set { this.RaiseAndSetIfChanged(ref checks, value); }
+        }
+
         /// <summary>
         /// Initializes the view model.
         /// </summary>
@@ -377,6 +388,8 @@ public async Task Load(PullRequestDetailModel pullRequest)
                 Body = !string.IsNullOrWhiteSpace(pullRequest.Body) ? pullRequest.Body : Resources.NoDescriptionProvidedMarkdown;
                 Reviews = PullRequestReviewSummaryViewModel.BuildByUser(Session.User, pullRequest).ToList();
 
+                Checks = PullRequestCheckViewModel.Build(viewViewModelFactory, pullRequest)?.ToList();
+
                 await Files.InitializeAsync(Session);
 
                 var localBranches = await pullRequestsService.GetLocalBranches(LocalRepository, pullRequest).ToList();

src/GitHub.App/ViewModels/GitHubPane/PullRequestListItemViewModel.cs

@@ -19,20 +19,24 @@ public PullRequestListItemViewModel(PullRequestListItemModel model)
         {
             Id = model.Id;
             Author = new ActorViewModel(model.Author);
+            Checks = model.Checks;
             CommentCount = model.CommentCount;
             Number = model.Number;
             Title = model.Title;
             UpdatedAt = model.UpdatedAt;
         }
 
         /// <inheritdoc/>
-        public string Id { get; protected set; }
+        public string Id { get; }
 
         /// <inheritdoc/>
-        public IActorViewModel Author { get; protected set; }
+        public IActorViewModel Author { get; }
 
         /// <inheritdoc/>
-        public int CommentCount { get; protected set; }
+        public PullRequestChecksState Checks { get; }
+
+        /// <inheritdoc/>
+        public int CommentCount { get; }
 
         /// <inheritdoc/>
         public bool IsCurrent
@@ -42,12 +46,12 @@ public bool IsCurrent
         }
 
         /// <inheritdoc/>
-        public int Number { get; protected set; }
+        public int Number { get; }
 
         /// <inheritdoc/>
-        public string Title { get; protected set; }
+        public string Title { get; }
 
         /// <inheritdoc/>
-        public DateTimeOffset UpdatedAt { get; protected set; }
+        public DateTimeOffset UpdatedAt { get; }
     }
 }

src/GitHub.Exports.Reactive/GitHub.Exports.Reactive.csproj

@@ -204,6 +204,7 @@
     <Compile Include="ViewModels\GitHubPane\ILoggedOutViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\INavigationViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\IPanePageViewModel.cs" />
+    <Compile Include="ViewModels\GitHubPane\IPullRequestCheckViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\IPullRequestFilesViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\IPullRequestListItemViewModel.cs" />
     <Compile Include="ViewModels\GitHubPane\IPullRequestListViewModel.cs" />