sched: fix pucch_resource_mng unittests

Signed-off-by: Carlo Galiotto <[email protected]>

Author: carlo-gal

lib/scheduler/pucch_scheduling/pucch_resource_manager.cpp

@@ -333,9 +333,9 @@ bool pucch_resource_manager::release_harq_resource(slot_point          slot_harq
   unsigned               ue_first_f1_res_id     = ue_res_id_set_for_harq.front();
   span<resource_tracker> slot_ue_res_array(&slot_res_array[ue_first_f1_res_id], slot_res_array.size());
 
-  auto* target_res = std::find_if(slot_ue_res_array.begin(), slot_ue_res_array.end(), [](const resource_tracker res) {
-    return res.rnti == INVALID_RNTI;
-  });
+  auto* target_res = std::find_if(slot_ue_res_array.begin(),
+                                  slot_ue_res_array.end(),
+                                  [crnti](const resource_tracker res) { return res.rnti == crnti; });
 
   // If the resources was found, then release it (i.e., remove the C-RNTI of the user allocated to it).
   if (target_res != slot_ue_res_array.end()) {

tests/unittests/scheduler/uci_and_pucch/pucch_res_manager_test.cpp

@@ -414,7 +414,7 @@ TEST_F(test_pucch_resource_manager, test_allocation_2_sr_resource)
 
   // Release resource and verify it was successful.
   ASSERT_TRUE(res_manager.release_sr_resource(sl_tx, to_rnti(0x4601), pucch_cfg));
-  ASSERT_TRUE(res_manager.release_sr_resource(sl_tx, to_rnti(0x4602), pucch_cfg));
+  ASSERT_TRUE(res_manager.release_sr_resource(sl_tx, to_rnti(0x4602), pucch_cfg_ue_2));
 
   // Attempt to allocate UEs over released SR resources.
   ASSERT_NE(nullptr, res_manager.reserve_sr_res_available(sl_tx, to_rnti(0x4603), pucch_cfg));