fix leaks in tests

sketcherMinimizerMolecule.cpp

@@ -24,8 +24,9 @@ sketcherMinimizerMolecule::sketcherMinimizerMolecule()
 
 sketcherMinimizerMolecule::~sketcherMinimizerMolecule()
 {
-    for (auto ring : _rings) {
+    for (auto& ring : _rings) {
         delete ring;
+        ring = nullptr;
     }
 }
 

test/test_coordgen.cpp

@@ -15,6 +15,9 @@
 #include "maeparser/MaeConstants.hpp"
 #include "maeparser/Reader.hpp"
 
+// The sketcherMinimizerFragment class is not exposed in the library
+#include "../sketcherMinimizerFragment.cpp"
+
 using std::unordered_set;
 using namespace schrodinger;
 
@@ -498,6 +501,15 @@ BOOST_AUTO_TEST_CASE(testClockwiseOrderedNaN)
     sketcherMinimizerAtom* neigh1 = atoms.at(1);
     neigh1->coordinates = sketcherMinimizerPointF(std::nanf("name"), std::nanf("name"));
     const auto orderedNeighbors = center->clockwiseOrderedNeighbors();
+
+    // We usually allow the sketcher minimizer to clean up bonds & atoms,
+    // but we don't have a minimizer here
+    for (auto& bond : mol->getBonds()) {
+        delete bond;
+    }
+    for (auto& atom : atoms) {
+        delete atom;
+    }
 }
 
 
@@ -639,4 +651,8 @@ BOOST_AUTO_TEST_CASE(testCoordgenFragmenter)
     // Fragment containing atoms (9, 10)
     BOOST_TEST(atoms[8]->fragment->constrained == false);
     BOOST_TEST(atoms[8]->fragment->constrainedFlip == false);
+
+    for (auto& fragment : mol->_fragments) {
+        delete fragment;
+    }
 }