[llvm-commits] [llvm] r61883 - /llvm/trunk/unittests/ADT/DenseMapTest.cpp

Misha Brukman brukman+llvm at gmail.com
Wed Jan 7 13:13:53 PST 2009


Author: brukman
Date: Wed Jan  7 15:13:53 2009
New Revision: 61883

URL: http://llvm.org/viewvc/llvm-project?rev=61883&view=rev
Log:
Minor cleanup for unittest:
* Fixed {copy,assignment} constructor test names
* s/EXPECT_EQ(true, ...)/ASSERT_TRUE(...)/

Patch by Talin.

Modified:
    llvm/trunk/unittests/ADT/DenseMapTest.cpp

Modified: llvm/trunk/unittests/ADT/DenseMapTest.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/ADT/DenseMapTest.cpp?rev=61883&r1=61882&r2=61883&view=diff

==============================================================================
--- llvm/trunk/unittests/ADT/DenseMapTest.cpp (original)
+++ llvm/trunk/unittests/ADT/DenseMapTest.cpp Wed Jan  7 15:13:53 2009
@@ -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";
   }
 }
 





More information about the llvm-commits mailing list