From 18a6ef780bab325915ba522df401d855e88489c0 Mon Sep 17 00:00:00 2001 From: Shane Loretz Date: Tue, 5 Jun 2018 10:21:23 -0700 Subject: [PATCH] free memory after tests pass --- rcl/test/rcl/test_arguments.cpp | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/rcl/test/rcl/test_arguments.cpp b/rcl/test/rcl/test_arguments.cpp index 7d36673f4..bd28342b2 100644 --- a/rcl/test/rcl/test_arguments.cpp +++ b/rcl/test/rcl/test_arguments.cpp @@ -315,6 +315,7 @@ TEST_F(CLASSNAME(TestArgumentsFixture, RMW_IMPLEMENTATION), test_param_argument_ int parameter_filecount = rcl_arguments_get_param_files_count(&parsed_args); EXPECT_EQ(0, parameter_filecount); + EXPECT_EQ(RCL_RET_OK, rcl_arguments_fini(&parsed_args)); } TEST_F(CLASSNAME(TestArgumentsFixture, RMW_IMPLEMENTATION), test_param_argument_single) { @@ -336,6 +337,12 @@ TEST_F(CLASSNAME(TestArgumentsFixture, RMW_IMPLEMENTATION), test_param_argument_ ret = rcl_arguments_get_param_files(&parsed_args, alloc, ¶meter_files); ASSERT_EQ(RCL_RET_OK, ret) << rcl_get_error_string_safe(); EXPECT_STREQ("parameter_filepath", parameter_files[0]); + + for (int i = 0; i < parameter_filecount; ++i) { + alloc.deallocate(parameter_files[i], alloc.state); + } + alloc.deallocate(parameter_files, alloc.state); + EXPECT_EQ(RCL_RET_OK, rcl_arguments_fini(&parsed_args)); } TEST_F(CLASSNAME(TestArgumentsFixture, RMW_IMPLEMENTATION), test_param_argument_multiple) { @@ -359,4 +366,9 @@ TEST_F(CLASSNAME(TestArgumentsFixture, RMW_IMPLEMENTATION), test_param_argument_ ASSERT_EQ(RCL_RET_OK, ret) << rcl_get_error_string_safe(); EXPECT_STREQ("parameter_filepath1", parameter_files[0]); EXPECT_STREQ("parameter_filepath2", parameter_files[1]); + for (int i = 0; i < parameter_filecount; ++i) { + alloc.deallocate(parameter_files[i], alloc.state); + } + alloc.deallocate(parameter_files, alloc.state); + EXPECT_EQ(RCL_RET_OK, rcl_arguments_fini(&parsed_args)); }