Make sure the unit tests cover one more scenario, tip from mind04

diff --git a/pdns/test-communicator_hh.cc b/pdns/test-communicator_hh.cc
index 0973b0300b7e2e377e3dda5d7405f87f7c13ca93..131da630fcc13353dab04e39137306f2765ad21c 100644 (file)
--- a/pdns/test-communicator_hh.cc
+++ b/pdns/test-communicator_hh.cc
@@ -69,6 +69,7 @@ BOOST_AUTO_TEST_CASE(test_axfr_queue_insert_and_priority_order_after_modify) {
     {DNSName("test5.com"),ComboAddress("0.0.0.0"),false,{SuckRequest::PdnsControl,3}},
   };
   SuckRequest rr = {DNSName("test3.com"),ComboAddress("0.0.0.0"),false,{SuckRequest::PdnsControl,5}};
+  SuckRequest rr2 = {DNSName("test4.com"),ComboAddress("0.0.0.0"),false,{SuckRequest::Api,6}};
 
   UniQueue suckDomains;
 
@@ -86,6 +87,14 @@ BOOST_AUTO_TEST_CASE(test_axfr_queue_insert_and_priority_order_after_modify) {
     }
   });
 
+  res = suckDomains.insert(rr2);
+  BOOST_CHECK(!res.second);
+  suckDomains.modify(res.first, [priorityAndOrder = rr2.priorityAndOrder] (SuckRequest& so) {
+    if (priorityAndOrder.first < so.priorityAndOrder.first) {
+      so.priorityAndOrder = priorityAndOrder;
+    }
+  });
+
   for (int i = 4; i >= 0; i--) {
     auto iter = suckDomains.begin();
     BOOST_CHECK_EQUAL(iter->domain, sr[i].domain);