[2615] Addressed review comments

Minor changes to commentary, etc.

diff --git a/src/bin/dhcp4/tests/out_of_range_unittest.cc b/src/bin/dhcp4/tests/out_of_range_unittest.cc
index 066cf45810134721f840e05797a5d4eb772441a0..a2920724edb121d9349e92fb6ef8a254131bf64c 100644 (file)
--- a/src/bin/dhcp4/tests/out_of_range_unittest.cc
+++ b/src/bin/dhcp4/tests/out_of_range_unittest.cc
@@ -19,16 +19,12 @@
 #include <dhcp_ddns/ncr_msg.h>
 #include <dhcp/tests/iface_mgr_test_config.h>
 #include <dhcpsrv/cfgmgr.h>
-#include <dhcpsrv/host.h>
-#include <dhcpsrv/host_mgr.h>
 #include <dhcpsrv/subnet_id.h>
 #include <dhcp4/tests/dhcp4_test_utils.h>
 #include <dhcp4/tests/dhcp4_client.h>
-#include <boost/shared_ptr.hpp>
 
 using namespace isc;
 using namespace isc::asiolink;
-using namespace isc::data;
 using namespace isc::dhcp;
 using namespace isc::dhcp::test;
 
@@ -46,10 +42,8 @@ namespace {
 ///   - 1 hostname reservation (dynamic host) for HW address:
 ///     dd:dd:dd:dd:dd:01,
 ///   - DDNS enabled
-/// - Configuration 1:
-///   - Same as configuration 1 but with a different pool range
-/// - Configuration 2 - same subnet as reference, different pool,
-///       no reservations
+/// - Configuration 1 - same subnet, different pool
+/// - Configuration 2 - same subnet, different pool, no reservations
 /// - Configuration 3 - different subnet with reservations
 /// - Configuration 4 - different subnet with no reservations
 /// - Configuration 5 - same as reference, no reservations
@@ -451,7 +445,7 @@ TEST_F(OutOfRangeTest, dynamicHostOutOfSubnet) {
 // whose address is within the configured subnet and pool, but whose
 // reservation has been removed:
 //
-// a: Is allowed to renew 
+// a: Is allowed to renew
 // b: Is released properly upon release, including DNS removal
 //
 TEST_F(OutOfRangeTest, dynamicHostReservationRemoved) {

diff --git a/src/bin/dhcp4/tests/release_unittest.cc b/src/bin/dhcp4/tests/release_unittest.cc
index 8d80994fb8fd04ac062df66e276ce38f09a05dc7..c1ac1fda9a8acf5d6381c1e58a1ba262eabdce82 100644 (file)
--- a/src/bin/dhcp4/tests/release_unittest.cc
+++ b/src/bin/dhcp4/tests/release_unittest.cc
@@ -286,7 +286,7 @@ TEST_F(ReleaseTest, releaseNonMatchingIPAddress) {
 }
 
 // This test verifies that an incoming RELEASE for an address within
-// a subnet that has been removed
+// a subnet that has been removed can still be released.
 TEST_F(ReleaseTest, releaseNoSubnet) {
     Dhcp4Client client(Dhcp4Client::SELECTING);
     // Configure DHCP server.

diff --git a/src/bin/dhcp6/tests/dhcp6_client.h b/src/bin/dhcp6/tests/dhcp6_client.h
index e1cf0a5ca062f1925a4ecffcd04049bb66898c05..35462321ce5a00f48aef9d20d77420ce6cdfde68 100644 (file)
--- a/src/bin/dhcp6/tests/dhcp6_client.h
+++ b/src/bin/dhcp6/tests/dhcp6_client.h
@@ -235,8 +235,6 @@ public:
     /// @todo Perform sanity checks on returned messages.
     void doRebind();
 
-    void doRelease();
-
     /// @brief Sends Request to the server and receives Reply.
     ///
     /// This function simulates sending the Request message to the server and