-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[tools] fix TestRunMain test #12675
[tools] fix TestRunMain test #12675
Conversation
@rdhabalia:Thanks for your contribution. For this PR, do we need to update docs? |
1 similar comment
@rdhabalia:Thanks for your contribution. For this PR, do we need to update docs? |
Hi @rdhabalia do we need to update docs for this PR? |
(cherry picked from commit 24b0f4f)
Motivation
PulsarAdminTool returns with exist-code
0
with empty argument.TestRunMain::runMainNoArguments checks incorrect assertion. this change was introduced by #12581
Modification