Implement set(double) in NestedMultiOutput #1486

NestedMultiOutput does not implement CommandOutput.set(double number) which causes problem with RediSearch FT.INFO command
redis · Nov 4, 2020 · 01f56f7 · 01f56f7
1 parent 1cab728
commit 01f56f7
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/src/main/java/io/lettuce/core/output/NestedMultiOutput.java b/src/main/java/io/lettuce/core/output/NestedMultiOutput.java
@@ -32,6 +32,7 @@
  * @param <V> Value type.
  * @author Will Glozer
  * @author Mark Paluch
+ * @author Julien Ruaux
  */
 public class NestedMultiOutput<K, V> extends CommandOutput<K, V, List<Object>> {
 
@@ -57,6 +58,16 @@ public void set(long integer) {
         output.add(integer);
     }
 
+    @Override
+    public void set(double number) {
+
+        if (!initialized) {
+            output = new ArrayList<>();
+        }
+
+        output.add(number);
+    }
+
     @Override
     public void set(ByteBuffer bytes) {
 

diff --git a/src/test/java/io/lettuce/core/output/NestedMultiOutputUnitTests.java b/src/test/java/io/lettuce/core/output/NestedMultiOutputUnitTests.java
@@ -35,4 +35,15 @@ void nestedMultiError() {
         output.setError(StandardCharsets.US_ASCII.encode("Oops!"));
         assertThat(output.getError()).isNotNull();
     }
+
+    @Test
+    void nestedMultiDouble() {
+        NestedMultiOutput<String, String> output = new NestedMultiOutput<>(StringCodec.UTF8);
+        double value = 123.456;
+        output.set(value);
+        assertThat(output.get()).isNotNull();
+        assertThat(output.get()).size().isEqualTo(1);
+        assertThat(output.get().get(0)).isEqualTo(value);
+    }
+
 }