epugh commented on code in PR #2820:
URL: https://github.com/apache/solr/pull/2820#discussion_r1824242674
##########
solr/packaging/test/test_snapshots.bats:
##########
@@ -59,15 +59,15 @@ teardown() {
@test "snapshot list" {
solr snapshot-create -c films --snapshot-name snapshot3 --solr-url
http://localhost:${SOLR_PORT}
- run solr snapshot-list -c films -url http://localhost:${SOLR_PORT}/solr
+ run solr snapshot-list -c films -s http://localhost:${SOLR_PORT}/solr
Review Comment:
i checked hte other bats tests, and yes, we normalize away the /solr. I am
going to add a comment though that it's here to confirm that we continue to
handle trailing `/solr` properly.
--
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
