[#489,!250] Addressed review comments.

diff --git a/src/hooks/dhcp/mysql_cb/tests/mysql_cb_dhcp4_unittest.cc b/src/hooks/dhcp/mysql_cb/tests/mysql_cb_dhcp4_unittest.cc
index c997cbe07edf896cf14cf4157715103cf6169ca9..7cb3ea59a79211af470132fa4c6b69d355ab08f6 100644 (file)
--- a/src/hooks/dhcp/mysql_cb/tests/mysql_cb_dhcp4_unittest.cc
+++ b/src/hooks/dhcp/mysql_cb/tests/mysql_cb_dhcp4_unittest.cc
@@ -632,7 +632,7 @@ TEST_F(MySqlConfigBackendDHCPv4Test, getSubnet4) {
 
 // Test that the information about unspecified optional parameters gets
 // propagated to the database.
-TEST_F(MySqlConfigBackendDHCPv4Test, getSubnet4WithDefaults) {
+TEST_F(MySqlConfigBackendDHCPv4Test, getSubnet4WithOptionalUnspecified) {
     // Insert new subnet.
     Subnet4Ptr subnet = test_subnets_[2];
     cbptr_->createUpdateSubnet4(ServerSelector::ALL(), subnet);
@@ -997,7 +997,7 @@ TEST_F(MySqlConfigBackendDHCPv4Test, getSharedNetwork4) {
 
 // Test that the information about unspecified optional parameters gets
 // propagated to the database.
-TEST_F(MySqlConfigBackendDHCPv4Test, getSharedNetwork4WithDefaults) {
+TEST_F(MySqlConfigBackendDHCPv4Test, getSharedNetwork4WithOptionalUnspecified) {
     // Insert new shared network.
     SharedNetwork4Ptr shared_network = test_networks_[2];
     cbptr_->createUpdateSharedNetwork4(ServerSelector::ALL(), shared_network);

diff --git a/src/lib/mysql/tests/mysql_binding_unittest.cc b/src/lib/mysql/tests/mysql_binding_unittest.cc
index f75f83e06593d13cd9b76807b401c040bfc50667..851b002da662c4adbb57b3eede8beb164c9cf35d 100644 (file)
--- a/src/lib/mysql/tests/mysql_binding_unittest.cc
+++ b/src/lib/mysql/tests/mysql_binding_unittest.cc
@@ -28,6 +28,7 @@ TEST(MySqlBindingTest, defaultString) {
     EXPECT_EQ("foo", binding->getStringOrDefault("foo"));
 
     binding = MySqlBinding::createString("bar");
+    ASSERT_FALSE(binding->amNull());
     EXPECT_EQ("bar", binding->getStringOrDefault("foo"));
 }
 
@@ -100,9 +101,11 @@ TEST(MySqlBindingTest, blobTypeMismatch) {
 // This test verifies that default number is returned if binding is null.
 TEST(MySqlBindingTest, defaultInteger) {
     auto binding = MySqlBinding::createNull();
+    ASSERT_TRUE(binding->amNull());
     EXPECT_EQ(123, binding->getIntegerOrDefault<uint32_t>(123));
 
     binding = MySqlBinding::createInteger<uint32_t>(1024);
+    ASSERT_FALSE(binding->amNull());
     EXPECT_EQ(1024, binding->getIntegerOrDefault<uint32_t>(123));
 }