refs #14111 - testrunner: enforce unique filename in ScopedFile to prevent conflicts in parallel execution [skip ci]

Author: firewave

test/helpers.cpp

@@ -50,12 +50,22 @@ class SuppressionList;
 
 const Settings SimpleTokenizer::s_settings;
 
+std::set<std::string> ScopedFile::s_scopedfiles;
+
 // TODO: better path-only usage
 ScopedFile::ScopedFile(std::string name, const std::string &content, std::string path)
     : mName(std::move(name))
     , mPath(Path::toNativeSeparators(std::move(path)))
     , mFullPath(Path::join(mPath, mName))
 {
+    if (mPath.empty() && mName.empty()) {
+        throw std::runtime_error("ScopedFile(" + mFullPath + ") - no file specified");
+    }
+
+    if (!s_scopedfiles.emplace(mFullPath).second) {
+        throw std::runtime_error("ScopedFile(" + mFullPath + ") - file not unique");
+    }
+
     if (!mPath.empty() && mPath != Path::getCurrentPath()) {
         if (Path::isDirectory(mPath))
             throw std::runtime_error("ScopedFile(" + mFullPath + ") - directory already exists");

test/helpers.h

@@ -157,6 +157,8 @@ class ScopedFile {
     const std::string mName;
     const std::string mPath;
     const std::string mFullPath;
+
+    static std::set<std::string> s_scopedfiles;
 };
 
 class PreprocessorHelper