Skip to content

Commit ada5ef9

Browse files
committed
Fix busted test
1 parent bcb824f commit ada5ef9

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

RubberduckTests/TodoControllerTests.cs

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
using System;
22
using System.Collections.Generic;
3+
using System.ComponentModel;
34
using Microsoft.VisualStudio.TestTools.UnitTesting;
45
using Moq;
56
using Rubberduck.Settings;
@@ -116,18 +117,16 @@ public void RemoveReallyDoesRemoveSelectedItem()
116117
{
117118
var markers = GetTestMarkers();
118119

119-
var view = new TodoListSettingsUserControl(markers, new Mock<GridViewSort<ToDoMarker>>("", false).Object);
120-
view.RemoveMarker += RemoveParam;
121-
view.SelectedIndex = 2;
120+
var view = new Mock<ITodoSettingsView>();
121+
view.SetupProperty(v => v.TodoMarkers, new BindingList<ToDoMarker>(markers));
122122

123-
RemoveParam(null, EventArgs.Empty);
123+
// Shut up R#, I need that to process the event
124+
// ReSharper disable once UnusedVariable
125+
var presenter = new TodoSettingPresenter(view.Object, new Mock<IAddTodoMarkerView>().Object);
124126

125-
Assert.AreEqual(2, view.TodoMarkers.Count);
126-
}
127+
view.Raise(v => v.RemoveMarker += null, EventArgs.Empty);
127128

128-
private void RemoveParam(object sender, EventArgs e)
129-
{
130-
129+
Assert.AreEqual(2, view.Object.TodoMarkers.Count);
131130
}
132131
}
133132
}

0 commit comments

Comments
 (0)