Skip to content

Commit

Permalink
Update expected buffer length for serialized message after message ch…
Browse files Browse the repository at this point in the history
…anged

Signed-off-by: Jacob Perron <[email protected]>
  • Loading branch information
jacobperron committed May 1, 2019
1 parent 86f5df6 commit f5cffc2
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions test_communication/test/test_message_serialization.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,8 @@ TEST_F(CLASSNAME(TestMessageSerialization, RMW_IMPLEMENTATION), de_serialize_c)

ret = rmw_serialize(&bounded_sequences_msg_c, message_c_typesupport, &serialized_message_c);
EXPECT_EQ(RMW_RET_OK, ret);
EXPECT_EQ(196u, serialized_message_c.buffer_length); // measured from wireshark
// TODO(jacobperron): This gets changed to '196' after uncommenting fields in the test message
EXPECT_EQ(188u, serialized_message_c.buffer_length); // measured from wireshark

printf("serialized data length: %zu\n", serialized_message_c.buffer_length);
print_serialized_buffer(serialized_message_c, "serialized message c");
Expand Down Expand Up @@ -149,7 +150,8 @@ TEST_F(CLASSNAME(TestMessageSerialization, RMW_IMPLEMENTATION), de_serialize_cpp
ret =
rmw_serialize(&bounded_sequences_msg_cpp, message_cpp_typesupport, &serialized_message_cpp);
EXPECT_EQ(RMW_RET_OK, ret);
EXPECT_EQ(196u, serialized_message_cpp.buffer_length);
// TODO(jacobperron): This gets changed to '196' after uncommenting fields in the test message
EXPECT_EQ(188u, serialized_message_cpp.buffer_length);
print_serialized_buffer(serialized_message_cpp, "serialized message cpp");

test_msgs::msg::BoundedSequences bounded_sequences_cpp_reverse;
Expand Down

0 comments on commit f5cffc2

Please sign in to comment.