[1967] ADD_FAILURE() improvements in Dhcpv6SrvTest::DUID

diff --git a/src/bin/dhcp6/tests/dhcp6_srv_unittest.cc b/src/bin/dhcp6/tests/dhcp6_srv_unittest.cc
index f1e2c3e306b6c1e69ca0fd44f25d17fa4dd7311e..f21366dfc7b75f19fbbb9933f4123544e7841ce1 100644 (file)
--- a/src/bin/dhcp6/tests/dhcp6_srv_unittest.cc
+++ b/src/bin/dhcp6/tests/dhcp6_srv_unittest.cc
@@ -143,12 +143,18 @@ TEST_F(Dhcpv6SrvTest, DUID) {
     }
     case DUID_LL: {
         // not supported yet
-        cout << "Test not implemented for DUID-LL yet." << endl;
+        cout << "Test not implemented for DUID-LL." << endl;
+
+        // No failure here. There's really no way for test LL DUID. It doesn't
+        // even make sense to check if that Link Layer is actually present on
+        // a physical interface. RFC3315 says a server should write its DUID
+        // and keep it despite hardware changes.
+        break;
     }
     case DUID_UUID: // not supported yet
     default:
-        cout << "Not supported duid type=" << duid_type << endl;
-        ADD_FAILURE();
+        ADD_FAILURE() << "Not supported duid type=" << duid_type << endl;
+        break;
     }
     delete srv; // destructor will close sockets, causing next test to succeed
 }