keep the event alive throught the assertion, preveiting the race. (#3099) (#3101)

mergify[bot] · fujitatomoya · web-flow · commit dca23fe11176 · 2026-03-18T08:30:29.000+09:00
(cherry picked from commit aadcb3b) Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com> Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
diff --git a/rclcpp/test/rclcpp/node_interfaces/test_node_graph.cpp b/rclcpp/test/rclcpp/node_interfaces/test_node_graph.cpp
@@ -131,7 +131,8 @@ TEST_F(TestNodeGraph, construct_from_node)
   EXPECT_NE(nullptr, node_graph()->get_graph_guard_condition());
 
   // get_graph_event is non-const
-  EXPECT_NE(nullptr, node()->get_node_graph_interface()->get_graph_event());
+  auto event = node()->get_node_graph_interface()->get_graph_event();
+  EXPECT_NE(nullptr, event);
   EXPECT_LE(1u, node_graph()->count_graph_users());
 }
 

Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,8 @@ TEST_F(TestNodeGraph, construct_from_node)`
`131`	`131`	`EXPECT_NE(nullptr, node_graph()->get_graph_guard_condition());`
`132`	`132`
`133`	`133`	`// get_graph_event is non-const`
`134`		`- EXPECT_NE(nullptr, node()->get_node_graph_interface()->get_graph_event());`
	`134`	`+ auto event = node()->get_node_graph_interface()->get_graph_event();`
	`135`	`+ EXPECT_NE(nullptr, event);`
`135`	`136`	`EXPECT_LE(1u, node_graph()->count_graph_users());`
`136`	`137`	`}`
`137`	`138`