Fix test_info_by_topic

Signed-off-by: Ivan Santiago Paunovic <[email protected]>
ros2 · Feb 4, 2020 · d14d52a · d14d52a
1 parent 8f09c8e
commit d14d52a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/rcl/test/rcl/test_info_by_topic.cpp b/rcl/test/rcl/test_info_by_topic.cpp
@@ -299,7 +299,7 @@ TEST_F(CLASSNAME(TestInfoByTopicFixture, RMW_IMPLEMENTATION),
   if (!is_fastrtps) {
     GTEST_SKIP();
   }
-  rmw_qos_profile_t default_qos_profile;
+  rmw_qos_profile_t default_qos_profile = rmw_qos_profile_default;
   default_qos_profile.history = RMW_QOS_POLICY_HISTORY_KEEP_LAST;
   default_qos_profile.depth = 0;
   default_qos_profile.reliability = RMW_QOS_POLICY_RELIABILITY_BEST_EFFORT;
@@ -341,7 +341,7 @@ TEST_F(CLASSNAME(TestInfoByTopicFixture, RMW_IMPLEMENTATION),
   ret = rcl_get_publishers_info_by_topic(&this->node,
       &allocator, fqdn.c_str(), false, &topic_endpoint_info_array_pub);
   EXPECT_EQ(ret, RCL_RET_OK) << rcl_get_error_string().str;
-  EXPECT_EQ(topic_endpoint_info_array_pub.count, 1u) << "Expected one publisher";
+  ASSERT_EQ(topic_endpoint_info_array_pub.count, 1u) << "Expected one publisher";
   rmw_topic_endpoint_info_t topic_endpoint_info_pub = topic_endpoint_info_array_pub.info_array[0];
   EXPECT_STREQ(topic_endpoint_info_pub.node_name, this->test_graph_node_name);
   EXPECT_STREQ(topic_endpoint_info_pub.node_namespace, "/");
@@ -353,7 +353,7 @@ TEST_F(CLASSNAME(TestInfoByTopicFixture, RMW_IMPLEMENTATION),
   ret = rcl_get_subscriptions_info_by_topic(&this->node,
       &allocator, fqdn.c_str(), false, &topic_endpoint_info_array_sub);
   EXPECT_EQ(ret, RCL_RET_OK) << rcl_get_error_string().str;
-  EXPECT_EQ(topic_endpoint_info_array_sub.count, 1u) << "Expected one subscription";
+  ASSERT_EQ(topic_endpoint_info_array_sub.count, 1u) << "Expected one subscription";
   rmw_topic_endpoint_info_t topic_endpoint_info_sub = topic_endpoint_info_array_sub.info_array[0];
   EXPECT_STREQ(topic_endpoint_info_sub.node_name, this->test_graph_node_name);
   EXPECT_STREQ(topic_endpoint_info_sub.node_namespace, "/");