Skip to content

Commit 94439e0

Browse files
committed
Get things working after adding the sort functionality.
1 parent 622b83d commit 94439e0

File tree

2 files changed

+7
-8
lines changed

2 files changed

+7
-8
lines changed

RetailCoder.VBE/UI/Settings/TodoListSettingsUserControl.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ public partial class TodoListSettingsUserControl : UserControl, ITodoSettingsVie
1313
private GridViewSort<ToDoMarker> _gridViewSort;
1414

1515
/// <summary> Parameterless Constructor is to enable design view only. DO NOT USE. </summary>
16+
/// <param name="markers"></param>
1617
public TodoListSettingsUserControl()
1718
{
1819
InitializeComponent();

RubberduckTests/TodoControllerTests.cs

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
using Microsoft.VisualStudio.TestTools.UnitTesting;
33
using Moq;
44
using Rubberduck.Settings;
5+
using Rubberduck.UI;
56
using Rubberduck.UI.Settings;
67

78
namespace RubberduckTests
@@ -40,11 +41,8 @@ public void ViewTextIsNotNullOrEmptyAfterControllerConstruction()
4041
//arrange
4142
var markers = new List<ToDoMarker> {new ToDoMarker("Todo:", TodoPriority.Medium)};
4243

43-
var view = new TodoListSettingsUserControl(markers);
44+
var view = new TodoListSettingsUserControl(markers, new Mock<GridViewSort<ToDoMarker>>("", false).Object);
4445
var addTodoMarkerView = new Mock<IAddTodoMarkerView>().Object;
45-
46-
//act
47-
var controller = new TodoSettingPresenter(view, addTodoMarkerView);
4846

4947
//assert
5048
Assert.AreEqual("Todo:", view.ActiveMarkerText);
@@ -56,7 +54,7 @@ public void SetActiveItemChangesViewSelectedIndex()
5654
//arrange
5755
var markers = GetTestMarkers();
5856

59-
var view = new TodoListSettingsUserControl(markers);
57+
var view = new TodoListSettingsUserControl(markers, new Mock<GridViewSort<ToDoMarker>>("", false).Object);
6058

6159
//act
6260
view.SelectedIndex = 1;
@@ -75,7 +73,7 @@ public void ViewPriorityMatchesAfterSelectionChange()
7573
new ToDoMarker("Bug:", TodoPriority.High)
7674
};
7775

78-
var view = new TodoListSettingsUserControl(markers);
76+
var view = new TodoListSettingsUserControl(markers, new Mock<GridViewSort<ToDoMarker>>("", false).Object);
7977

8078
//act
8179
view.SelectedIndex = 2;
@@ -92,7 +90,7 @@ public void ViewTextMatchesAfterSelectionChange()
9290
new ToDoMarker("Note:", TodoPriority.Low)
9391
};
9492

95-
var view = new TodoListSettingsUserControl(markers);
93+
var view = new TodoListSettingsUserControl(markers, new Mock<GridViewSort<ToDoMarker>>("", false).Object);
9694

9795
//act
9896
view.SelectedIndex = 1;
@@ -105,7 +103,7 @@ public void MarkerChangeSavedOnPriorityChanged()
105103
{
106104
var markers = GetTestMarkers();
107105

108-
var view = new TodoListSettingsUserControl(markers);
106+
var view = new TodoListSettingsUserControl(markers, new Mock<GridViewSort<ToDoMarker>>("", false).Object);
109107

110108
view.ActiveMarkerPriority = TodoPriority.High;
111109

0 commit comments

Comments
 (0)