dajac commented on pull request #8808:
URL: https://github.com/apache/kafka/pull/8808#issuecomment-644192842


   @tombentley I am not sure to follow what the `AppInfoParser` is doing 
here... Could you elaborate a bit more?
   
   Regarding option 2), I had a quick look at the implementation and it seems 
that it verifies that no errors are printed out but it does not really verify 
that the output is correct. For instance, if we would remove the 
`listAcls(adminClient)` at L115, we would not catch it, isn't it?
   
   Perhaps, we could add a small test for the `listAcls()` method which adds 
few ACLs, graps the output with `TestUtils.grabConsoleOutput` and verifies it. 
I think that it is probably enough given the scope of the change.


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to