From: Misha Brukman <brukman+llvm@gmail.com>
Date: Wed, 7 Jan 2009 21:13:53 +0000 (+0000)
Subject: Minor cleanup for unittest:
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=c870bb5c96c14da387770685dc6f200442a7549d;p=oota-llvm.git

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
---

diff --git a/unittests/ADT/DenseMapTest.cpp b/unittests/ADT/DenseMapTest.cpp
index dff0aff9b3e..15a5379fb8d 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";
   }
 }