JamesMurkin commented on PR #1419:
URL: 
https://github.com/apache/pulsar-client-go/pull/1419#issuecomment-3253616082

   OK I ended up updating TestPeekMessageWithProperties with a 
batched/non-batched variant which should well cover this scenario.
    - It also now validates the batch number property gets set when using 
batched mode
   
   I'm not sure if you'll like the style, I could split them into separate 
tests but from a functionality point of view I think it'll work.
   
   I had to use the same "cheat" as the other batched unit test, but I've made 
it more explicit / not fail if you pause it in a debugger.
   
   Let me know if you want something different :) .


-- 
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]

Reply via email to