Arguments.toString(): use Java compatible quoting, don't use comma as…

… delimiter
xzel23 · Jan 28, 2024 · ea43b3d · ea43b3d
1 parent c505cd4
commit ea43b3d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/utility/src/main/java/com/dua3/utility/options/Arguments.java b/utility/src/main/java/com/dua3/utility/options/Arguments.java
@@ -217,11 +217,11 @@ public String toString() {
                 if (entry.option instanceof Flag) {
                     fmt.format("  %s\n", entry.option.name());
                 } else {
-                    fmt.format("  %s %s\n", entry.option.name(), TextUtil.joinQuoted(entry.getParams()));
+                    fmt.format("  %s %s\n", entry.option.name(), TextUtil.joinQuoted(entry.getParams(), " "));
                 }
             }
             if (!positionalArgs().isEmpty()) {
-                fmt.format("  %s\n", TextUtil.joinQuoted(positionalArgs()));
+                fmt.format("  %s\n", TextUtil.joinQuoted(positionalArgs(), " "));
             }
             fmt.format("}");
             return fmt.toString();

diff --git a/utility/src/test/java/com/dua3/utility/options/ArgumentsParserTest.java b/utility/src/test/java/com/dua3/utility/options/ArgumentsParserTest.java
@@ -61,7 +61,7 @@ public void testFlag() {
         String expectedToString = """
                 Arguments{
                   --print
-                  "hello", "Bob"
+                  "hello" "Bob"
                 }""";
         assertEquals(expectedToString, args.toString());
     }