laminelam commented on PR #1798: URL: https://github.com/apache/solr/pull/1798#issuecomment-1673715418
> Thanks for looking at this.. one thought, would you want to add some BATS test to confirm these fixes? They have been great for verifying a lot of the SolrCLI things... Down the road, I hope to eliminate the need for the `zkcli.sh` completely, in favour of using `bin/solr` commands... Having some BATS tests would help that migration ;-). Hi @epugh Thanks for the great suggestion. Didn't know about BATS. I have added a couple of BATS tests to test both 'solr,home" and "solrhome" options. Wondering if it's worth it keeping "solr.home" option, it's redundant and confusing -- 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