epugh commented on a change in pull request #264:
URL: https://github.com/apache/solr/pull/264#discussion_r694316602
##########
File path:
solr/core/src/test/org/apache/solr/handler/admin/ShowFileRequestHandlerTest.java
##########
@@ -84,7 +84,7 @@ public void testDirList() throws SolrServerException,
IOException {
public void testGetRawFile() throws SolrServerException, IOException {
SolrClient client = getSolrClient();
//assertQ(req("qt", "/admin/file")); TODO file bug that SolrJettyTestBase
extends SolrTestCaseJ4
- QueryRequest request = new QueryRequest(params("file",
"managed-schema.xml"));
+ QueryRequest request = new QueryRequest(params("file", "managed-schema"));
Review comment:
Yeah, for now, till I write a better integration style test. I
discovered that while the unit tests might all pass perfectly, when I fired up
solr in cloud mode and non cloud mode it wasn't working ;-( So I went back
to the old way to make srue I was properly testing.
--
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-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
