commit c870bb5c96c14da387770685dc6f200442a7549d
author Misha Brukman <brukman+llvm@gmail.com> Wed Jan 07 21:13:53 2009 +0000
committer Misha Brukman <brukman+llvm@gmail.com> Wed Jan 07 21:13:53 2009 +0000
tree bc30ecf48dc393aa7ed73a6776d6bec5d1c56171
parent 177d84e048d3c0b94ff2d2ff1a9bda8d498c1397

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

unittests/ADT/DenseMapTest.cpp

diff --git a/unittests/ADT/DenseMapTest.cpp b/unittests/ADT/DenseMapTest.cpp
index dff0aff..15a5379 100644
--- a/unittests/ADT/DenseMapTest.cpp
+++ b/unittests/ADT/DenseMapTest.cpp
@@ -125,7 +125,7 @@
 }
 
 // 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 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 @@
 
   // 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";
   }
 }