fixed out of range NoCRouterId bug in link removal test

Author: soheilshahrouz

vpr/test/test_noc_storage.cpp

@@ -279,7 +279,7 @@ TEST_CASE("test_remove_link", "[vpr_noc]") {
     std::mt19937 rand_num_gen(device());
 
     // random number generation to determine routers of the link to remove
-    std::uniform_int_distribution<std::mt19937::result_type> src_router(0, NUM_OF_ROUTERS);
+    std::uniform_int_distribution<std::mt19937::result_type> src_router(0, NUM_OF_ROUTERS-1);
     std::uniform_int_distribution<std::mt19937::result_type> sink_router(1, NOC_CONNECTIVITY - 1);
 
     // create the NoC
@@ -349,9 +349,9 @@ TEST_CASE("test_remove_link", "[vpr_noc]") {
 
         auto& outgoing_links = test_noc.get_noc_router_connections(link_to_remove_src_router);
         // go through all the outgoing links  of the source router in the link we removed and check that the link does not exist there as well.
-        for (auto outgoing_link_id = outgoing_links.begin(); outgoing_link_id != outgoing_links.end(); outgoing_link_id++) {
+        for (auto outgoing_link : outgoing_links) {
             // get the current outgoing link
-            const NocLink& curr_outgoing_link = test_noc.get_single_noc_link(*outgoing_link_id);
+            const NocLink& curr_outgoing_link = test_noc.get_single_noc_link(outgoing_link);
 
             if ((curr_outgoing_link.get_source_router() == link_to_remove_src_router) && (curr_outgoing_link.get_sink_router() == link_to_remove_sink_router)) {
                 link_removed_from_outgoing_vector = false;
@@ -363,9 +363,9 @@ TEST_CASE("test_remove_link", "[vpr_noc]") {
         const auto& links_in_noc = test_noc.get_noc_links();
         // go through the links and make sure that none of them have the source and sink router of the link
         // that we removed. THe removed link should have the source and sink routers set to invalid values.
-        for (auto single_link = links_in_noc.begin(); single_link != links_in_noc.end(); single_link++) {
+        for (auto single_link : links_in_noc) {
             // check whether the source and sink router of the current link matches the routers in the link to remove
-            if ((single_link->get_source_router() == link_to_remove_src_router) && (single_link->get_sink_router() == link_to_remove_sink_router)) {
+            if ((single_link.get_source_router() == link_to_remove_src_router) && (single_link.get_sink_router() == link_to_remove_sink_router)) {
                 // this indicates that the link was not set to an invalid state and not removed properly
                 link_removed_from_outgoing_vector = false;
                 break;