[5305] Minor tweaks in tests.

diff --git a/src/lib/dhcpsrv/tests/cfg_shared_networks4_unittest.cc b/src/lib/dhcpsrv/tests/cfg_shared_networks4_unittest.cc
index ff625e87aa204ab33f59fa835c7f83914141f53e..c90d21f0e85eb90d68a19f55166e1ca48572ae8f 100644 (file)
--- a/src/lib/dhcpsrv/tests/cfg_shared_networks4_unittest.cc
+++ b/src/lib/dhcpsrv/tests/cfg_shared_networks4_unittest.cc
@@ -29,6 +29,9 @@ TEST(CfgSharedNetworks4Test, getByName) {
     ASSERT_TRUE(returned_network1);
     SharedNetwork4Ptr returned_network2 = cfg.getByName("dog");
     ASSERT_TRUE(returned_network2);
+
+    // Check that non-existent name does not return bogus data.
+    EXPECT_FALSE(cfg.getByName("ant"));
 }
 
 // This test verifies that it is possible to delete a network.
@@ -53,6 +56,10 @@ TEST(CfgSharedNetworks4Test, deleteByName) {
     ASSERT_NO_THROW(cfg.del(network2->getName()));
     ASSERT_FALSE(cfg.getByName(network1->getName()));
     ASSERT_FALSE(cfg.getByName(network2->getName()));
+
+    // Check that attempting to delete the same subnet twice will fail.
+    ASSERT_THROW(cfg.del(network1->getName()), BadValue);
+    ASSERT_THROW(cfg.del(network2->getName()), BadValue);
 }
 
 // This test verifies that shared networks must have unique names.

diff --git a/src/lib/dhcpsrv/tests/cfg_shared_networks6_unittest.cc b/src/lib/dhcpsrv/tests/cfg_shared_networks6_unittest.cc
index 3f261782fb78e7b6ccf220e7d6fb20a2a8f24002..5763cd6bd9c36785296b96d3ea892d80953d5963 100644 (file)
--- a/src/lib/dhcpsrv/tests/cfg_shared_networks6_unittest.cc
+++ b/src/lib/dhcpsrv/tests/cfg_shared_networks6_unittest.cc
@@ -29,6 +29,9 @@ TEST(CfgSharedNetworks6Test, getByName) {
     ASSERT_TRUE(returned_network1);
     SharedNetwork6Ptr returned_network2 = cfg.getByName("dog");
     ASSERT_TRUE(returned_network2);
+
+    // Check that non-existent name does not return bogus data.
+    EXPECT_FALSE(cfg.getByName("ant"));
 }
 
 // This test verifies that it is possible to delete a network.
@@ -53,6 +56,10 @@ TEST(CfgSharedNetworks6Test, deleteByName) {
     ASSERT_NO_THROW(cfg.del(network2->getName()));
     ASSERT_FALSE(cfg.getByName(network1->getName()));
     ASSERT_FALSE(cfg.getByName(network2->getName()));
+
+    // Check that attempting to delete the same subnet twice will fail.
+    ASSERT_THROW(cfg.del(network1->getName()), BadValue);
+    ASSERT_THROW(cfg.del(network2->getName()), BadValue);
 }
 
 // This test verifies that shared networks must have unique names.

diff --git a/src/lib/dhcpsrv/tests/shared_network_unittest.cc b/src/lib/dhcpsrv/tests/shared_network_unittest.cc
index 8a90bb51c1581dc41d04784ed48b338b47f6a48d..7a342ff8924c6eb81fccb611b2546155ed9aedf5 100644 (file)
--- a/src/lib/dhcpsrv/tests/shared_network_unittest.cc
+++ b/src/lib/dhcpsrv/tests/shared_network_unittest.cc
@@ -287,7 +287,7 @@ TEST(SharedNetwork4Test, destructSharedNetwork) {
 // This test verifies that shared network can be given a name and that
 // this name can be retrieved.
 TEST(SharedNetwork6Test, getName) {
-    // Create shared network with an initial name "dog".
+    // Create shared network with an initial name "frog".
     SharedNetwork6Ptr network(new SharedNetwork6("frog"));
     EXPECT_EQ("frog", network->getName());