Skip to content

Commit 7ac3828

Browse files
committed
Refine some test
1 parent ea5d02c commit 7ac3828

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

tests/integration/TestKafkaConsumer.cc

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1885,8 +1885,11 @@ TEST(KafkaAutoCommitConsumer, CreateTopicAfterSubscribe)
18851885

18861886
KafkaAutoCommitConsumer consumer(KafkaTestUtility::GetKafkaClientCommonConfig());
18871887

1888+
bool errCbTriggered = false;
1889+
18881890
// The error would be triggered while consumer tries to subscribe a non-existed topic.
1889-
consumer.setErrorCallback([](const Error& error) {
1891+
consumer.setErrorCallback([&errCbTriggered](const Error& error) {
1892+
errCbTriggered = true;
18901893
KafkaTestUtility::DumpError(error);
18911894
EXPECT_EQ(RD_KAFKA_RESP_ERR_UNKNOWN_TOPIC_OR_PART, error.value());
18921895
});
@@ -1898,6 +1901,7 @@ TEST(KafkaAutoCommitConsumer, CreateTopicAfterSubscribe)
18981901
EXPECT_KAFKA_NO_THROW(consumer.subscribe({topic}));
18991902
std::cout << "[" << Utility::getCurrentTime() << "] Consumer just subscribed" << std::endl;
19001903

1904+
EXPECT_TRUE(errCbTriggered);
19011905
EXPECT_FALSE(consumer.assignment().empty());
19021906
}
19031907

0 commit comments

Comments
 (0)