test: fix CI failures from parallel ctest and LeakSanitizer

The per-backend suite registrations run as concurrent ctest processes,
and unseeded std::rand() produces the same sequence in each one, so
the rand-named temp files in stream_file, random_access_file, and
tls_context collided across processes. Name them with a random_device
seed plus an atomic counter instead.

Tests that destroy the io_context with ops still parked abandon the
suspended coroutine frames: op destroy() must not resume or destroy
them, so the frames are unreachable at exit and LeakSanitizer reports
them. Gate those teardown tests behind COROSIO_TEST_HAS_ASAN.

Author: sgerbino

test/unit/context.hpp

@@ -93,4 +93,20 @@
     COROSIO_TEST_KQUEUE_(impl, name)              \
     COROSIO_TEST_SELECT_(impl, name)
 
+// Tests that destroy the io_context with ops still parked abandon the
+// suspended coroutine frames: op destroy() must not resume or destroy
+// them (resuming runs user code during teardown; destroying recurses
+// through the promise destructor). LeakSanitizer rightly reports those
+// frames, so such tests are skipped under ASAN.
+#if defined(__SANITIZE_ADDRESS__)
+#define COROSIO_TEST_HAS_ASAN 1
+#elif defined(__has_feature)
+#if __has_feature(address_sanitizer)
+#define COROSIO_TEST_HAS_ASAN 1
+#endif
+#endif
+#ifndef COROSIO_TEST_HAS_ASAN
+#define COROSIO_TEST_HAS_ASAN 0
+#endif
+
 #endif

test/unit/local_stream_socket.cpp

@@ -916,7 +916,10 @@ struct local_stream_socket_test
         testStopTokenAccept();
         testAcceptPendingConnection();
         testAcceptWithoutListen();
+#if !COROSIO_TEST_HAS_ASAN
+        // Abandons a parked coroutine frame by design; see context.hpp.
         testDestroyWithParkedAccept();
+#endif
 #endif
         testAcceptorOnClosedNoOp();
         testAcceptorBindClosedThrows();

test/unit/random_access_file.cpp

@@ -29,6 +29,8 @@
 #include <cstring>
 #include <filesystem>
 #include <fstream>
+#include <random>
+#include <atomic>
 #include <limits>
 #include <stop_token>
 #include <string>
@@ -47,14 +49,25 @@ namespace boost::corosio {
 
 namespace {
 
+// Unique across concurrently running per-backend test processes;
+// unseeded std::rand() yields the same sequence in every process.
+inline std::string
+unique_path_suffix()
+{
+    static unsigned const seed = std::random_device{}();
+    static std::atomic<unsigned> counter{0};
+    return std::to_string(seed) + "_"
+        + std::to_string(counter.fetch_add(1, std::memory_order_relaxed));
+}
+
 struct temp_file
 {
     std::filesystem::path path;
 
     temp_file(std::string_view prefix = "corosio_raf_test_")
     {
         path = std::filesystem::temp_directory_path()
-             / (std::string(prefix) + std::to_string(std::rand()));
+             / (std::string(prefix) + unique_path_suffix());
     }
 
     temp_file(std::string_view prefix, std::string_view contents)

test/unit/reactor_paths.cpp

@@ -1666,7 +1666,10 @@ struct reactor_paths_test
         testLocalDgramShutdownReceive();
         testStopTokenLocalStreamRead();
         testLocalStreamWriteEAGAIN();
+#if !COROSIO_TEST_HAS_ASAN
+        // Abandons parked coroutine frames by design; see context.hpp.
         testShutdownWithParkedOps();
+#endif
 #endif
     }
 };

test/unit/stream_file.cpp

@@ -29,6 +29,8 @@
 #include <cstring>
 #include <filesystem>
 #include <fstream>
+#include <random>
+#include <atomic>
 #include <limits>
 #include <stop_token>
 #include <string>
@@ -48,6 +50,17 @@ namespace boost::corosio {
 
 namespace {
 
+// Unique across concurrently running per-backend test processes;
+// unseeded std::rand() yields the same sequence in every process.
+inline std::string
+unique_path_suffix()
+{
+    static unsigned const seed = std::random_device{}();
+    static std::atomic<unsigned> counter{0};
+    return std::to_string(seed) + "_"
+        + std::to_string(counter.fetch_add(1, std::memory_order_relaxed));
+}
+
 // RAII helper that creates a temp file and removes it on destruction.
 struct temp_file
 {
@@ -56,7 +69,7 @@ struct temp_file
     temp_file(std::string_view prefix = "corosio_test_")
     {
         path = std::filesystem::temp_directory_path()
-             / (std::string(prefix) + std::to_string(std::rand()));
+             / (std::string(prefix) + unique_path_suffix());
     }
 
     // Create with initial contents

test/unit/tcp_acceptor.cpp

@@ -907,7 +907,10 @@ struct tcp_acceptor_test
 #ifndef _WIN32
         testAcceptPendingConnection();
         testAcceptWithoutListen();
+#if !COROSIO_TEST_HAS_ASAN
+        // Abandons a parked coroutine frame by design; see context.hpp.
         testDestroyWithParkedAccept();
+#endif
 #endif
     }
 };

test/unit/tls_context.cpp

@@ -20,13 +20,26 @@
 #include <cstdlib>
 #include <filesystem>
 #include <fstream>
+#include <random>
+#include <atomic>
 #include <string>
 #include <system_error>
 
 namespace boost::corosio {
 
 namespace {
 
+// Unique across concurrently running per-backend test processes;
+// unseeded std::rand() yields the same sequence in every process.
+inline std::string
+unique_path_suffix()
+{
+    static unsigned const seed = std::random_device{}();
+    static std::atomic<unsigned> counter{0};
+    return std::to_string(seed) + "_"
+        + std::to_string(counter.fetch_add(1, std::memory_order_relaxed));
+}
+
 // RAII helper that creates a temp file with given contents and removes it.
 struct temp_file
 {
@@ -35,7 +48,7 @@ struct temp_file
     temp_file(std::string_view prefix, std::string_view contents)
     {
         path = std::filesystem::temp_directory_path()
-             / (std::string(prefix) + std::to_string(std::rand()));
+             / (std::string(prefix) + unique_path_suffix());
         std::ofstream ofs(path, std::ios::binary);
         ofs.write(contents.data(), static_cast<std::streamsize>(contents.size()));
     }