RobertIndie commented on PR #1211: URL: https://github.com/apache/pulsar-client-go/pull/1211#issuecomment-2103990398
> do we really need a unit test case for the `String()` method? In my opinion, simply testing the `String()` method has less value. I think it's worth adding these tests. This is a public method exposed to users, and we need to ensure the correctness of the returned content. Moreover, other tests also rely on this String's output. We can see that this change has caused failures in other tests in the CI. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
