X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/81755fa502502324157b2ef592288e5782298049..89feb425bdeb4351ac0e27a38d9a9b3463fff433:/src/kernel/routing/StarZone_test.cpp?ds=sidebyside diff --git a/src/kernel/routing/StarZone_test.cpp b/src/kernel/routing/StarZone_test.cpp index 849489c90f..afc79b61cf 100644 --- a/src/kernel/routing/StarZone_test.cpp +++ b/src/kernel/routing/StarZone_test.cpp @@ -11,30 +11,32 @@ #include "simgrid/s4u/Host.hpp" #include "simgrid/s4u/NetZone.hpp" #include "src/surf/network_interface.hpp" -#include "src/surf/surf_interface.hpp" // create models -#include "src/surf/xml/platf_private.hpp" // RouteCreationArgs and friends - -namespace { -class EngineWrapper { - int argc = 1; - char* argv; - simgrid::s4u::Engine e; - -public: - explicit EngineWrapper(std::string name) : argv(&name[0]), e(&argc, &argv) {} -}; -} // namespace TEST_CASE("kernel::routing::StarZone: Creating Zone", "[creation]") { - EngineWrapper e("test"); + simgrid::s4u::Engine e("test"); REQUIRE(simgrid::s4u::create_star_zone("test")); } +TEST_CASE("kernel::routing::StarZone: Create links: exceptions", "") +{ + simgrid::s4u::Engine e("test"); + auto* zone = simgrid::s4u::create_star_zone("test"); + SECTION("create_link: invalid bandwidth") + { + REQUIRE_THROWS_AS(zone->create_link("link", "speed"), std::invalid_argument); + } + + SECTION("split-duplex create_link: invalid bandwidth") + { + REQUIRE_THROWS_AS(zone->create_split_duplex_link("link", "speed"), std::invalid_argument); + } +} + TEST_CASE("kernel::routing::StarZone: Adding routes (hosts): exception", "") { - EngineWrapper e("test"); + simgrid::s4u::Engine e("test"); auto* zone = new simgrid::kernel::routing::StarZone("test"); auto* netpoint1 = new simgrid::kernel::routing::NetPoint("netpoint1", simgrid::kernel::routing::NetPoint::Type::Host); auto* netpoint2 = new simgrid::kernel::routing::NetPoint("netpoint2", simgrid::kernel::routing::NetPoint::Type::Host); @@ -57,12 +59,14 @@ TEST_CASE("kernel::routing::StarZone: Adding routes (hosts): exception", "") TEST_CASE("kernel::routing::StarZone: Adding routes (netzones): exception", "") { - EngineWrapper e("test"); + simgrid::s4u::Engine e("test"); auto* zone = new simgrid::kernel::routing::StarZone("test"); auto* netpoint1 = new simgrid::kernel::routing::NetPoint("netpoint1", simgrid::kernel::routing::NetPoint::Type::NetZone); auto* netpoint2 = new simgrid::kernel::routing::NetPoint("netpoint2", simgrid::kernel::routing::NetPoint::Type::NetZone); + auto zone3 = std::make_unique("test3"); + auto* netpoint3 = zone3->create_router("netpoint3"); SECTION("src: is a netzone and gw_src: nullptr") { @@ -83,31 +87,40 @@ TEST_CASE("kernel::routing::StarZone: Adding routes (netzones): exception", "") { REQUIRE_THROWS_AS(zone->add_route(nullptr, netpoint2, nullptr, netpoint1, {}, false), std::invalid_argument); } + + SECTION("issue71: gw_src isn't member of the src netzone") + { + REQUIRE_THROWS_AS(zone->add_route(zone->get_netpoint(), nullptr, netpoint3, nullptr, {}, false), + std::invalid_argument); + } + + SECTION("issue71: gw_dst isn't member of the dst netzone") + { + REQUIRE_THROWS_AS(zone->add_route(nullptr, zone->get_netpoint(), nullptr, netpoint3, {}, false), + std::invalid_argument); + } } // One day we may be able to test contracts and asserts with catch2 // https://github.com/catchorg/Catch2/issues/853 TEST_CASE("kernel::routing::StarZone: Get routes: assert", "[.][assert]") { - /* workaround to initialize things, they must be done in this particular order */ - EngineWrapper e("test"); + simgrid::s4u::Engine e("test"); auto* zone = new simgrid::kernel::routing::StarZone("test"); - surf_network_model_init_LegrandVelho(); - surf_cpu_model_init_Cas01(); const auto* host1 = zone->create_host("netpoint1", {100}); const auto* host2 = zone->create_host("netpoint2", {100}); - std::vector links; - links.push_back(zone->create_link("link1", {100})->get_impl()); - std::vector links2; - links2.push_back(zone->create_link("link2", {100})->get_impl()); + std::vector links; + links.emplace_back(zone->create_link("link1", {100})); + std::vector links2; + links2.emplace_back(zone->create_link("link2", {100})); SECTION("Get route: no UP link") { zone->add_route(host1->get_netpoint(), nullptr, nullptr, nullptr, links, true); zone->add_route(nullptr, host2->get_netpoint(), nullptr, nullptr, links2, false); double lat; - simgrid::kernel::routing::RouteCreationArgs route; + simgrid::kernel::routing::Route route; zone->get_local_route(host2->get_netpoint(), host1->get_netpoint(), &route, &lat); } @@ -116,14 +129,14 @@ TEST_CASE("kernel::routing::StarZone: Get routes: assert", "[.][assert]") zone->add_route(host1->get_netpoint(), nullptr, nullptr, nullptr, links, false); zone->add_route(host2->get_netpoint(), nullptr, nullptr, nullptr, links2, true); double lat; - simgrid::kernel::routing::RouteCreationArgs route; + simgrid::kernel::routing::Route route; zone->get_local_route(host2->get_netpoint(), host1->get_netpoint(), &route, &lat); } } TEST_CASE("kernel::routing::StarZone: Adding routes (hosts): valid", "") { - EngineWrapper e("test"); + simgrid::s4u::Engine e("test"); auto* zone = new simgrid::kernel::routing::StarZone("test"); auto* netpoint = new simgrid::kernel::routing::NetPoint("netpoint1", simgrid::kernel::routing::NetPoint::Type::Host); @@ -147,7 +160,7 @@ TEST_CASE("kernel::routing::StarZone: Adding routes (hosts): valid", "") TEST_CASE("kernel::routing::StarZone: Adding routes (netzones): valid", "") { - EngineWrapper e("test"); + simgrid::s4u::Engine e("test"); auto* zone = new simgrid::kernel::routing::StarZone("test"); auto* netpoint = new simgrid::kernel::routing::NetPoint("netpoint1", simgrid::kernel::routing::NetPoint::Type::Host); auto* gw = new simgrid::kernel::routing::NetPoint("gw1", simgrid::kernel::routing::NetPoint::Type::Router); @@ -159,115 +172,122 @@ TEST_CASE("kernel::routing::StarZone: Adding routes (netzones): valid", "") TEST_CASE("kernel::routing::StarZone: Get routes (hosts)", "") { - /* workaround to initialize things, they must be done in this particular order */ - EngineWrapper e("test"); + simgrid::s4u::Engine e("test"); auto* zone = new simgrid::kernel::routing::StarZone("test"); - surf_network_model_init_LegrandVelho(); - surf_cpu_model_init_Cas01(); const auto* host1 = zone->create_host("netpoint1", {100}); const auto* host2 = zone->create_host("netpoint2", {100}); SECTION("Get route: no shared link") { - std::vector links; - links.push_back(zone->create_link("link1", {100})->set_latency(10)->get_impl()); - std::vector links2; - links2.push_back(zone->create_link("link2", {200})->set_latency(20)->get_impl()); + std::vector links; + links.emplace_back(zone->create_link("link1", {100})->set_latency(10)); + std::vector links2; + links2.emplace_back(zone->create_link("link2", {200})->set_latency(20)); zone->add_route(host1->get_netpoint(), nullptr, nullptr, nullptr, links, true); zone->add_route(host2->get_netpoint(), nullptr, nullptr, nullptr, links2, true); zone->seal(); double lat = 0.0; - simgrid::kernel::routing::RouteCreationArgs route; + simgrid::kernel::routing::Route route; zone->get_local_route(host1->get_netpoint(), host2->get_netpoint(), &route, &lat); REQUIRE(lat == 30); - REQUIRE(route.gw_src == nullptr); - REQUIRE(route.gw_dst == nullptr); - REQUIRE(route.link_list.size() == 2); - REQUIRE(route.link_list[0]->get_name() == "link1"); - REQUIRE(route.link_list[1]->get_name() == "link2"); + REQUIRE(route.gw_src_ == nullptr); + REQUIRE(route.gw_dst_ == nullptr); + REQUIRE(route.link_list_.size() == 2); + REQUIRE(route.link_list_[0]->get_name() == "link1"); + REQUIRE(route.link_list_[1]->get_name() == "link2"); } SECTION("Get route: shared link(backbone)") { - auto* backbone = zone->create_link("backbone", {1000})->set_latency(100)->get_impl(); - std::vector links; - links.push_back(zone->create_link("link1", {100})->set_latency(10)->get_impl()); - links.push_back(backbone); - std::vector links2; - links2.push_back(zone->create_link("link2", {200})->set_latency(20)->get_impl()); - links2.push_back(backbone); + auto* backbone = zone->create_link("backbone", {1000})->set_latency(100); + std::vector links; + links.emplace_back(zone->create_link("link1", {100})->set_latency(10)); + links.emplace_back(backbone); + std::vector links2; + links2.emplace_back(zone->create_link("link2", {200})->set_latency(20)); + links2.emplace_back(backbone); zone->add_route(host1->get_netpoint(), nullptr, nullptr, nullptr, links, true); zone->add_route(host2->get_netpoint(), nullptr, nullptr, nullptr, links2, true); zone->seal(); double lat = 0.0; - simgrid::kernel::routing::RouteCreationArgs route; + simgrid::kernel::routing::Route route; zone->get_local_route(host1->get_netpoint(), host2->get_netpoint(), &route, &lat); REQUIRE(lat == 130); - REQUIRE(route.link_list.size() == 3); - REQUIRE(route.link_list[0]->get_name() == "link1"); - REQUIRE(route.link_list[1]->get_name() == "backbone"); - REQUIRE(route.link_list[2]->get_name() == "link2"); + REQUIRE(route.link_list_.size() == 3); + REQUIRE(route.link_list_[0]->get_name() == "link1"); + REQUIRE(route.link_list_[1]->get_name() == "backbone"); + REQUIRE(route.link_list_[2]->get_name() == "link2"); } SECTION("Get route: loopback") { - auto* backbone = zone->create_link("backbone", {1000})->set_latency(100)->get_impl(); - std::vector links; - links.push_back(zone->create_link("link1", {100})->set_latency(10)->get_impl()); - links.push_back(backbone); + auto* backbone = zone->create_link("backbone", {1000})->set_latency(100); + std::vector links; + links.emplace_back(zone->create_link("link1", {100})->set_latency(10)); + links.emplace_back(backbone); zone->add_route(host1->get_netpoint(), host1->get_netpoint(), nullptr, nullptr, links, true); zone->seal(); double lat = 0.0; - simgrid::kernel::routing::RouteCreationArgs route; + simgrid::kernel::routing::Route route; zone->get_local_route(host1->get_netpoint(), host1->get_netpoint(), &route, &lat); REQUIRE(lat == 110); - REQUIRE(route.link_list.size() == 2); - REQUIRE(route.link_list[0]->get_name() == "link1"); - REQUIRE(route.link_list[1]->get_name() == "backbone"); + REQUIRE(route.link_list_.size() == 2); + REQUIRE(route.link_list_[0]->get_name() == "link1"); + REQUIRE(route.link_list_[1]->get_name() == "backbone"); } } TEST_CASE("kernel::routing::StarZone: Get routes (netzones)", "") { - /* workaround to initialize things, they must be done in this particular order */ - EngineWrapper e("test"); + simgrid::s4u::Engine e("test"); auto* zone = new simgrid::kernel::routing::StarZone("test"); - surf_network_model_init_LegrandVelho(); - surf_cpu_model_init_Cas01(); - - auto* subzone1 = - (new simgrid::kernel::routing::NetPoint("subzone1", simgrid::kernel::routing::NetPoint::Type::NetZone)) - ->set_englobing_zone(zone); - auto* subzone2 = - (new simgrid::kernel::routing::NetPoint("subzone2", simgrid::kernel::routing::NetPoint::Type::NetZone)) - ->set_englobing_zone(zone); - auto* router1 = new simgrid::kernel::routing::NetPoint("router1", simgrid::kernel::routing::NetPoint::Type::Router); - auto* router2 = new simgrid::kernel::routing::NetPoint("router2", simgrid::kernel::routing::NetPoint::Type::Router); + + auto* subzone1 = new simgrid::kernel::routing::StarZone("subzone1"); + subzone1->set_parent(zone); + auto* subzone2 = new simgrid::kernel::routing::StarZone("subzone2"); + subzone2->set_parent(zone); + + auto* router1 = subzone1->create_router("router1"); + auto* router2 = subzone2->create_router("router2"); SECTION("Get route: netzone") { - std::vector links; - links.push_back(zone->create_link("link1", {100})->set_latency(10)->get_impl()); - std::vector links2; - links2.push_back(zone->create_link("link2", {200})->set_latency(20)->get_impl()); - zone->add_route(subzone1, nullptr, router1, nullptr, links, true); - zone->add_route(subzone2, nullptr, router2, nullptr, links2, true); + std::vector links; + links.emplace_back(zone->create_link("link1", {100})->set_latency(10)); + std::vector links2; + links2.emplace_back(zone->create_link("link2", {200})->set_latency(20)); + zone->add_route(subzone1->get_netpoint(), nullptr, router1, nullptr, links, true); + zone->add_route(subzone2->get_netpoint(), nullptr, router2, nullptr, links2, true); zone->seal(); double lat = 0.0; - simgrid::kernel::routing::RouteCreationArgs route; - zone->get_local_route(subzone1, subzone2, &route, &lat); + simgrid::kernel::routing::Route route; + zone->get_local_route(subzone1->get_netpoint(), subzone2->get_netpoint(), &route, &lat); REQUIRE(lat == 30); - REQUIRE(route.gw_src == router1); - REQUIRE(route.gw_dst == router2); - REQUIRE(route.link_list.size() == 2); - REQUIRE(route.link_list[0]->get_name() == "link1"); - REQUIRE(route.link_list[1]->get_name() == "link2"); + REQUIRE(route.gw_src_ == router1); + REQUIRE(route.gw_dst_ == router2); + REQUIRE(route.link_list_.size() == 2); + REQUIRE(route.link_list_[0]->get_name() == "link1"); + REQUIRE(route.link_list_[1]->get_name() == "link2"); + } +} + +TEST_CASE("kernel::routing::StarZone: mix new routes and hosts", "") +{ + simgrid::s4u::Engine e("test"); + auto* zone = simgrid::s4u::create_star_zone("test"); + + const simgrid::s4u::Link* link = zone->create_link("my_link", 1e6)->seal(); + for (int i = 0; i < 10; i++) { + std::string cpu_name = "CPU" + std::to_string(i); + const simgrid::s4u::Host* cpu = zone->create_host(cpu_name, 1e9)->seal(); + REQUIRE_NOTHROW( + zone->add_route(cpu->get_netpoint(), nullptr, nullptr, nullptr, {simgrid::s4u::LinkInRoute(link)}, true)); } }