[2833] added test cases where 'allowed' is false

diff --git a/src/lib/datasrc/tests/cache_config_unittest.cc b/src/lib/datasrc/tests/cache_config_unittest.cc
index 43cc6672e6652621b5e275308ef170096d7d5bf3..00959f1cae0daa6b49f0b2c2fc1f9aaca4cba0e4 100644 (file)
--- a/src/lib/datasrc/tests/cache_config_unittest.cc
+++ b/src/lib/datasrc/tests/cache_config_unittest.cc
@@ -99,6 +99,11 @@ TEST_F(CacheConfigTest, badConstructMasterFiles) {
                              *Element::fromJSON("{\"cache-enable\": false,"
                                                 " \"params\": {}}"), true),
                  CacheConfigError);
+    // cache enabled but not "allowed"
+    EXPECT_THROW(CacheConfig("MasterFiles", 0,
+                             *Element::fromJSON("{\"cache-enable\": false,"
+                                                " \"params\": {}}"), false),
+                 CacheConfigError);
     // type error for cache-enable
     EXPECT_THROW(CacheConfig("MasterFiles", 0,
                              *Element::fromJSON("{\"cache-enable\": 1,"
@@ -162,8 +167,13 @@ TEST_F(CacheConfigTest, constructWithMock) {
     const ConstElementPtr config_elem_disabled(
         Element::fromJSON("{\"cache-enable\": false,"
                           " \"cache-zones\": [\"example.com\"]}"));
-    EXPECT_TRUE(CacheConfig("mock", &mock_client_, *config_elem_disabled, true).
-                getZoneConfig().empty());
+    EXPECT_FALSE(CacheConfig("mock", &mock_client_, *config_elem_disabled,
+                             true).isEnabled());
+    // enabled but not "allowed".  same effect.
+    EXPECT_FALSE(CacheConfig("mock", &mock_client_,
+                             *Element::fromJSON("{\"cache-enable\": true,"
+                                                " \"cache-zones\": []}"),
+                             false).isEnabled());
 }
 
 TEST_F(CacheConfigTest, badConstructWithMock) {