Description: Avoid Catch2 assertions in multithreaded test helper
 Catch2 assertion macros are not thread-safe (prior catch2 3.12) and cause
 assertion failures in OutputRedirect when invoked concurrently across worker
 threads.
Author: Maximiliano Curia <maxy@debian.org>
Forwarded: no
--- a/Code/GraphMol/DistGeomHelpers/testDgeomHelpers.cpp
+++ b/Code/GraphMol/DistGeomHelpers/testDgeomHelpers.cpp
@@ -943,17 +943,17 @@ void runblock(const std::vector<std::sha
           .useLegacyImplementation = legacyETKDG};
       std::vector<int> cids =
           DGeomHelpers::EmbedMultipleConfs(*mol, 10, params);
-      REQUIRE(cids.size() == 10);
+      TEST_ASSERT(cids.size() == 10);
       std::unique_ptr<ForceFields::ForceField> field(
           UFF::constructForceField(*mol, 100, cids[0]));
-      REQUIRE(field);
+      TEST_ASSERT(field);
       field->initialize();
       double eng = field->calcEnergy();
       if (!feq(eng, energies[i])) {
         std::cerr << i << " iter " << j << " " << energies[i] << " != " << eng
                   << std::endl;
       }
-      CHECK(feq(eng, energies[i]));
+      TEST_ASSERT(feq(eng, energies[i]));
     }
   }
 }