Skip to content

Commit b548118

Browse files
committed
Refs #20165. Apply suggestions
Signed-off-by: Ricardo González Moreno <[email protected]>
1 parent 5b5eaeb commit b548118

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

src/cpp/fastdds/builtin/type_lookup_service/TypeLookupReplyListener.cpp

+1
Original file line numberDiff line numberDiff line change
@@ -218,6 +218,7 @@ void TypeLookupReplyListener::check_get_types_reply(
218218
}
219219
else
220220
{
221+
typelookup_manager_->notify_callbacks(RETCODE_NO_DATA, type_id);
221222
EPROSIMA_LOG_WARNING(TYPELOOKUP_SERVICE_REPLY_LISTENER,
222223
"Received reply with no types.");
223224
register_result = RETCODE_ERROR;

test/unittest/dds/xtypes/type_lookup_service/TypeLookupServiceTests.cpp

+3-2
Original file line numberDiff line numberDiff line change
@@ -618,6 +618,7 @@ TEST_F(TypeLookupServiceTests, TypeLookupReplyListener_wrong_replyid)
618618
tlm_->async_get_type_requests_.emplace(valid_sampleidentity(), tidws);
619619

620620
TypeLookup_Reply reply;
621+
reply.header().relatedRequestId(valid_sampleidentity());
621622
reply.return_value()._default();
622623
ReplyWithServerGUID reply_with_guid;
623624
reply_with_guid.reply = reply;
@@ -632,7 +633,7 @@ TEST_F(TypeLookupServiceTests, TypeLookupReplyListener_wrong_replyid)
632633
TEST_F(TypeLookupServiceTests, TypeLookupReplyListener_getTypes_empty)
633634
{
634635
MockLogConsumer* log_consumer =
635-
setup_expect_log_msg("TYPELOOKUP_SERVICE_REPLY_LISTENER", "Reply no contains any type.");
636+
setup_expect_log_msg("TYPELOOKUP_SERVICE_REPLY_LISTENER", "Received reply with no types.");
636637

637638
TypeIdentfierWithSize tidws;
638639
tlm_->async_get_type_requests_.emplace(valid_sampleidentity(), tidws);
@@ -852,7 +853,7 @@ TEST_F(TypeLookupServiceTests, TypeLookupRequestListener_getTypes_empty)
852853
{
853854
MockLogConsumer* log_consumer =
854855
setup_expect_log_msg("TYPELOOKUP_SERVICE_REQUEST_LISTENER",
855-
"Request no contains any type identifier.");
856+
"Received request with no type identifiers.");
856857

857858
TypeIdentfierWithSize tidws;
858859
tlm_->async_get_type_requests_.emplace(valid_sampleidentity(), tidws);

0 commit comments

Comments
 (0)