Fix money text decoding for negative values (#1359)

Fixes #1358

Signed-off-by: Thomas Segismont <[email protected]>

Author: tsegismont

vertx-pg-client/src/main/java/io/vertx/pgclient/impl/codec/DataTypeCodec.java

@@ -1507,10 +1507,10 @@ private static Inet textDecodeInet(int index, int len, ByteBuf buff) {
 
   private static Money textDecodeMoney(int index, int len, ByteBuf buff) {
     String s = textDecodeVARCHAR(index, len, buff);
-    s = s.substring(1);
+    boolean negative = s.charAt(0) == '-';
     long integerPart = 0;
     int decimalPart = 0;
-    int idx = 0;
+    int idx = negative ? 2 : 1;
     char c;
     while (idx < s.length() && (c = s.charAt(idx++)) != '.') {
       if (c >= '0' && c <= '9') {
@@ -1523,7 +1523,7 @@ private static Money textDecodeMoney(int index, int len, ByteBuf buff) {
         decimalPart = decimalPart * 10 + (c - '0');
       }
     }
-    return new Money(integerPart, decimalPart);
+    return new Money(negative ? -integerPart : integerPart, decimalPart);
   }
 
   /**

vertx-pg-client/src/test/java/io/vertx/pgclient/data/MonetaryTypeSimpleCodecTest.java

@@ -25,14 +25,13 @@ public void testMoney(TestContext ctx) {
 
   @Test
   public void testNegativeMoney(TestContext ctx) {
-    // Does not look possible with text format
-    Money expected = new Money(1234, 56);
+    Money expected = new Money(-1234, 56);
     testDecodeGeneric(ctx, "-1234.56", "MONEY", "money", Tuple::getValue, Row::getValue, expected);
   }
 
   @Test
   public void testMoneyArray(TestContext ctx) {
-    Money expected = new Money(1234, 56);
-    testDecodeGenericArray(ctx, "ARRAY ['1234.56' :: MONEY]", "money", Tuple::getValue, Row::getValue, expected);
+    Money[] expected = {new Money(1234, 56), new Money(-1234, 56)};
+    testDecodeGenericArray(ctx, "ARRAY ['1234.56' :: MONEY , '-1234.56' :: MONEY]", "money", Tuple::getValue, Row::getValue, expected);
   }
 }