Addressed PR comments

Signed-off-by: Guian Gumpac <guian.gumpac@improving.com>

Author: GumpacG

CHANGELOG.md

@@ -1,5 +1,4 @@
 #### Changes
-* Java: Add test for GlideString ([#2271](https://github.com/valkey-io/valkey-glide/pull/2271))
 * Java: Fetch server version using info command ([#2258](https://github.com/valkey-io/valkey-glide/pull/2258))
 * Node: Added binary variant for commands which have `Record` as input or output ([#2207](https://github.com/valkey-io/valkey-glide/pull/2207))
 * Node: Renamed `ReturnType` to `GlideReturnType` ([#2241](https://github.com/valkey-io/valkey-glide/pull/2241))
@@ -119,6 +118,7 @@
 * Core: Change FUNCTION STATS command to return multi node response for standalone mode ([#2117](https://github.com/valkey-io/valkey-glide/pull/2117))
 
 #### Fixes
+* Java: Java: Fix byte conversion to GlideString bug ([#2271](https://github.com/valkey-io/valkey-glide/pull/2271))
 * Java: Add overloads for XADD to allow duplicate entry keys ([#1970](https://github.com/valkey-io/valkey-glide/pull/1970))
 * Node: Fix ZADD bug where command could not be called with only the `changed` optional parameter ([#1995](https://github.com/valkey-io/valkey-glide/pull/1995))
 * Java: `XRange`/`XRevRange` should return `null` instead of `GlideException` when given a negative count ([#1920](https://github.com/valkey-io/valkey-glide/pull/1920))

java/client/src/main/java/glide/utils/ArrayTransformUtils.java

@@ -32,8 +32,7 @@ public static String[] convertMapToKeyValueStringArray(Map<String, ?> args) {
     }
 
     /**
-     * Converts a map of GlideString keys and values of any type in to an array of GlideStrings with
-     * alternating keys and values.
+     * Converts a map of GlideString keys and values to an array of GlideStrings.
      *
      * @param args Map of GlideString keys to values of GlideString.
      * @return Array of strings [key1, gs(value1.toString()), key2, gs(value2.toString()), ...].
@@ -65,10 +64,10 @@ public static String[] convertNestedArrayToKeyValueStringArray(String[][] args)
     }
 
     /**
-     * Converts a nested array of GlideString keys and values of any type in to an array of
-     * GlideStrings with alternating keys and values.
+     * Converts a nested array of GlideString keys and values in to an array of GlideStrings with
+     * alternating keys and values.
      *
-     * @param args Nested array of GlideString keys to values of any type to convert.
+     * @param args Nested array of GlideString keys and values to convert.
      * @return Array of strings [key1, gs(value1.toString()), key2, gs(value2.toString()), ...].
      */
     public static GlideString[] convertNestedArrayToKeyValueGlideStringArray(GlideString[][] args) {

java/integTest/src/test/java/glide/SharedCommandTests.java

@@ -996,20 +996,25 @@ public void non_UTF8_GlideString_test(BaseClient client) {
         String stringField = "field";
         Map<GlideString, GlideString> fieldValueMap = Map.of(gs(stringField), value);
 
-        // Non UTF-8 key and value
+        // Testing keys and values using byte[] that cannot be converted to UTF-8 Strings.
         assertEquals(OK, client.set(key, value).get());
         assertEquals(value, client.get(key).get());
 
-        // Non UTF-8 field value map for a set
+        // Testing set values using byte[] that cannot be converted to UTF-8 Strings.
         assertEquals(1, client.hset(hashKey, fieldValueMap).get());
         assertDeepEquals(new GlideString[] {gs(stringField)}, client.hkeys(hashKey).get());
         assertThrows(
                 ExecutionException.class, () -> client.hget(hashKey.toString(), stringField).get());
 
-        // Non UTF-8 set key and field value map
+        // Testing keys for a set using byte[] that cannot be converted to UTF-8 Strings returns bytes.
         assertEquals(1, client.hset(hashNonUTF8Key, fieldValueMap).get());
         assertDeepEquals(new GlideString[] {gs(stringField)}, client.hkeys(hashNonUTF8Key).get());
         assertEquals(value, client.hget(hashNonUTF8Key, gs(stringField)).get());
+
+        // Converting non UTF-8 bytes result to String returns a message.
+        assertEquals(
+                "Value not convertible to string: byte[] 13",
+                client.hget(hashNonUTF8Key, gs(stringField)).get().toString());
     }
 
     @SneakyThrows