Browse Source

[3773] Updated comments

Francis Dupont 9 years ago
parent
commit
541c639fd0
2 changed files with 8 additions and 8 deletions
  1. 4 4
      src/bin/dhcp6/tests/confirm_unittest.cc
  2. 4 4
      src/bin/dhcp6/tests/rebind_unittest.cc

+ 4 - 4
src/bin/dhcp6/tests/confirm_unittest.cc

@@ -99,20 +99,20 @@ public:
 };
 
 
-// Test that clientID is mandatory and serverID forbidden for Confirm messages
+// Test that client-id is mandatory and server-id forbidden for Confirm messages
 TEST_F(ConfirmTest, sanityCheck) {
     NakedDhcpv6Srv srv(0);
 
-    // No clientID should fail
+    // A message with no client-id should fail
     Pkt6Ptr confirm = Pkt6Ptr(new Pkt6(DHCPV6_CONFIRM, 1234));
     EXPECT_THROW(srv.processConfirm(confirm), RFCViolation);
 
-    // A clientID should succeed
+    // A message with a single client-id should succeed
     OptionPtr clientid = generateClientId();
     confirm->addOption(clientid);
     EXPECT_NO_THROW(srv.processConfirm(confirm));
 
-    // A serverID should fail
+    // A message with server-id present should fail
     confirm->addOption(srv.getServerID());
     EXPECT_THROW(srv.processConfirm(confirm), RFCViolation);
 }

+ 4 - 4
src/bin/dhcp6/tests/rebind_unittest.cc

@@ -245,20 +245,20 @@ public:
     }
 };
 
-// Test that clientID is mandatory and serverID forbidden for Rebind messages
+// Test that client-id is mandatory and server-id forbidden for Rebind messages
 TEST_F(RebindTest, sanityCheck) {
     NakedDhcpv6Srv srv(0);
 
-    // No clientID should fail
+    // A message with no client-id should fail
     Pkt6Ptr rebind = Pkt6Ptr(new Pkt6(DHCPV6_REBIND, 1234));
     EXPECT_THROW(srv.processRebind(rebind), RFCViolation);
 
-    // A clientID should succeed
+    // A message with a single client-id should succeed
     OptionPtr clientid = generateClientId();
     rebind->addOption(clientid);
     EXPECT_NO_THROW(srv.processRebind(rebind));
 
-    // A serverID should fail
+    // A message with server-id present should fail
     rebind->addOption(srv.getServerID());
     EXPECT_THROW(srv.processRebind(rebind), RFCViolation);
 }