fix: Fix bug in JsonUrlTestAppender.addCodePoint()

Author: dmaccormack

module/jsonurl-core/src/main/java/org/jsonurl/text/JsonUrlTextAppender.java

@@ -384,7 +384,11 @@ private static <T extends Appendable> void appendCodePoint(
             dest.append("%27");
 
         } else {
-            PercentCodec.encode(dest, codePoint, HEXENCODE_UNQUOTED, 0);
+            PercentCodec.encode(
+                dest,
+                codePoint,
+                optionAQF(options) ? HEXENCODE_AQF : HEXENCODE_UNQUOTED,
+                0);
         }
     }
 

module/jsonurl-core/src/test/java/org/jsonurl/text/JsonUrlStringBuilderTest.java

@@ -151,6 +151,21 @@ void testAddCodePointEncoded(char codePoint) throws IOException {
         assertCodePoint(expected, codePoint);
     }
 
+    @ParameterizedTest
+    @ValueSource(chars = {
+        '+', '(', ')', ',', ':', '!'
+    })
+    void testAddCodePointEscaped(char codePoint) throws IOException {
+        String actual = new JsonUrlStringBuilder(JsonUrlOption.AQF)
+            .addCodePoint(codePoint)
+            .build();
+
+        assertEquals(
+            "!" + codePoint,
+            actual,
+            "addCodePoint(AQF," + codePoint + ") is escaped");
+    }
+
     @Test
     void testEmptyObject() throws IOException {
         final String testName = "empty object";