madrob commented on a change in pull request #736:
URL: https://github.com/apache/solr/pull/736#discussion_r826209848
##########
File path: solr/core/src/test/org/apache/solr/handler/TestContainerPlugin.java
##########
@@ -548,16 +473,28 @@ public void m2(SolrQueryRequest req, SolrQueryResponse
rsp) {
}
}
+ private Callable<V2Response> getPlugin(String path) {
Review comment:
I added randomization to either forceV2 or not, and manually tested it
both ways. I'm not sure what the intent was before, but let's improve test
coverage while we're in the area.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]