You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by "Divij Vaidya (Jira)" <ji...@apache.org> on 2023/07/01 19:58:00 UTC
[jira] [Created] (KAFKA-15140) Improve TopicCommandIntegrationTest to be less flaky
Divij Vaidya created KAFKA-15140:
------------------------------------
Summary: Improve TopicCommandIntegrationTest to be less flaky
Key: KAFKA-15140
URL: https://issues.apache.org/jira/browse/KAFKA-15140
Project: Kafka
Issue Type: Test
Components: unit tests
Reporter: Divij Vaidya
*This is a good Jira for folks who are new to contributing to Kafka.*
Tests in TopicCommandIntegrationTest get flaky from time to time. The objective of the task is to make them more robust by doing the following:
1. Replace the usage createAndWaitTopic() method and other places where were are creating a topic (without waiting) with
TestUtils.createTopicWithAdmin(). The latter method already contains the functionality to create a topic and wait for metadata to sync up.
2. Replace the number 6 at places such as
"adminClient.createTopics(
Collections.singletonList(new NewTopic("foo_bar", 1, 6.toShort)))" with a meaningful constant.
3. Add logs if an assertion fails, for example, lines such as "
assertTrue(rows(0).startsWith(s"\tTopic: $testTopicName"), output)" should have a third argument which prints the actual output printed so that we can observe in the test logs on what was the output when assertion failed.
4. Replace occurrences of "\n" with System.lineSeparator() which is platform independent
5. We should wait for reassignment to complete whenever we are re-assigning partitions using alterconfig before we call describe to validate it. We could use
TestUtils.waitForAllReassignmentsToComplete()
--
This message was sent by Atlassian Jira
(v8.20.10#820010)