[#3826] addressed review comments

Author: Razvan Becheriu

ChangeLog

@@ -1,4 +1,4 @@
-2382.	[build]		razvan
+2382.	[func]		razvan
 	Implemented the 'lease6-get-by-hw-address' command used to query
 	IPv6 leases by HW Address.
 	(Gitlab #3826)

src/hooks/dhcp/lease_cmds/lease_cmds.cc

@@ -1597,7 +1597,7 @@ LeaseCmdsImpl::leaseGetByHwAddressHandler(CalloutHandle& handle) {
             isc_throw(BadValue, "'hw-address' parameter must be a string");
         }
 
-        if (!v4 && hw_address->stringValue().empty()) {
+        if (hw_address->stringValue().empty()) {
             isc_throw(BadValue, "'hw-address' parameter must not be empty");
         }
 

src/hooks/dhcp/lease_cmds/libloadtests/lease_cmds4_unittest.cc

@@ -1767,6 +1767,17 @@ void Lease4CmdsTest::testLease4GetByHwAddressParams() {
     exp_rsp = "'hw-address' parameter must be a string";
     testCommand(cmd, CONTROL_RESULT_ERROR, exp_rsp);
 
+    // Empty HWAddr.
+    cmd =
+        "{\n"
+        "    \"command\": \"lease4-get-by-hw-address\",\n"
+        "    \"arguments\": {"
+        "        \"hw-address\": \"\"\n"
+        "    }\n"
+        "}";
+    exp_rsp = "'hw-address' parameter must not be empty";
+    testCommand(cmd, CONTROL_RESULT_ERROR, exp_rsp);
+
     // Simply bad value.
     cmd =
         "{\n"
@@ -1795,15 +1806,6 @@ void Lease4CmdsTest::testLease4GetByHwAddressFind0() {
     string exp_rsp = "0 IPv4 lease(s) found.";
     testCommand(cmd, CONTROL_RESULT_EMPTY, exp_rsp);
 
-    // Empty HWAddr.
-    cmd =
-        "{\n"
-        "    \"command\": \"lease4-get-by-hw-address\",\n"
-        "    \"arguments\": {"
-        "        \"hw-address\": \"\"\n"
-        "    }\n"
-        "}";
-    testCommand(cmd, CONTROL_RESULT_EMPTY, exp_rsp);
 }
 
 void Lease4CmdsTest::testLease4GetByHwAddressFind2() {