Merge pull request #1238 from pervazea/mem_leaks

Memory leak fixes for the unit tests

Author: pervazea

test/optimizer/optimizer_rule_test.cpp

@@ -162,6 +162,7 @@ TEST_F(OptimizerRuleTests, SimpleAssociativeRuleTest) {
   auto child_join_op = output_join->Children()[1]->Op().As<LogicalInnerJoin>();
   EXPECT_EQ(2, parent_join_op->join_predicates.size());
   EXPECT_EQ(0, child_join_op->join_predicates.size());
+  delete root_context;
 }
 
 TEST_F(OptimizerRuleTests, SimpleAssociativeRuleTest2) {
@@ -256,6 +257,7 @@ TEST_F(OptimizerRuleTests, SimpleAssociativeRuleTest2) {
   auto child_join_op = output_join->Children()[1]->Op().As<LogicalInnerJoin>();
   EXPECT_EQ(1, parent_join_op->join_predicates.size());
   EXPECT_EQ(1, child_join_op->join_predicates.size());
+  delete root_context;
 }
 
 }  // namespace test

test/planner/plan_util_test.cpp

@@ -163,6 +163,7 @@ TEST_F(PlanUtilTests, GetAffectedIndexesTest) {
 
   // no indexes are affected
   EXPECT_EQ(0, static_cast<int>(affected_indexes.size()));
+  txn_manager.CommitTransaction(txn);  
 }
 
 }  // namespace test