Skip to content

Commit d11baed

Browse files
CloneDeathaalhour
authored andcommitted
Fixed warnings in the unit tests (aalhour#63)
1 parent dc5b4d4 commit d11baed

File tree

3 files changed

+10
-10
lines changed

3 files changed

+10
-10
lines changed

UnitTest/DataStructuresTests/BinarySearchTreeTest.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@ public static void AssertTreeWithDuplicatesElements()
2222
// Insert values with duplicates
2323
binarySearchTree.Insert(values);
2424

25-
// ASSERT COUNT = 20 (allows duplicates)
26-
Assert.Equal(binarySearchTree.Count, 21);
25+
// ASSERT COUNT = 21 (allows duplicates)
26+
Assert.Equal(21, binarySearchTree.Count);
2727

2828
// Test contains/find
2929
Assert.True(binarySearchTree.Contains(10), "Wrong element.");
@@ -86,7 +86,7 @@ public static void AssertTreeWithDuplicatesElements()
8686

8787
var enumerator = binarySearchTree.GetInOrderEnumerator();
8888
enumerator.MoveNext();
89-
Assert.Equal(enumerator.Current, 7);
89+
Assert.Equal(7, enumerator.Current);
9090

9191
enumerator.MoveNext();
9292
enumerator.MoveNext();

UnitTest/DataStructuresTests/TrieMapTest.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,8 @@ public static void DoTest()
6868
// Assert there are only two words under the prefix "acti" -> active, & acting
6969
var someActiWords = trieMap.SearchByPrefix("acti").Select(item => item.Key).ToList<string>();
7070
Assert.Equal(2, someActiWords.Count);
71-
Assert.True(someActiWords.Contains(word_acting));
72-
Assert.True(someActiWords.Contains(word_active));
71+
Assert.Contains(word_acting, someActiWords);
72+
Assert.Contains(word_active, someActiWords);
7373

7474
// Assert that "acto" is not a word
7575
Assert.False(trieMap.ContainsWord("acto"));
@@ -123,8 +123,8 @@ public static void DoTest()
123123
Assert.Equal(8, trieMap.Count);
124124

125125
someActiWords = trieMap.SearchByPrefix("acti").Select(item => item.Key).ToList<string>();
126-
Assert.Equal(1, someActiWords.Count);
127-
Assert.True(someActiWords.Contains(word_active));
126+
Assert.Single(someActiWords);
127+
Assert.Contains(word_active, someActiWords);
128128

129129
//
130130
// TEST ENUMERATOR

UnitTest/DataStructuresTests/TrieTest.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,8 @@ public static void DoTest()
6868
// Assert there are only two words under the prefix "acti" -> active, & acting
6969
var someActiWords = trie.SearchByPrefix("acti").ToList<string>();
7070
Assert.True(someActiWords.Count == 2);
71-
Assert.True(someActiWords.Contains(word_acting));
72-
Assert.True(someActiWords.Contains(word_active));
71+
Assert.Contains(word_acting, someActiWords);
72+
Assert.Contains(word_active, someActiWords);
7373

7474
// Assert that "acto" is not a word
7575
Assert.False(trie.ContainsWord("acto"));
@@ -119,7 +119,7 @@ public static void DoTest()
119119

120120
someActiWords = trie.SearchByPrefix("acti").ToList<string>();
121121
Assert.True(someActiWords.Count == 1);
122-
Assert.True(someActiWords.Contains(word_active));
122+
Assert.Contains(word_active, someActiWords);
123123

124124

125125

0 commit comments

Comments
 (0)