Improved test_subscription_options (#1358)

* Improved test_subscription_options

Signed-off-by: ahcorde <ahcorde@gmail.com>

* used RCLCPP_EXPECT_THROW_EQ in test_subcription_options

Signed-off-by: ahcorde <ahcorde@gmail.com>

* make linters happy

Signed-off-by: ahcorde <ahcorde@gmail.com>
This commit is contained in:
Alejandro Hernández Cordero 2020-09-30 16:15:35 +02:00 committed by brawner
parent 8cc331f38c
commit cb6ac99a49

View file

@ -23,6 +23,8 @@
#include "rclcpp/node_options.hpp"
#include "rclcpp/subscription_options.hpp"
#include "../utils/rclcpp_gtest_macros.hpp"
using namespace std::chrono_literals;
namespace
@ -85,4 +87,17 @@ TEST_F(TestSubscriptionOptions, topic_statistics_options_node_default_mode) {
rclcpp::detail::resolve_enable_topic_statistics(
subscription_options,
*(node->get_node_base_interface())));
subscription_options.topic_stats_options.state = rclcpp::TopicStatisticsState::Disable;
EXPECT_FALSE(
rclcpp::detail::resolve_enable_topic_statistics(
subscription_options,
*(node->get_node_base_interface())));
subscription_options.topic_stats_options.state = static_cast<rclcpp::TopicStatisticsState>(5);
RCLCPP_EXPECT_THROW_EQ(
rclcpp::detail::resolve_enable_topic_statistics(
subscription_options,
*(node->get_node_base_interface())),
std::runtime_error("Unrecognized EnableTopicStatistics value"));
}