Browse Source

[1627] Add a couple more testcases

 * Case when no file is specified
 * Case where both origin and file are not specified
Mukund Sivaraman 13 years ago
parent
commit
f95e202d7e
1 changed files with 14 additions and 0 deletions
  1. 14 0
      src/bin/auth/tests/config_unittest.cc

+ 14 - 0
src/bin/auth/tests/config_unittest.cc

@@ -313,6 +313,13 @@ TEST_F(MemoryDatasrcConfigTest, addDuplicateZones) {
 }
 
 TEST_F(MemoryDatasrcConfigTest, addBadZone) {
+    // origin and file are missing
+    EXPECT_THROW(parser->build(
+                     Element::fromJSON(
+                         "[{\"type\": \"memory\","
+                         "  \"zones\": [{}]}]")),
+                 AuthConfigError);
+
     // origin is missing
     EXPECT_THROW(parser->build(
                      Element::fromJSON(
@@ -320,6 +327,13 @@ TEST_F(MemoryDatasrcConfigTest, addBadZone) {
                          "  \"zones\": [{\"file\": \"example.zone\"}]}]")),
                  AuthConfigError);
 
+    // file is missing
+    EXPECT_THROW(parser->build(
+                     Element::fromJSON(
+                         "[{\"type\": \"memory\","
+                         "  \"zones\": [{\"origin\": \"example.com\"}]}]")),
+                 AuthConfigError);
+
     // missing zone file
     EXPECT_THROW(parser->build(
                      Element::fromJSON(