[3035] Added functions to construct DHCID object from raw buffer.

diff --git a/src/lib/dhcp_ddns/ncr_msg.cc b/src/lib/dhcp_ddns/ncr_msg.cc
index 3c7a18eceaabfba35a3541c854b1d791d0c7c5e2..d5a36f5887b710d9102b782285e34b355df280b7 100644 (file)
--- a/src/lib/dhcp_ddns/ncr_msg.cc
+++ b/src/lib/dhcp_ddns/ncr_msg.cc
@@ -34,6 +34,10 @@ D2Dhcid::D2Dhcid(const std::string& data) {
     fromStr(data);
 }
 
+D2Dhcid::D2Dhcid(const std::vector<uint8_t>& data)
+    : bytes_(data) {
+}
+
 D2Dhcid::D2Dhcid(const isc::dhcp::DUID& duid,
                  const std::vector<uint8_t>& wire_fqdn) {
     fromDUID(duid, wire_fqdn);
@@ -55,6 +59,11 @@ D2Dhcid::toStr() const {
     return (isc::util::encode::encodeHex(bytes_));
 }
 
+void
+D2Dhcid::fromBytes(const std::vector<uint8_t>& data) {
+    bytes_ = data;
+}
+
 void
 D2Dhcid::fromDUID(const isc::dhcp::DUID& duid,
                   const std::vector<uint8_t>& wire_fqdn) {

diff --git a/src/lib/dhcp_ddns/ncr_msg.h b/src/lib/dhcp_ddns/ncr_msg.h
index ceb8715c19039b4d25c7156579e946065f47281b..db693dfb9e474c57b398dfe7bb39fd16cc0e6c66 100644 (file)
--- a/src/lib/dhcp_ddns/ncr_msg.h
+++ b/src/lib/dhcp_ddns/ncr_msg.h
@@ -78,6 +78,12 @@ public:
     /// or there is an odd number of digits.
     D2Dhcid(const std::string& data);
 
+    /// @brief Constructor, creates new instance of the @c D2Dhcid using a
+    /// vector holding raw DHCID.
+    ///
+    /// @param data An vector holding DHCID in the raw format.
+    D2Dhcid(const std::vector<uint8_t>& data);
+
     /// @brief Constructor, creates an instance of the @c D2Dhcid from the
     /// @c isc::dhcp::DUID.
     ///
@@ -101,6 +107,11 @@ public:
     /// or there is an odd number of digits.
     void fromStr(const std::string& data);
 
+    /// @brief Sets the DHCID value represented by raw data.
+    ///
+    /// @param data Holds the raw bytes representing DHCID.
+    void fromBytes(const std::vector<uint8_t>& data);
+
     /// @brief Sets the DHCID value based on the DUID and FQDN.
     ///
     /// This function requires that the FQDN conforms to the section 3.5

diff --git a/src/lib/dhcp_ddns/tests/ncr_unittests.cc b/src/lib/dhcp_ddns/tests/ncr_unittests.cc
index df1a50a69aae2f06ef0576181125f930489b3d14..418658e1cc32ef48fae87810e9b1a869eb0e03e5 100644 (file)
--- a/src/lib/dhcp_ddns/tests/ncr_unittests.cc
+++ b/src/lib/dhcp_ddns/tests/ncr_unittests.cc
@@ -387,6 +387,32 @@ TEST(NameChangeRequestTest, dhcidFromMaxDUID) {
     EXPECT_EQ(dhcid_ref, dhcid.toStr());
 }
 
+// Test that DHCID is correctly created from the buffer holding DHCID data
+// in raw format.
+TEST(NameChangeRequestTest, dhcidFromBytes) {
+    // Create a buffer holding raw DHCID data.
+    std::vector<uint8_t> dhcid_data;
+    for (int i = 0; i < 64; ++i) {
+        dhcid_data.push_back(i);
+    }
+    // Construct new object and initialize it with the DHCID data.
+    D2Dhcid dhcid(dhcid_data);
+
+    // Make sure that the DHCID is valid.
+    EXPECT_TRUE(std::equal(dhcid.getBytes().begin(), dhcid.getBytes().end(),
+                           dhcid_data.begin()));
+
+    // Modify the buffer and reinitialize DHCID with the new buffer.
+    for (int i = 64; i < 128; ++i) {
+        dhcid_data.push_back(i);
+    }
+    ASSERT_NO_THROW(dhcid.fromBytes(dhcid_data));
+
+    // Make sure that the DHCID is still valid.
+    EXPECT_TRUE(std::equal(dhcid.getBytes().begin(), dhcid.getBytes().end(),
+                           dhcid_data.begin()));
+
+}
 
 /// @brief Verifies the fundamentals of converting from and to JSON.
 /// It verifies that: