niklas/rclcpp
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix time type issue in time unittest (#453)

Browse source 
Obviously it mean RCL_SYSTEM_TIME but not RCL_ROS_TIME in some test cases

Signed-off-by: jwang <jing.j.wang@intel.com>
This commit is contained in:
jwang11 2018-03-21 06:43:40 +08:00 • committed by Tully Foote
parent 9ce5aaa792
commit 947e3f7e67
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
rclcpp/test/test_time.cpp
View file

@ -63,7 +63,7 @@ TEST(TestTime, time_sources) {
EXPECT_NE(0, ros_now.sec); EXPECT_NE(0, ros_now.sec);
EXPECT_NE(0u, ros_now.nanosec); EXPECT_NE(0u, ros_now.nanosec);
rclcpp::Clock system_clock(RCL_ROS_TIME); rclcpp::Clock system_clock(RCL_SYSTEM_TIME);
Time system_now = system_clock.now(); Time system_now = system_clock.now();
EXPECT_NE(0, system_now.sec); EXPECT_NE(0, system_now.sec);
EXPECT_NE(0u, system_now.nanosec); EXPECT_NE(0u, system_now.nanosec);
@ -75,7 +75,7 @@ TEST(TestTime, time_sources) {
} }
TEST(TestTime, conversions) { TEST(TestTime, conversions) {
rclcpp::Clock system_clock(RCL_ROS_TIME); rclcpp::Clock system_clock(RCL_SYSTEM_TIME);
rclcpp::Time now = system_clock.now(); rclcpp::Time now = system_clock.now();
builtin_interfaces::msg::Time now_msg = now; builtin_interfaces::msg::Time now_msg = now;
@ -135,7 +135,7 @@ TEST(TestTime, operators) {
EXPECT_ANY_THROW((void)(system_time > steady_time)); EXPECT_ANY_THROW((void)(system_time > steady_time));
EXPECT_ANY_THROW((void)(system_time - steady_time)); EXPECT_ANY_THROW((void)(system_time - steady_time));
rclcpp::Clock system_clock(RCL_ROS_TIME); rclcpp::Clock system_clock(RCL_SYSTEM_TIME);
rclcpp::Clock steady_clock(RCL_STEADY_TIME); rclcpp::Clock steady_clock(RCL_STEADY_TIME);
rclcpp::Time now = system_clock.now(); rclcpp::Time now = system_clock.now();