Expect rmw_destroy_wait_set to ret RMW_RET_INVALID_ARGUMENT (#234)

christophebedard-apexai · web-flow · commit d412eafcaee8 · 2024-06-13T15:12:01.000-04:00
Signed-off-by: Christophe Bedard &lt;christophe.bedard@apex.ai&gt;
diff --git a/test_rmw_implementation/test/test_wait_set.cpp b/test_rmw_implementation/test/test_wait_set.cpp
@@ -384,7 +384,7 @@ TEST_F(TestWaitSet, rmw_destroy_wait_set)
 {
   // Try to destroy a nullptr
   rmw_ret_t ret = rmw_destroy_wait_set(nullptr);
-  EXPECT_EQ(ret, RMW_RET_ERROR) << rcutils_get_error_string().str;
+  EXPECT_EQ(ret, RMW_RET_INVALID_ARGUMENT) << rcutils_get_error_string().str;
   rmw_reset_error();
 
   // Created a valid wait set

Original file line number	Diff line number	Diff line change
`@@ -384,7 +384,7 @@ TEST_F(TestWaitSet, rmw_destroy_wait_set)`
`384`	`384`	`{`
`385`	`385`	`// Try to destroy a nullptr`
`386`	`386`	`rmw_ret_t ret = rmw_destroy_wait_set(nullptr);`
`387`		`- EXPECT_EQ(ret, RMW_RET_ERROR) << rcutils_get_error_string().str;`
	`387`	`+ EXPECT_EQ(ret, RMW_RET_INVALID_ARGUMENT) << rcutils_get_error_string().str;`
`388`	`388`	`rmw_reset_error();`
`389`	`389`
`390`	`390`	`// Created a valid wait set`