Pass MainWindow to PullRequestStatusBarManager.Initialize

This will make it more conducive to testing.

Author: jcansdale

src/GitHub.InlineReviews/PullRequestStatusBarPackage.cs

@@ -1,4 +1,5 @@
 using System;
+using System.Windows;
 using System.Threading;
 using System.Runtime.InteropServices;
 using Microsoft.VisualStudio.Shell;
@@ -19,7 +20,7 @@ protected override async Task InitializeAsync(CancellationToken cancellationToke
             var componentModel = (IComponentModel)await GetServiceAsync(typeof(SComponentModel));
             var exportProvider = componentModel.DefaultExportProvider;
             var pullRequestStatusManager = exportProvider.GetExportedValue<IPullRequestStatusBarManager>();
-            pullRequestStatusManager.Initialize();
+            pullRequestStatusManager.Initialize(Application.Current.MainWindow);
         }
     }
 }

src/GitHub.InlineReviews/Services/IPullRequestStatusBarManager.cs

@@ -1,7 +1,9 @@
-namespace GitHub.InlineReviews.Services
+using System.Windows;
+
+namespace GitHub.InlineReviews.Services
 {
     public interface IPullRequestStatusBarManager
     {
-        void Initialize();
+        void Initialize(Window mainWindow);
     }
 }

src/GitHub.InlineReviews/Services/PullRequestStatusBarManager.cs

@@ -25,11 +25,11 @@ public class PullRequestStatusBarManager : IPullRequestStatusBarManager
         const string StatusBarPartName = "PART_SccStatusBarHost";
 
         readonly IVSGitExt gitExt;
-        readonly Window mainWindow;
         readonly Lazy<IPullRequestSessionManager> pullRequestSessionManager;
         readonly IUsageTracker usageTracker;
         readonly IGitHubServiceProvider serviceProvider;
 
+        Window mainWindow;
         bool initialized;
 
         [ImportingConstructor]
@@ -40,11 +40,11 @@ public PullRequestStatusBarManager(IVSGitExt gitExt, Lazy<IPullRequestSessionMan
             this.pullRequestSessionManager = pullRequestSessionManager;
             this.usageTracker = usageTracker;
             this.serviceProvider = serviceProvider;
-            mainWindow = Application.Current.MainWindow;
         }
 
-        public void Initialize()
+        public void Initialize(Window window)
         {
+            mainWindow = window;
             TryInitialize();
             gitExt.ActiveRepositoriesChanged += TryInitialize;
         }