From 94f34aee7633db809b588bb5b5f0756e1bfa48ef Mon Sep 17 00:00:00 2001
From: Eric Pugh <epugh@opensourceconnections.com>
Date: Sun, 29 Sep 2024 06:53:03 -0400
Subject: [PATCH] Fix method signature and improve tests

---
 solr/core/src/java/org/apache/solr/cli/ConfigTool.java | 8 +++++---
 solr/packaging/test/test_config.bats                   | 6 ++++++
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/solr/core/src/java/org/apache/solr/cli/ConfigTool.java b/solr/core/src/java/org/apache/solr/cli/ConfigTool.java
index 51d7b114de0..1ffdb5a401f 100644
--- a/solr/core/src/java/org/apache/solr/cli/ConfigTool.java
+++ b/solr/core/src/java/org/apache/solr/cli/ConfigTool.java
@@ -107,7 +107,7 @@ public List<Option> getOptions() {
             .longOpt("value")
             .argName("VALUE")
             .hasArg()
-            .required(false) // should be true when -v is removed.
+            .required(false)
             .desc("Set the property to this value; accepts JSON objects and strings.")
             .build(),
         SolrCLI.OPTION_SOLRURL,
@@ -128,7 +128,9 @@ public void runImpl(CommandLine cli) throws Exception {
     if (property == null) {
       throw new MissingArgumentException("'property' is a required option.");
     }
-    if (value == null) {
+
+    // value is required unless the property is one of the unset- type.
+    if (!action.contains("unset-") && value == null) {
       throw new MissingArgumentException("'value' is a required option.");
     }
     Map<String, Object> jsonObj = new HashMap<>();
@@ -147,7 +149,7 @@ public void runImpl(CommandLine cli) throws Exception {
     String updatePath = "/" + collection + "/config";
 
     echo("\nPOSTing request to Config API: " + solrUrl + updatePath);
-    echoIfVerbose(jsonBody, cli);
+    echoIfVerbose(jsonBody);
 
     try (SolrClient solrClient =
         SolrCLI.getSolrClient(
diff --git a/solr/packaging/test/test_config.bats b/solr/packaging/test/test_config.bats
index 9a7e5cb2855..f7f8ee7753e 100644
--- a/solr/packaging/test/test_config.bats
+++ b/solr/packaging/test/test_config.bats
@@ -41,8 +41,14 @@ teardown() {
 @test "setting property" {
   solr create -c COLL_NAME
 
+  run solr config -c COLL_NAME --action set-property --property updateHandler.autoCommit.maxDocs --solr-url http://localhost:${SOLR_PORT}
+  assert_output --partial "'value' is a required option."
+  
   run solr config -c COLL_NAME --action set-property --property updateHandler.autoCommit.maxDocs --value 100 --solr-url http://localhost:${SOLR_PORT}
   assert_output --partial "Successfully set-property updateHandler.autoCommit.maxDocs to 100"
+  
+  run solr config -c COLL_NAME --action unset-property --property updateHandler.autoCommit.maxDocs --solr-url http://localhost:${SOLR_PORT}
+  assert_output --partial "Successfully unset-property updateHandler.autoCommit.maxDocs"
 }
 
 @test "short form of setting property" {