[GitHub] [solr] TheSench edited a comment on pull request #296: SOLR-15626: Fix `config-read` permission.

[GitBox <gi...@apache.org>]

TheSench edited a comment on pull request #296:
URL: https://github.com/apache/solr/pull/296#issuecomment-922010030


   > Also, as far as "integration" tests, in the Solr world, we just put them all together! If it's a long running test, we label them with that attribute, but in terms of the code layout, they all just go together.
   
   Well, shall we open a new issue for three lack of permissions testing in general and keep this one as-is? If I have some spare time in the next few weeks, I don't mind taking a first crack at those tests under a new issue.
   
   I think my biggest current knowledge-gap is the list of possible endpoints to run tests for. Also, it was easy to map up some of the permissions to their endpoints, but others have a very roundabout way of going from an endpoint to where the permission is assigned. The documentation covers most of this, but a few of the built-in permissions were more vague to me.


-- 
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: issues-unsubscribe@solr.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org