[HOT FIX] Stabilize macOS CI tests: OpenMP linking, MPI env, per-test isolation (#669)

- MPI runs: propagate PPC_NUM_THREADS and OMP_NUM_THREADS via mpirun -x
…; fix
    gtest filters for _all_/_mpi_ across functional/perf runs.
  - Per-test isolation: add ScopedPerTestEnv that sets PPC_TEST_UID and
PPC_TEST_TMPDIR; integrated into functional and performance test
runners.

---------

Co-authored-by: Arseniy Obolenskiy <[email protected]>

Author: allnes

modules/util/include/func_test_util.hpp

@@ -61,6 +61,8 @@ class BaseRunFuncTests : public ::testing::TestWithParam<FuncTestParam<InType, O
 
     ValidateTestName(test_name);
 
+    const auto test_env_scope = ppc::util::test::MakePerTestEnvForCurrentGTest(test_name);
+
     if (IsTestDisabled(test_name)) {
       GTEST_SKIP();
     }

modules/util/include/perf_test_util.hpp

@@ -77,6 +77,8 @@ class BaseRunPerfTests : public ::testing::TestWithParam<PerfTestParam<InType, O
       GTEST_SKIP();
     }
 
+    const auto test_env_scope = ppc::util::test::MakePerTestEnvForCurrentGTest(test_name);
+
     task_ = task_getter(GetTestInputData());
     ppc::performance::Perf perf(task_);
     ppc::performance::PerfAttr perf_attr;

modules/util/include/util.hpp

@@ -1,10 +1,16 @@
 #pragma once
 
+#include <algorithm>
 #include <atomic>
+#include <cctype>
 #include <cstdint>
 #include <cstdlib>
+#include <filesystem>
 #include <memory>
+#include <sstream>
 #include <string>
+#include <string_view>
+#include <system_error>
 #include <typeinfo>
 #ifdef __GNUG__
 #  include <cxxabi.h>
@@ -17,6 +23,9 @@
 #  pragma warning(disable : 4459)
 #endif
 
+#include <gtest/gtest.h>
+
+#include <libenvpp/detail/environment.hpp>
 #include <nlohmann/json.hpp>
 
 /// @brief JSON namespace used for settings and config parsing.
@@ -90,4 +99,57 @@ inline std::shared_ptr<nlohmann::json> InitJSONPtr() {
 
 bool IsUnderMpirun();
 
+namespace test {
+
+[[nodiscard]] inline std::string SanitizeToken(std::string_view token_sv) {
+  std::string token{token_sv};
+  auto is_allowed = [](char c) {
+    return std::isalnum(static_cast<unsigned char>(c)) || c == '_' || c == '-' || c == '.';
+  };
+  std::ranges::replace(token, ' ', '_');
+  for (char &ch : token) {
+    if (!is_allowed(ch)) {
+      ch = '_';
+    }
+  }
+  return token;
+}
+
+class ScopedPerTestEnv {
+ public:
+  explicit ScopedPerTestEnv(const std::string &token)
+      : set_uid_("PPC_TEST_UID", token), set_tmp_("PPC_TEST_TMPDIR", CreateTmpDir(token)) {}
+
+ private:
+  static std::string CreateTmpDir(const std::string &token) {
+    namespace fs = std::filesystem;
+    const fs::path tmp = fs::temp_directory_path() / (std::string("ppc_test_") + token);
+    std::error_code ec;
+    fs::create_directories(tmp, ec);
+    (void)ec;
+    return tmp.string();
+  }
+
+  env::detail::set_scoped_environment_variable set_uid_;
+  env::detail::set_scoped_environment_variable set_tmp_;
+};
+
+[[nodiscard]] inline std::string MakeCurrentGTestToken(std::string_view fallback_name) {
+  const auto *unit = ::testing::UnitTest::GetInstance();
+  const auto *info = (unit != nullptr) ? unit->current_test_info() : nullptr;
+  std::ostringstream os;
+  if (info != nullptr) {
+    os << info->test_suite_name() << "." << info->name();
+  } else {
+    os << fallback_name;
+  }
+  return SanitizeToken(os.str());
+}
+
+inline ScopedPerTestEnv MakePerTestEnvForCurrentGTest(std::string_view fallback_name) {
+  return ScopedPerTestEnv(MakeCurrentGTestToken(fallback_name));
+}
+
+}  // namespace test
+
 }  // namespace ppc::util

scripts/run_tests.py

@@ -142,7 +142,7 @@ def run_processes(self, additional_mpi_args):
                 self.__run_exec(
                     mpi_running
                     + [str(self.work_dir / "ppc_func_tests")]
-                    + self.__get_gtest_settings(1, "_" + task_type)
+                    + self.__get_gtest_settings(1, "_" + task_type + "_")
                 )
 
     def run_performance(self):
@@ -152,13 +152,13 @@ def run_performance(self):
                 self.__run_exec(
                     mpi_running
                     + [str(self.work_dir / "ppc_perf_tests")]
-                    + self.__get_gtest_settings(1, "_" + task_type)
+                    + self.__get_gtest_settings(1, "_" + task_type + "_")
                 )
 
         for task_type in ["omp", "seq", "stl", "tbb"]:
             self.__run_exec(
                 [str(self.work_dir / "ppc_perf_tests")]
-                + self.__get_gtest_settings(1, "_" + task_type)
+                + self.__get_gtest_settings(1, "_" + task_type + "_")
             )