Skip to content

Commit 320ee73

Browse files
committed
Fix test setup.
1 parent 00b8d3a commit 320ee73

File tree

1 file changed

+10
-11
lines changed

1 file changed

+10
-11
lines changed

RubberduckTests/CodeExplorer/CodeExplorerReferenceFolderViewModelTests.cs

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -195,21 +195,23 @@ public void ErrorStateCanNotBeSet(ReferenceKind type)
195195
public void Constructor_PlacesAllReferences_TypeLibraries()
196196
{
197197
var references = AddRemoveReferencesSetup.DummyReferencesList;
198-
// ReSharper disable once UnusedVariable
198+
var expected = references.Count;
199+
199200
var folder = new CodeExplorerReferenceFolderViewModel(null, null, references, ReferenceKind.TypeLibrary);
200201

201-
Assert.AreEqual(references.Count, folder.Children.Count);
202+
Assert.AreEqual(expected, folder.Children.Count);
202203
}
203204

204205
[Test]
205206
[Category("Code Explorer")]
206207
public void Constructor_PlacesAllReferences_Projects()
207208
{
208209
var references = AddRemoveReferencesSetup.DummyProjectsList;
209-
// ReSharper disable once UnusedVariable
210+
var expected = references.Count;
211+
210212
var folder = new CodeExplorerReferenceFolderViewModel(null, null, references, ReferenceKind.Project);
211213

212-
Assert.AreEqual(references.Count, folder.Children.Count);
214+
Assert.AreEqual(expected, folder.Children.Count);
213215
}
214216

215217
[Test]
@@ -282,12 +284,11 @@ public void Synchronize_PlacesAllReferences_ProjectAdded()
282284
[Category("Code Explorer")]
283285
public void Synchronize_RemovesReference_TypeLibraryRemoved()
284286
{
285-
var references = AddRemoveReferencesSetup.DummyReferencesList;
286-
var folder = new CodeExplorerReferenceFolderViewModel(null, null, references, ReferenceKind.TypeLibrary);
287-
287+
var references = AddRemoveReferencesSetup.DummyReferencesList.ToList();
288288
var updates = AddRemoveReferencesSetup.DummyReferencesList.Take(references.Count - 1).ToList();
289289
var expected = updates.Select(reference => reference.Name).OrderBy(_ => _).ToList();
290290

291+
var folder = new CodeExplorerReferenceFolderViewModel(null, null, references, ReferenceKind.TypeLibrary);
291292
folder.Synchronize(null, updates);
292293
var actual = folder.Children.Cast<CodeExplorerReferenceViewModel>().Select(reference => reference.Reference.Name).OrderBy(_ => _);
293294

@@ -298,13 +299,11 @@ public void Synchronize_RemovesReference_TypeLibraryRemoved()
298299
[Category("Code Explorer")]
299300
public void Synchronize_RemovesReference_ProjectRemoved()
300301
{
301-
var references = AddRemoveReferencesSetup.DummyProjectsList;
302-
// ReSharper disable once UnusedVariable
303-
var folder = new CodeExplorerReferenceFolderViewModel(null, null, references, ReferenceKind.Project);
304-
302+
var references = AddRemoveReferencesSetup.DummyProjectsList.ToList();
305303
var updates = AddRemoveReferencesSetup.DummyProjectsList.Take(references.Count - 1).ToList();
306304
var expected = updates.Select(reference => reference.Name).OrderBy(_ => _).ToList();
307305

306+
var folder = new CodeExplorerReferenceFolderViewModel(null, null, references, ReferenceKind.Project);
308307
folder.Synchronize(null, updates);
309308
var actual = folder.Children.Cast<CodeExplorerReferenceViewModel>().Select(reference => reference.Reference.Name).OrderBy(_ => _);
310309

0 commit comments

Comments
 (0)