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

Commit 5f36b1a

Browse files
committed
Fix failing tests.
Filter now gets set on initialize.
1 parent aa2726b commit 5f36b1a

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

src/UnitTests/GitHub.App/ViewModels/PullRequestListViewModelTests.cs

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,10 @@ public void SelectingAssigneeShouldTriggerFilter()
2323
var prViewModel = new PullRequestListViewModel(repositoryHost, repository, settings);
2424

2525
prViewModel.Initialize(null);
26-
prViewModel.PullRequests.DidNotReceive().Filter = AnyFilter;
26+
prViewModel.PullRequests.Received(1).Filter = AnyFilter;
2727

2828
prViewModel.SelectedAssignee = prViewModel.PullRequests[0].Assignee;
29-
prViewModel.PullRequests.Received(1).Filter = AnyFilter;
29+
prViewModel.PullRequests.Received(2).Filter = AnyFilter;
3030
}
3131

3232
[Fact]
@@ -38,16 +38,16 @@ public void ResettingAssigneeToNoneShouldNotTriggerFilter()
3838
var prViewModel = new PullRequestListViewModel(repositoryHost, repository, settings);
3939

4040
prViewModel.Initialize(null);
41-
prViewModel.PullRequests.DidNotReceive().Filter = AnyFilter;
41+
prViewModel.PullRequests.Received(1).Filter = AnyFilter;
4242

4343
prViewModel.SelectedAssignee = prViewModel.PullRequests[0].Assignee;
44-
prViewModel.PullRequests.Received(1).Filter = AnyFilter;
44+
prViewModel.PullRequests.Received(2).Filter = AnyFilter;
4545

4646
// Setting the Assignee filter to [None] should not trigger a filter:
4747
// doing this will remove the [None] entry from Assignees, which will cause
4848
// the selection in the view to be set to null which will reset the filter.
4949
prViewModel.SelectedAssignee = prViewModel.EmptyUser;
50-
prViewModel.PullRequests.Received(1).Filter = AnyFilter;
50+
prViewModel.PullRequests.Received(2).Filter = AnyFilter;
5151
}
5252

5353
[Fact]
@@ -59,10 +59,10 @@ public void SelectingAuthorShouldTriggerFilter()
5959
var prViewModel = new PullRequestListViewModel(repositoryHost, repository, settings);
6060

6161
prViewModel.Initialize(null);
62-
prViewModel.PullRequests.DidNotReceive().Filter = AnyFilter;
62+
prViewModel.PullRequests.Received(1).Filter = AnyFilter;
6363

6464
prViewModel.SelectedAuthor = prViewModel.PullRequests[0].Author;
65-
prViewModel.PullRequests.Received(1).Filter = AnyFilter;
65+
prViewModel.PullRequests.Received(2).Filter = AnyFilter;
6666
}
6767

6868
[Fact]
@@ -74,16 +74,16 @@ public void ResettingAuthorToNoneShouldNotTriggerFilter()
7474
var prViewModel = new PullRequestListViewModel(repositoryHost, repository, settings);
7575

7676
prViewModel.Initialize(null);
77-
prViewModel.PullRequests.DidNotReceive().Filter = AnyFilter;
77+
prViewModel.PullRequests.Received(1).Filter = AnyFilter;
7878

7979
prViewModel.SelectedAuthor = prViewModel.PullRequests[0].Author;
80-
prViewModel.PullRequests.Received(1).Filter = AnyFilter;
80+
prViewModel.PullRequests.Received(2).Filter = AnyFilter;
8181

8282
// Setting the Author filter to [None] should not trigger a filter:
8383
// doing this will remove the [None] entry from Authors, which will cause
8484
// the selection in the view to be set to null which will reset the filter.
8585
prViewModel.SelectedAuthor = prViewModel.EmptyUser;
86-
prViewModel.PullRequests.Received(1).Filter = AnyFilter;
86+
prViewModel.PullRequests.Received(2).Filter = AnyFilter;
8787
}
8888

8989
Func<IPullRequestModel, int, IList<IPullRequestModel>, bool> AnyFilter =>

0 commit comments

Comments
 (0)