* randomize test port usage

Author: lpcvoid

src/server.hpp

@@ -283,6 +283,7 @@ class server {
                 close_and_free();
                 continue;
             }
+            _listener_sock->set_reuseaddr(true);
             _listener_sock->set_nonblocking(true); // we want to be able to join
             int32_t res = ::bind(_listener_sock->get_raw().value(), res_addrinfo->ai_addr, res_addrinfo->ai_addrlen);
             if (res < 0) {

tests/main.cpp

@@ -1,6 +1,14 @@
 #define DOCTEST_CONFIG_IMPLEMENT
+#include <random>
 #include "../doctest/doctest/doctest.h"
 
+// generate random port for server, which gets used in all tests
+std::random_device random_device;
+std::mt19937 gen(random_device());
+std::uniform_int_distribution<> distr(10000, 65000);
+
+uint16_t test_port = distr(gen);
+
 int main(int argc, char **argv)
 {
     doctest::Context context;

tests/test_client_server.cpp

@@ -10,10 +10,7 @@ using namespace std::chrono_literals;
 static const std::string hello_msg = "hello!";
 static const std::vector<uint8_t> client_message = {1, 2, 3};
 
-std::random_device random_device;
-std::mt19937 gen(random_device());
-std::uniform_int_distribution<> distr(10000, 65000);
-static uint16_t test_port = distr(gen);
+extern uint16_t test_port;
 
 std::string to_hex_array(const std::vector<uint8_t> &data)
 {

tests/test_client_server_disconnect.cpp

@@ -2,11 +2,10 @@
 #include "../src/netlib.hpp"
 
 using namespace std::chrono_literals;
-static const uint16_t test_port = 8889;
+extern uint16_t test_port;
 
 TEST_CASE("Test disconnect handling")
 {
-
     netlib::server server;
     netlib::client client;
     bool client_was_connected = false;

tests/test_large_data_transfer.cpp

@@ -4,10 +4,7 @@
 
 using namespace std::chrono_literals;
 
-std::random_device rd;
-std::uniform_int_distribution<uint16_t> dist(1000, 65000);
-std::mt19937 mt(rd());
-uint16_t test_port = dist(rd);
+extern uint16_t test_port;
 
 TEST_CASE("Large data transfer")
 {

tests/test_raii.cpp

@@ -3,7 +3,7 @@
 #include <atomic>
 
 using namespace std::chrono_literals;
-static const uint16_t test_port = 8888;
+extern uint16_t test_port;
 
 TEST_CASE("RAII test for client and server")
 {