ctubbsii commented on pull request #1832: URL: https://github.com/apache/accumulo/pull/1832#issuecomment-744630413
> I'm wondering if a more complete solution here is to create a test for the MasterClientServiceHander (the Master's API) as that is where the SecurityOperation.canFlush method is called. Many of those operations already have their own specific end-to-end ITs, as this class represents RPC endpoints. I *thought* the PermissionsIT was intended to specifically check the permissions of each of these endpoints, so that other tests can focus on testing the end-to-end behavior, but I could be wrong about the scope of the PermissionsIT. ---------------------------------------------------------------- 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: [email protected]
