see-quick commented on PR #12692: URL: https://github.com/apache/kafka/pull/12692#issuecomment-1264624416
> Thanks for the PR. I don't understand the goal of the `toString` tests since they basically check that the toString doesn't have `@`? The main purpose of such test cases is to check that it does print something useful. Meaning not printing object reference from the inherited superclass (i.e., `java.lang.Object@7fbe847c`). If you do not see any value in such test cases I can remove them. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org