Add test case for expressions as logging param

aprotyas · aprotyas · commit 287b7edc9993 · 2021-11-10T01:21:37.000-05:00
Signed-off-by: Abrar Rahman Protyasha &lt;aprotyas@u.rochester.edu&gt;
diff --git a/rclcpp/test/rclcpp/test_logging.cpp b/rclcpp/test/rclcpp/test_logging.cpp
@@ -229,6 +229,12 @@ TEST_F(TestLoggingMacros, test_throttle) {
   }
 }
 
+TEST_F(TestLoggingMacros, test_parameter_expression) {
+  RCLCPP_DEBUG_STREAM(*&g_logger, "message");
+  EXPECT_EQ(1u, g_log_calls);
+  EXPECT_EQ("message", g_last_log_event.message);
+}
+
 bool log_function(rclcpp::Logger logger)
 {
   RCLCPP_INFO(logger, "successful log");

Original file line number	Diff line number	Diff line change
`@@ -229,6 +229,12 @@ TEST_F(TestLoggingMacros, test_throttle) {`
`229`	`229`	`}`
`230`	`230`	`}`
`231`	`231`
	`232`	`+TEST_F(TestLoggingMacros, test_parameter_expression) {`
	`233`	`+ RCLCPP_DEBUG_STREAM(*&g_logger, "message");`
	`234`	`+ EXPECT_EQ(1u, g_log_calls);`
	`235`	`+ EXPECT_EQ("message", g_last_log_event.message);`
	`236`	`+}`
	`237`	`+`
`232`	`238`	`bool log_function(rclcpp::Logger logger)`
`233`	`239`	`{`
`234`	`240`	`RCLCPP_INFO(logger, "successful log");`