rename MPTokenHolder to Holder

XRPLF · Nov 9, 2024 · b34cd1e · b34cd1e
1 parent 4133b2e
commit b34cd1e
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 34 deletions.
diff --git a/xrpl4j-core/src/main/java/org/xrpl/xrpl4j/codec/binary/types/AmountType.java b/xrpl4j-core/src/main/java/org/xrpl/xrpl4j/codec/binary/types/AmountType.java
@@ -24,6 +24,7 @@
 import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.databind.node.TextNode;
+import com.google.common.base.Strings;
 import com.google.common.primitives.UnsignedLong;
 import org.xrpl.xrpl4j.codec.addresses.ByteUtils;
 import org.xrpl.xrpl4j.codec.addresses.UnsignedByte;
@@ -319,7 +320,7 @@ public boolean isMpt() {
    */
   public boolean isPositive() {
     // 2nd bit in 1st byte is set to 1 for positive amounts
-    return toHex().startsWith("00000000000000", 2) || (toBytes()[0] & 0x40) > 0;
+    return (toBytes()[0] & 0x40) > 0;
   }
 
 }
diff --git a/xrpl4j-core/src/main/resources/definitions.json b/xrpl4j-core/src/main/resources/definitions.json
@@ -2143,7 +2143,7 @@
       }
     ],
     [
-      "MPTokenHolder",
+      "Holder",
       {
         "nth": 11,
         "isVLEncoded": true,

diff --git a/xrpl4j-core/src/test/java/org/xrpl/xrpl4j/codec/binary/types/BaseSerializerTypeTest.java b/xrpl4j-core/src/test/java/org/xrpl/xrpl4j/codec/binary/types/BaseSerializerTypeTest.java
@@ -57,8 +57,9 @@ void fixtureTests(ValueTest fixture) throws IOException {
     } else {
       SerializedType<?> serialized = serializedType.fromJson(value);
       if (fixture.type().equals("Amount")) {
-        assertThat(((AmountType) serialized).isPositive()).isEqualTo(!fixture.isNegative());
-        assertThat(((AmountType) serialized).isNative()).isEqualTo(fixture.isNative());
+        AmountType amountType = (AmountType) serialized;
+        assertThat(amountType.isPositive()).isEqualTo(!fixture.isNegative());
+        assertThat(amountType.isNative()).isEqualTo(fixture.isNative());
       }
       assertThat(serialized.toHex()).isEqualTo(fixture.expectedHex());
     }

diff --git a/xrpl4j-core/src/test/resources/data-driven-tests.json b/xrpl4j-core/src/test/resources/data-driven-tests.json
@@ -2524,21 +2524,6 @@
       "is_negative": false,
       "exponent": -15
     },
-    {
-      "test_json": {
-        "currency": "USD",
-        "value": "0",
-        "issuer": "rrrrrrrrrrrrrrrrrrrrBZbvji"
-      },
-      "significant_digits": 1,
-      "type_id": 6,
-      "is_native": false,
-      "mantissa": "0000000000000000",
-      "type": "Amount",
-      "expected_hex": "800000000000000000000000000000000000000055534400000000000000000000000000000000000000000000000001",
-      "is_negative": false,
-      "exponent": -15
-    },
     {
       "test_json": {
         "currency": "USD",
@@ -2569,21 +2554,6 @@
       "is_negative": false,
       "exponent": -15
     },
-    {
-      "test_json": {
-        "currency": "USD",
-        "value": "0.0",
-        "issuer": "rrrrrrrrrrrrrrrrrrrrBZbvji"
-      },
-      "significant_digits": 1,
-      "type_id": 6,
-      "is_native": false,
-      "mantissa": "0000000000000000",
-      "type": "Amount",
-      "expected_hex": "800000000000000000000000000000000000000055534400000000000000000000000000000000000000000000000001",
-      "is_negative": false,
-      "exponent": -15
-    },
     {
       "test_json": {
         "currency": "USD",