Use Array.Empty in tests.

Author: grokys

src/GitHub.App/SampleData/PullRequestDetailViewModelDesigner.cs

@@ -96,7 +96,7 @@ public PullRequestDetailViewModelDesigner()
 
             Files = new PullRequestFilesViewModelDesigner();
 
-            Checks = new PullRequestCheckViewModelDesigner[0];
+            Checks = Array.Empty<PullRequestCheckViewModelDesigner>();
         }
 
         public PullRequestDetailModel Model { get; }

src/GitHub.App/SampleData/PullRequestUserReviewsViewModelDesigner.cs

@@ -27,7 +27,7 @@ public PullRequestUserReviewsViewModelDesigner()
                 {
                     IsExpanded = true,
                     HasDetails = true,
-                    FileComments = new IPullRequestReviewFileCommentViewModel[0],
+                    FileComments = Array.Empty<IPullRequestReviewFileCommentViewModel>(),
                     StateDisplay = "approved",
                     Model = new PullRequestReviewModel
                     {

src/GitHub.App/Services/ModelService.cs

@@ -320,7 +320,7 @@ IObservable<IReadOnlyList<IRemoteRepositoryModel>> GetUserRepositories(Repositor
                         log.Error(e,
                             "Retrieving {RepositoryType} user repositories failed because user is not stored in the cache",
                             repositoryType);
-                        return Observable.Return(new IRemoteRepositoryModel[] {});
+                        return Observable.Return(Array.Empty<IRemoteRepositoryModel>());
                     });
         }
 
@@ -355,7 +355,7 @@ IObservable<IReadOnlyList<IRemoteRepositoryModel>> GetOrganizationRepositories(s
                     {
                         log.Error(e, "Retrieveing {Organization} org repositories failed because user is not stored in the cache",
                             organization);
-                        return Observable.Return(new IRemoteRepositoryModel[] {});
+                        return Observable.Return(Array.Empty<IRemoteRepositoryModel>());
                     });
         }
 

src/GitHub.App/Services/StandardUserErrors.cs

@@ -189,7 +189,7 @@ public static IObservable<RecoveryOptionResult> ShowUserErrorThatRequiresNavigat
 
             return exception.DisplayErrorMessage(
                 errorType,
-                new object[] { },
+                Array.Empty<object>(),
                 new[] { OpenBrowser("View Plans", account.Billing()), Cancel });
         }
 

src/GitHub.App/ViewModels/Dialog/RepositoryCreationViewModel.cs

@@ -182,7 +182,7 @@ public async Task InitializeAsync(IConnection connection)
 
             accounts = modelService.GetAccounts()
                 .ObserveOn(RxApp.MainThreadScheduler)
-                .ToProperty(this, vm => vm.Accounts, initialValue: new ReadOnlyCollection<IAccount>(new IAccount[] { }));
+                .ToProperty(this, vm => vm.Accounts, initialValue: new ReadOnlyCollection<IAccount>(Array.Empty<IAccount>()));
 
             this.WhenAny(x => x.Accounts, x => x.Value)
                 .Select(accts => accts?.FirstOrDefault())

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

@@ -51,7 +51,7 @@ public static IEnumerable<IPullRequestCheckViewModel> Build(IViewViewModelFactor
                 pullRequestCheckViewModel.DetailsUrl = !string.IsNullOrEmpty(model.TargetUrl) ? new Uri(model.TargetUrl) : null;
 
                 return pullRequestCheckViewModel;
-            }) ?? new PullRequestCheckViewModel[0];
+            }) ?? Array.Empty<PullRequestCheckViewModel>();
 
             var checks = pullRequest.CheckSuites?.SelectMany(model => model.CheckRuns)
                 .Select(model =>
@@ -97,7 +97,7 @@ public static IEnumerable<IPullRequestCheckViewModel> Build(IViewViewModelFactor
                     pullRequestCheckViewModel.DetailsUrl = new Uri(model.DetailsUrl);
 
                     return pullRequestCheckViewModel;
-                }) ?? new PullRequestCheckViewModel[0];
+                }) ?? Array.Empty<PullRequestCheckViewModel>();
 
             return statuses.Concat(checks).OrderBy(model => model.Title);
         }

src/GitHub.App/ViewModels/TeamExplorer/RepositoryPublishViewModel.cs

@@ -72,7 +72,7 @@ public RepositoryPublishViewModel(
                 .SelectMany(async c => (await modelServiceFactory.CreateAsync(c)).GetAccounts())
                 .Switch()
                 .ObserveOn(RxApp.MainThreadScheduler)
-                .ToProperty(this, x => x.Accounts, initialValue: new ReadOnlyCollection<IAccount>(new IAccount[] {}));
+                .ToProperty(this, x => x.Accounts, initialValue: new ReadOnlyCollection<IAccount>(Array.Empty<IAccount>()));
 
             this.WhenAny(x => x.Accounts, x => x.Value)
                 .WhereNotNull()

src/GitHub.Exports.Reactive/Collections/TrackingCollection.cs

@@ -52,7 +52,7 @@ public static ObservableCollection<T> CreateListenerCollection<T>(this ITracking
         {
             if (stickieItemsOnTop == null)
             {
-                stickieItemsOnTop = new T[0];
+                stickieItemsOnTop = Array.Empty<T>();
             }
 
             var col = new ObservableCollection<T>(stickieItemsOnTop.Concat(tcol));

test/GitHub.App.UnitTests/Models/AccountModelTests.cs

@@ -120,7 +120,7 @@ public static bool BitmapSourcesAreEqual(BitmapSource image1, BitmapSource image
 
         public static byte[] BitmapSourceToBytes(BitmapSource image)
         {
-            byte[] data = new byte[] { };
+            byte[] data = Array.Empty<byte>();
             if (image != null)
             {
                 try

test/GitHub.App.UnitTests/Services/TeamExplorerContextTests.cs

@@ -245,7 +245,7 @@ static IVSGitExt CreateGitExt()
 
         static void SetActiveRepository(IVSGitExt gitExt, ILocalRepositoryModel repo)
         {
-            var repos = repo != null ? new[] { repo } : new ILocalRepositoryModel[0];
+            var repos = repo != null ? new[] { repo } : Array.Empty<ILocalRepositoryModel>();
             gitExt.ActiveRepositories.Returns(repos);
             gitExt.ActiveRepositoriesChanged += Raise.Event<Action>();
         }