BIND 10 trac2940, updated. 91cb5a837770be0fe38b64441ede9bd461443018 [2940] Extended unit test which covers the null client id leases.

[BIND 10 source code commits]

The branch, trac2940 has been updated
       via  91cb5a837770be0fe38b64441ede9bd461443018 (commit)
      from  606b06cef9bae2038a10dad2ac89db8f93708e84 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 91cb5a837770be0fe38b64441ede9bd461443018
Author: Marcin Siodelski <marcin at isc.org>
Date:   Thu Nov 14 14:58:22 2013 +0100

    [2940] Extended unit test which covers the null client id leases.
    
    Also applied a couple of minor editorial fixes.

-----------------------------------------------------------------------

Summary of changes:
 src/lib/dhcpsrv/memfile_lease_mgr.cc               |    2 +-
 src/lib/dhcpsrv/memfile_lease_mgr.h                |    4 +--
 .../dhcpsrv/tests/memfile_lease_mgr_unittest.cc    |   38 +++++++++++++++-----
 src/lib/dhcpsrv/tests/test_utils.cc                |    2 +-
 4 files changed, 34 insertions(+), 12 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/lib/dhcpsrv/memfile_lease_mgr.cc b/src/lib/dhcpsrv/memfile_lease_mgr.cc
index 9eefb1f..075a082 100644
--- a/src/lib/dhcpsrv/memfile_lease_mgr.cc
+++ b/src/lib/dhcpsrv/memfile_lease_mgr.cc
@@ -81,7 +81,7 @@ Memfile_LeaseMgr::getLease4(const HWAddr& hwaddr) const {
 
         // Every Lease4 has a hardware address, so we can compare it
         if((*lease)->hwaddr_ == hwaddr.hwaddr_) {
-            collection.push_back((* lease));
+            collection.push_back((*lease));
         }
     }
 
diff --git a/src/lib/dhcpsrv/memfile_lease_mgr.h b/src/lib/dhcpsrv/memfile_lease_mgr.h
index 2012a5b..44d07d3 100644
--- a/src/lib/dhcpsrv/memfile_lease_mgr.h
+++ b/src/lib/dhcpsrv/memfile_lease_mgr.h
@@ -102,8 +102,8 @@ public:
 
     /// @brief Returns existing IPv4 lease for specified client-id
     ///
-    /// @param clientid client identifier
-    virtual Lease4Collection getLease4(const ClientId& clientid) const;
+    /// @param client_id client identifier
+    virtual Lease4Collection getLease4(const ClientId& client_id) const;
 
     /// @brief Returns IPv4 lease for specified client-id/hwaddr/subnet-id tuple
     ///
diff --git a/src/lib/dhcpsrv/tests/memfile_lease_mgr_unittest.cc b/src/lib/dhcpsrv/tests/memfile_lease_mgr_unittest.cc
index 2f12929..0117e38 100644
--- a/src/lib/dhcpsrv/tests/memfile_lease_mgr_unittest.cc
+++ b/src/lib/dhcpsrv/tests/memfile_lease_mgr_unittest.cc
@@ -167,14 +167,32 @@ TEST_F(MemfileLeaseMgrTest, getLease4NullClientId) {
     Lease4Ptr leaseA = initializeLease4(straddress4_[4]);
     ClientIdPtr client_id = leaseA->client_id_;
     leaseA->client_id_ = ClientIdPtr();
-    EXPECT_TRUE(lease_mgr->addLease(leaseA));
+    ASSERT_TRUE(lease_mgr->addLease(leaseA));
 
     Lease4Collection returned = lease_mgr->getLease4(*client_id);
     // Shouldn't have our previous lease ...
-    ASSERT_EQ(0, returned.size());
-    Lease4Ptr leaseB = initializeLease4(straddress4_[5]);
+    ASSERT_TRUE(returned.empty());
+
+    // Add another lease with the non-NULL client id, and make sure that the
+    // lookup will not break due to existence of both leases with non-NULL and
+    // NULL client ids.
+    Lease4Ptr leaseB = initializeLease4(straddress4_[0]);
     // Shouldn't throw any null pointer exception
-    EXPECT_NO_THROW(lease_mgr->addLease(leaseB));
+    ASSERT_TRUE(lease_mgr->addLease(leaseB));
+    // Try to get the lease.
+    returned = lease_mgr->getLease4(*client_id);
+    ASSERT_TRUE(returned.empty());
+
+    // Let's make it more interesting and add another lease with NULL client id.
+    Lease4Ptr leaseC = initializeLease4(straddress4_[5]);
+    leaseC->client_id_.reset();
+    ASSERT_TRUE(lease_mgr->addLease(leaseC));
+    returned = lease_mgr->getLease4(*client_id);
+    ASSERT_TRUE(returned.empty());
+
+    // But getting the lease with non-NULL client id should be successful.
+    returned = lease_mgr->getLease4(*leaseB->client_id_);
+    ASSERT_EQ(1, returned.size());
 }
 
 // Checks lease4 retrieval through HWAddr
@@ -219,19 +237,23 @@ TEST_F(MemfileLeaseMgrTest, getLease4ClientIdHWAddrSubnetId) {
     EXPECT_TRUE(lease == Lease4Ptr());
 }
 
+// This test verifies that the client id can be returned as a vector.
+// @todo This test should be moved to the Lease specific unit tests once
+// these tests are created.
 TEST_F(MemfileLeaseMgrTest, getLease4ClientIdVector) {
     const LeaseMgr::ParameterMap pmap;
     boost::scoped_ptr<Memfile_LeaseMgr> lease_mgr(new Memfile_LeaseMgr(pmap));
 
-    const std::vector<uint8_t> vec;
     Lease4Ptr lease = initializeLease4(straddress4_[7]);
     // Check that this lease has null client-id
     ASSERT_TRUE(lease->client_id_ == ClientIdPtr());
-    // Check that this return empty vector
-    ASSERT_TRUE(lease->getClientIdVector() == vec);
+    // Check that this returns empty vector
+    ASSERT_TRUE(lease->getClientIdVector().empty());
+
     // Let's take a lease with client-id not null
     lease = initializeLease4(straddress4_[6]);
-    // Check that they return same client-id value
+    ASSERT_TRUE(lease->client_id_);
+    // Check that they return the same client-id value
     ASSERT_TRUE(lease->client_id_->getClientId() == lease->getClientIdVector());
 }
 
diff --git a/src/lib/dhcpsrv/tests/test_utils.cc b/src/lib/dhcpsrv/tests/test_utils.cc
index 9b4263f..12f474a 100644
--- a/src/lib/dhcpsrv/tests/test_utils.cc
+++ b/src/lib/dhcpsrv/tests/test_utils.cc
@@ -217,7 +217,7 @@ GenericLeaseMgrTest::initializeLease4(std::string address) {
         lease->fqdn_rev_ = false;
         lease->fqdn_fwd_ = false;
         lease->hostname_ = "otherhost.example.com.";
-        } else if (address == straddress4_[6]) {
+    } else if (address == straddress4_[6]) {
         lease->hwaddr_ = vector<uint8_t>(6, 0x6e);
         // Same ClientId as straddress4_1
         lease->client_id_ = ClientIdPtr(