Browse Source

[2906] Update/add some assertions in tests

Mukund Sivaraman 12 years ago
parent
commit
b5b6b43f34
1 changed files with 3 additions and 1 deletions
  1. 3 1
      src/lib/datasrc/tests/zone_table_accessor_unittest.cc

+ 3 - 1
src/lib/datasrc/tests/zone_table_accessor_unittest.cc

@@ -62,7 +62,7 @@ protected:
 TEST_F(ZoneTableAccessorTest, iteratorFromCache) {
     // Confirm basic iterator behavior.
     ZoneTableAccessor::IteratorPtr it = accessor_.getIterator();
-    EXPECT_TRUE(it);
+    ASSERT_TRUE(it);
     EXPECT_FALSE(it->isLast());
     EXPECT_EQ(0, it->getCurrent().index); // index is always 0 for this version
     EXPECT_EQ(Name("example.com"), it->getCurrent().origin);
@@ -87,6 +87,7 @@ TEST_F(ZoneTableAccessorTest, emptyTable) {
             "{\"cache-enable\": true, \"params\": {}}"), true);
     ZoneTableAccessorCache accessor(empty_config);
     ZoneTableAccessor::IteratorPtr it = accessor.getIterator();
+    ASSERT_TRUE(it);
     EXPECT_TRUE(it->isLast());
     EXPECT_THROW(it->getCurrent(), isc::InvalidOperation);
     EXPECT_THROW(it->next(), isc::InvalidOperation);
@@ -102,6 +103,7 @@ TEST_F(ZoneTableAccessorTest, disabledTable) {
             " \"cache-zones\": [\"example.com\", \"example.org\"]}"), true);
     ZoneTableAccessorCache accessor(mock_config);
     ZoneTableAccessor::IteratorPtr it = accessor.getIterator();
+    ASSERT_TRUE(it);
     EXPECT_TRUE(it->isLast());
     EXPECT_THROW(it->getCurrent(), isc::InvalidOperation);
     EXPECT_THROW(it->next(), isc::InvalidOperation);