epugh commented on code in PR #2785:
URL: https://github.com/apache/solr/pull/2785#discussion_r1809151984
##########
solr/packaging/test/test_status.bats:
##########
@@ -33,9 +33,36 @@ teardown() {
assert_output --partial "No Solr nodes are running."
run solr start
run solr status
- assert_output --partial "Found 1 Solr nodes:"
+ assert_output --partial "running on port ${SOLR_PORT}"
run solr stop
run solr status
assert_output --partial "No Solr nodes are running."
+}
+
+@test "status with --solr-url from user" {
+ solr start
+ run solr status --solr-url http://localhost:${SOLR_PORT}
+ assert_output --partial "\"solr_home\":"
+ solr stop
+}
+
+@test "status with --port from user" {
+ solr start
+ run solr status --port ${SOLR_PORT}
+ assert_output --partial "running on port ${SOLR_PORT}"
+ solr stop
+}
+
+@test "status with invalid --solr-url from user" {
+ solr start
+ run solr status --solr-url http://invalidhost:${SOLR_PORT}
+ assert_output --partial "Solr at http://invalidhost:${SOLR_PORT} not online"
+ solr stop
+}
+@test "status with --short format" {
Review Comment:
One question, and this isn't about this backport... Is `--short` similar
to `--quiet`(which we use in bin/solr start)? Any other commands that *should*
have a `--short` style flag?
--
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
