Skip to content

MINOR: renamed testAsyncConsumerClassicConsumerSubscribeInvalidTopicCanUnsubscribe to align test case#20407

Merged
frankvicky merged 1 commit into
apache:trunkfrom
kirktrue:minor-rename-misnamed-test-method
Aug 26, 2025
Merged

MINOR: renamed testAsyncConsumerClassicConsumerSubscribeInvalidTopicCanUnsubscribe to align test case#20407
frankvicky merged 1 commit into
apache:trunkfrom
kirktrue:minor-rename-misnamed-test-method

Conversation

@kirktrue

@kirktrue kirktrue commented Aug 26, 2025

Copy link
Copy Markdown
Contributor

Simple test method rename.

Reviewers: TengYao Chi frankvicky@apache.org

…anUnsubscribe to testAsyncConsumerSubscribeInvalidTopicCanUnsubscribe
@github-actions github-actions Bot added triage PRs from the community tests Test fixes (including flaky tests) clients small Small PRs labels Aug 26, 2025
@frankvicky frankvicky changed the title MINOR: renamed testAsyncConsumerClassicConsumerSubscribeInvalidTopicC anUnsubscribe to testAsyncConsumerSubscribeInvalidTopicCanUnsubscribe MINOR: renamed testAsyncConsumerClassicConsumerSubscribeInvalidTopicCanUnsubscribe to align test case Aug 26, 2025
@frankvicky
frankvicky merged commit 4e0d8c9 into apache:trunk Aug 26, 2025
26 checks passed
@github-actions github-actions Bot removed the triage PRs from the community label Aug 27, 2025
@kirktrue
kirktrue deleted the minor-rename-misnamed-test-method branch September 1, 2025 22:37
eduwercamacaro pushed a commit to littlehorse-enterprises/kafka that referenced this pull request Nov 12, 2025
…anUnsubscribe to align test case (apache#20407)

`testAsyncConsumerClassicConsumerSubscribeInvalidTopicCanUnsubscribe` does not align with the test case. This patch renames the test name to describe the test case more precisely.
Reviewers: TengYao Chi <frankvicky@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-approved clients small Small PRs tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants