Stopped reusing port numbers in case that's causing a test failure

Author: mpretty-cyro

tests/test_session_network.cpp

@@ -1221,7 +1221,7 @@ TEST_CASE("Network", "[network][build_path_if_needed]") {
 }
 
 TEST_CASE("Network", "[network][establish_connection]") {
-    auto test_server = create_test_server(5000);
+    auto test_server = create_test_server(5100);
     auto network = TestNetwork(std::nullopt, true, true, false);
     std::promise<std::pair<connection_info, std::optional<std::string>>> prom;
 
@@ -1248,7 +1248,7 @@ TEST_CASE("Network", "[network][check_request_queue_timeouts]") {
     // Test that it doesn't start checking for timeouts when the request doesn't have
     // a build paths timeout
     network.emplace(std::nullopt, true, true, false);
-    test_server.emplace(create_test_server(5001));
+    test_server.emplace(create_test_server(5201));
     network->send_onion_request(
             (*test_server)->node,
             to_vector("{\"method\":\"info\",\"params\":{}}"),
@@ -1265,7 +1265,7 @@ TEST_CASE("Network", "[network][check_request_queue_timeouts]") {
     // Test that it does start checking for timeouts when the request has a
     // paths build timeout
     network.emplace(std::nullopt, true, true, false);
-    test_server.emplace(create_test_server(5002));
+    test_server.emplace(create_test_server(5202));
     network->ignore_calls_to("build_path");
     network->send_onion_request(
             (*test_server)->node,
@@ -1283,7 +1283,7 @@ TEST_CASE("Network", "[network][check_request_queue_timeouts]") {
     // Test that it fails the request with a timeout if it has a build path timeout
     // and the path build takes too long
     network.emplace(std::nullopt, true, true, false);
-    test_server.emplace(create_test_server(5003));
+    test_server.emplace(create_test_server(5203));
     network->ignore_calls_to("build_path");
     network->send_onion_request(
             (*test_server)->node,
@@ -1307,7 +1307,7 @@ TEST_CASE("Network", "[network][check_request_queue_timeouts]") {
 }
 
 TEST_CASE("Network", "[network][send_request]") {
-    auto test_server = create_test_server(5000);
+    auto test_server = create_test_server(5300);
     auto network = TestNetwork(std::nullopt, true, true, false);
     std::promise<Result> prom;
 
@@ -1360,7 +1360,7 @@ TEST_CASE("Network", "[network][send_request]") {
 }
 
 TEST_CASE("Network", "[network][send_onion_request]") {
-    auto test_server = create_test_server(5000);
+    auto test_server = create_test_server(5400);
     auto network = TestNetwork(std::nullopt, true, true, false);
     auto [test_path_servers, test_path] = network.create_test_path();
     network.handle_onion_requests_as_plaintext = true;
@@ -1402,7 +1402,7 @@ TEST_CASE("Network", "[network][send_onion_request]") {
 }
 
 TEST_CASE("Network", "[network][c][network_send_onion_request]") {
-    auto test_server_cpp = create_test_server(5000);
+    auto test_server_cpp = create_test_server(5500);
     auto test_network = std::make_unique<TestNetwork>(std::nullopt, true, true, false);
     std::optional<std::pair<std::vector<std::shared_ptr<TestServer>>, onion_path>> test_path_data;
     test_path_data.emplace(test_network->create_test_path());