Minor cleanup for unittest:

* Fixed {copy,assignment} constructor test names
* s/EXPECT_EQ(true, ...)/ASSERT_TRUE(...)/

Patch by Talin.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61883 91177308-0d34-0410-b5e6-96231b3b80d8

1 files changed, 3 insertions, 3 deletions

diff --git a/unittests/ADT/DenseMapTest.cpp b/unittests/ADT/DenseMapTest.cpp
index dff0aff9b3..15a5379fb8 100644
--- a/unittests/ADT/DenseMapTest.cpp
+++ b/unittests/ADT/DenseMapTest.cpp
@@ -125,7 +125,7 @@ TEST_F(DenseMapTest, InsertTest) {
 }
 
 // Test copy constructor method
-TEST_F(DenseMapTest, AssignmentTest) {
+TEST_F(DenseMapTest, CopyConstructorTest) {
   uintMap[0] = 1;
   DenseMap<uint32_t, uint32_t> copyMap(uintMap);
 
@@ -134,7 +134,7 @@ TEST_F(DenseMapTest, AssignmentTest) {
 }
 
 // Test assignment operator method
-TEST_F(DenseMapTest, CopyConstructorTest) {
+TEST_F(DenseMapTest, AssignmentTest) {
   uintMap[0] = 1;
   DenseMap<uint32_t, uint32_t> copyMap = uintMap;
 
@@ -160,7 +160,7 @@ TEST_F(DenseMapTest, IterationTest) {
 
   // Ensure every number was visited.
   for (int i = 0; i < 100; ++i) {
-    EXPECT_EQ(true, visited[i]);
+    ASSERT_TRUE(visited[i]) << "Entry #" << i << " was never visited";
   }
 }