Skip to content

Commit f394e9b

Browse files
committed
Another test works
1 parent e02451c commit f394e9b

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

RetailCoder.VBE/Settings/ToDoMarkers.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ public override bool Equals(object obj)
6868

6969
public override int GetHashCode()
7070
{
71-
return base.GetHashCode();
71+
return Text.GetHashCode();
7272
}
7373
}
7474
}

RubberduckTests/TodoControllerTests.cs

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -216,9 +216,13 @@ public void AddMarkerReallyDoesAddNewMarker()
216216
{
217217
var markers = GetTestMarkers();
218218

219+
var newMarkerText = "Hi, guys:";
220+
var newMarkerPriority = TodoPriority.High;
221+
219222
var addView = new Mock<IAddTodoMarkerView>();
220-
addView.SetupProperty(a => a.MarkerText, "Hi, guys:");
221-
addView.SetupProperty(a => a.MarkerPriority, TodoPriority.High);
223+
addView.SetupProperty(a => a.MarkerText, newMarkerText);
224+
addView.SetupProperty(a => a.MarkerPriority, newMarkerPriority);
225+
addView.SetupProperty(a => a.TodoMarkers, markers);
222226

223227
var view = new Mock<ITodoSettingsView>();
224228
view.SetupProperty(v => v.TodoMarkers, new BindingList<ToDoMarker>(markers));
@@ -229,7 +233,7 @@ public void AddMarkerReallyDoesAddNewMarker()
229233

230234
addView.Raise(a => a.AddMarker += null, EventArgs.Empty);
231235

232-
Assert.IsTrue(view.Object.TodoMarkers.Contains(new ToDoMarker(addView.Object.MarkerText, addView.Object.MarkerPriority)));
236+
Assert.IsTrue(view.Object.TodoMarkers.Contains(new ToDoMarker(newMarkerText, newMarkerPriority)));
233237
}
234238
}
235239
}

0 commit comments

Comments
 (0)