Merge pull request #275 from /issues/274

Issues/274 - Refactor JSON deserialization to use @JsonCreator to ensure that cl_type is parsed prior to setting any value fields in AbstractCLValueWithChildren and CLValueByteArray classes.
casper-network · May 20, 2024 · b848eda · b848eda
2 parents 67be93d + 1cfee7e
commit b848eda
Show file tree

Hide file tree

Showing 11 changed files with 264 additions and 23 deletions.
diff --git a/src/main/java/com/casper/sdk/model/clvalue/AbstractCLValueWithChildren.java b/src/main/java/com/casper/sdk/model/clvalue/AbstractCLValueWithChildren.java
@@ -20,6 +20,7 @@
  */
 @EqualsAndHashCode(callSuper = true)
 public abstract class AbstractCLValueWithChildren<T, P extends AbstractCLTypeWithChildren> extends AbstractCLValue<T, P> {
+
     protected abstract void setChildTypes(T value);
 
     /**
@@ -30,7 +31,7 @@ public abstract class AbstractCLValueWithChildren<T, P extends AbstractCLTypeWit
      */
     @SneakyThrows({ValueDeserializationException.class})
     protected void childTypesSet() {
-        if (!getBytes().isEmpty()) {
+        if (!getBytes().isEmpty() && getClType().isDeserializable()) {
             this.deserialize(new DeserializerBuffer(this.getBytes()));
         }
     }

diff --git a/src/main/java/com/casper/sdk/model/clvalue/CLValueAny.java b/src/main/java/com/casper/sdk/model/clvalue/CLValueAny.java
@@ -2,7 +2,9 @@
 
 import com.casper.sdk.annotation.ExcludeFromJacocoGeneratedReport;
 import com.casper.sdk.model.clvalue.cltype.CLTypeAny;
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonGetter;
+import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.annotation.JsonSetter;
 import dev.oak3.sbs4j.DeserializerBuffer;
 import dev.oak3.sbs4j.SerializerBuffer;
@@ -28,8 +30,19 @@
 @Setter
 @NoArgsConstructor
 public class CLValueAny extends AbstractCLValue<byte[], CLTypeAny> {
+
     private CLTypeAny clType = new CLTypeAny();
 
+    @JsonCreator
+    public CLValueAny(@JsonProperty("cl_type") final CLTypeAny clType,
+                      @JsonProperty("bytes") final String bytes,
+                      @JsonProperty("parsed") final Object parsed) throws ValueSerializationException {
+        setBytes(bytes);
+        setClType(clType);
+        setValue(Hex.decode(bytes));
+        setParsed(parsed);
+    }
+
     @JsonSetter("cl_type")
     @ExcludeFromJacocoGeneratedReport
     protected void setJsonClType(final CLTypeAny clType) {

diff --git a/src/main/java/com/casper/sdk/model/clvalue/CLValueByteArray.java b/src/main/java/com/casper/sdk/model/clvalue/CLValueByteArray.java
@@ -2,6 +2,7 @@
 
 import com.casper.sdk.annotation.ExcludeFromJacocoGeneratedReport;
 import com.casper.sdk.model.clvalue.cltype.CLTypeByteArray;
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import dev.oak3.sbs4j.DeserializerBuffer;
 import dev.oak3.sbs4j.SerializerBuffer;
@@ -27,9 +28,19 @@
 @Setter
 @NoArgsConstructor
 public class CLValueByteArray extends AbstractCLValue<byte[], CLTypeByteArray> {
+
     @JsonProperty("cl_type")
     private CLTypeByteArray clType = new CLTypeByteArray();
 
+    @JsonCreator
+    public CLValueByteArray(@JsonProperty("cl_type") final CLTypeByteArray clType,
+                            @JsonProperty("bytes") final String bytes,
+                            @JsonProperty("parsed") final String parsed) throws ValueSerializationException {
+        this.clType = clType;
+        setValue(Hex.decode(bytes));
+        setParsed(parsed);
+    }
+
     public CLValueByteArray(final byte[] value) throws ValueSerializationException {
         this.setValue(value);
         this.clType.setLength(value.length);

diff --git a/src/main/java/com/casper/sdk/model/clvalue/CLValueList.java b/src/main/java/com/casper/sdk/model/clvalue/CLValueList.java
@@ -2,8 +2,8 @@
 
 import com.casper.sdk.model.clvalue.cltype.CLTypeData;
 import com.casper.sdk.model.clvalue.cltype.CLTypeList;
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
-import com.fasterxml.jackson.annotation.JsonSetter;
 import dev.oak3.sbs4j.DeserializerBuffer;
 import dev.oak3.sbs4j.SerializerBuffer;
 import dev.oak3.sbs4j.exception.ValueSerializationException;
@@ -30,20 +30,29 @@
 @EqualsAndHashCode(callSuper = true)
 @NoArgsConstructor
 public class CLValueList extends AbstractCLValueWithChildren<List<? extends AbstractCLValue<?, ?>>, CLTypeList> {
+
     @JsonProperty("cl_type")
     private CLTypeList clType = new CLTypeList();
 
-    @JsonSetter("cl_type")
-    public void setClType(final CLTypeList clType) {
-        this.clType = clType;
-        childTypesSet();
+    @JsonCreator
+    public CLValueList(@JsonProperty("cl_type") final CLTypeList clType,
+                       @JsonProperty("bytes") final String bytes,
+                       @JsonProperty("parsed") final Object parsed)  {
+        setBytes(bytes);
+        setClType(clType);
+        setParsed(parsed);
     }
 
     public CLValueList(final List<? extends AbstractCLValue<?, ?>> value) throws ValueSerializationException {
         setChildTypes(value);
         this.setValue(value);
     }
 
+    public void setClType(final CLTypeList clType) {
+        this.clType = clType;
+        childTypesSet();
+    }
+
     @Override
     protected void serializeValue(final SerializerBuffer ser) throws ValueSerializationException {
 

diff --git a/src/main/java/com/casper/sdk/model/clvalue/CLValueMap.java b/src/main/java/com/casper/sdk/model/clvalue/CLValueMap.java
@@ -2,9 +2,9 @@
 
 import com.casper.sdk.model.clvalue.cltype.CLTypeData;
 import com.casper.sdk.model.clvalue.cltype.CLTypeMap;
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonIgnore;
 import com.fasterxml.jackson.annotation.JsonProperty;
-import com.fasterxml.jackson.annotation.JsonSetter;
 import dev.oak3.sbs4j.DeserializerBuffer;
 import dev.oak3.sbs4j.SerializerBuffer;
 import dev.oak3.sbs4j.exception.ValueSerializationException;
@@ -33,10 +33,19 @@
 @NoArgsConstructor
 public class CLValueMap extends
         AbstractCLValueWithChildren<Map<? extends AbstractCLValue<?, ?>, ? extends AbstractCLValue<?, ?>>, CLTypeMap> {
+
     @JsonProperty("cl_type")
     private CLTypeMap clType = new CLTypeMap();
 
-    @JsonSetter("cl_type")
+    @JsonCreator
+    public CLValueMap(@JsonProperty("cl_type") final CLTypeMap clType,
+                      @JsonProperty("bytes") final String bytes,
+                      @JsonProperty("parsed") final Object parsed) {
+        setBytes(bytes);
+        setClType(clType);
+        setParsed(parsed);
+    }
+
     public void setClType(CLTypeMap clType) {
         this.clType = clType;
         childTypesSet();

diff --git a/src/main/java/com/casper/sdk/model/clvalue/CLValueOption.java b/src/main/java/com/casper/sdk/model/clvalue/CLValueOption.java
@@ -3,8 +3,8 @@
 import com.casper.sdk.model.clvalue.cltype.AbstractCLTypeWithChildren;
 import com.casper.sdk.model.clvalue.cltype.CLTypeData;
 import com.casper.sdk.model.clvalue.cltype.CLTypeOption;
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
-import com.fasterxml.jackson.annotation.JsonSetter;
 import dev.oak3.sbs4j.DeserializerBuffer;
 import dev.oak3.sbs4j.SerializerBuffer;
 import dev.oak3.sbs4j.exception.ValueSerializationException;
@@ -29,10 +29,19 @@
 @EqualsAndHashCode(callSuper = true)
 @NoArgsConstructor
 public class CLValueOption extends AbstractCLValueWithChildren<Optional<AbstractCLValue<?, ?>>, CLTypeOption> {
+
     @JsonProperty("cl_type")
     private CLTypeOption clType = new CLTypeOption();
 
-    @JsonSetter("cl_type")
+    @JsonCreator
+    public CLValueOption(@JsonProperty("cl_type") final CLTypeOption clType,
+                         @JsonProperty("bytes") final String bytes,
+                         @JsonProperty("parsed") final Object parsed) {
+        setBytes(bytes);
+        setClType(clType);
+        setParsed(parsed);
+    }
+
     public void setClType(final CLTypeOption clType) {
         this.clType = clType;
         childTypesSet();

diff --git a/src/main/java/com/casper/sdk/model/clvalue/CLValueTuple1.java b/src/main/java/com/casper/sdk/model/clvalue/CLValueTuple1.java
@@ -2,8 +2,8 @@
 
 import com.casper.sdk.model.clvalue.cltype.CLTypeData;
 import com.casper.sdk.model.clvalue.cltype.CLTypeTuple1;
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
-import com.fasterxml.jackson.annotation.JsonSetter;
 import dev.oak3.sbs4j.DeserializerBuffer;
 import dev.oak3.sbs4j.SerializerBuffer;
 import dev.oak3.sbs4j.exception.ValueSerializationException;
@@ -30,10 +30,19 @@
 @EqualsAndHashCode(callSuper = true)
 @NoArgsConstructor
 public class CLValueTuple1 extends AbstractCLValueWithChildren<Unit<? extends AbstractCLValue<?, ?>>, CLTypeTuple1> {
+
     @JsonProperty("cl_type")
     private CLTypeTuple1 clType = new CLTypeTuple1();
 
-    @JsonSetter("cl_type")
+    @JsonCreator
+    public CLValueTuple1(@JsonProperty("cl_type") final CLTypeTuple1 clType,
+                         @JsonProperty("bytes") final String bytes,
+                         @JsonProperty("parsed") final Object parsed) {
+        setBytes(bytes);
+        setClType(clType);
+        setParsed(parsed);
+    }
+
     public void setClType(final CLTypeTuple1 clType) {
         this.clType = clType;
         childTypesSet();

diff --git a/src/main/java/com/casper/sdk/model/clvalue/CLValueTuple2.java b/src/main/java/com/casper/sdk/model/clvalue/CLValueTuple2.java
@@ -2,8 +2,8 @@
 
 import com.casper.sdk.model.clvalue.cltype.CLTypeData;
 import com.casper.sdk.model.clvalue.cltype.CLTypeTuple2;
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
-import com.fasterxml.jackson.annotation.JsonSetter;
 import dev.oak3.sbs4j.DeserializerBuffer;
 import dev.oak3.sbs4j.SerializerBuffer;
 import dev.oak3.sbs4j.exception.ValueSerializationException;
@@ -31,20 +31,29 @@
 @NoArgsConstructor
 public class CLValueTuple2
         extends AbstractCLValueWithChildren<Pair<? extends AbstractCLValue<?, ?>, ? extends AbstractCLValue<?, ?>>, CLTypeTuple2> {
+
     @JsonProperty("cl_type")
     private CLTypeTuple2 clType = new CLTypeTuple2();
 
-    @JsonSetter("cl_type")
-    public void setClType(CLTypeTuple2 clType) {
-        this.clType = clType;
-        childTypesSet();
+    @JsonCreator
+    public CLValueTuple2(@JsonProperty("cl_type") final CLTypeTuple2 clType,
+                         @JsonProperty("bytes") final String bytes,
+                         @JsonProperty("parsed") final Object parsed) {
+        setBytes(bytes);
+        setClType(clType);
+        setParsed(parsed);
     }
 
     public CLValueTuple2(Pair<? extends AbstractCLValue<?, ?>, ? extends AbstractCLValue<?, ?>> value) throws ValueSerializationException {
         setChildTypes(value);
         this.setValue(value);
     }
 
+    public void setClType(CLTypeTuple2 clType) {
+        this.clType = clType;
+        childTypesSet();
+    }
+
     @Override
     protected void serializeValue(final SerializerBuffer ser) throws ValueSerializationException {
         final SerializerBuffer serVal = new SerializerBuffer();

diff --git a/src/main/java/com/casper/sdk/model/clvalue/CLValueTuple3.java b/src/main/java/com/casper/sdk/model/clvalue/CLValueTuple3.java
@@ -2,8 +2,8 @@
 
 import com.casper.sdk.model.clvalue.cltype.CLTypeData;
 import com.casper.sdk.model.clvalue.cltype.CLTypeTuple3;
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
-import com.fasterxml.jackson.annotation.JsonSetter;
 import dev.oak3.sbs4j.DeserializerBuffer;
 import dev.oak3.sbs4j.SerializerBuffer;
 import dev.oak3.sbs4j.exception.ValueSerializationException;
@@ -31,20 +31,29 @@
 @NoArgsConstructor
 public class CLValueTuple3 extends
         AbstractCLValueWithChildren<Triplet<? extends AbstractCLValue<?, ?>, ? extends AbstractCLValue<?, ?>, ? extends AbstractCLValue<?, ?>>, CLTypeTuple3> {
+
     @JsonProperty("cl_type")
     private CLTypeTuple3 clType = new CLTypeTuple3();
 
-    @JsonSetter("cl_type")
-    public void setClType(final CLTypeTuple3 clType) {
-        this.clType = clType;
-        childTypesSet();
+    @JsonCreator
+    public CLValueTuple3(@JsonProperty("cl_type") final CLTypeTuple3 clType,
+                         @JsonProperty("bytes") final String bytes,
+                         @JsonProperty("parsed") final Object parsed) {
+        setBytes(bytes);
+        setClType(clType);
+        setParsed(parsed);
     }
 
     public CLValueTuple3(final Triplet<? extends AbstractCLValue<?, ?>, ? extends AbstractCLValue<?, ?>, ? extends AbstractCLValue<?, ?>> value) throws ValueSerializationException {
         setChildTypes(value);
         this.setValue(value);
     }
 
+    public void setClType(final CLTypeTuple3 clType) {
+        this.clType = clType;
+        childTypesSet();
+    }
+
     @Override
     protected void serializeValue(final SerializerBuffer ser) throws ValueSerializationException {
         final SerializerBuffer serVal = new SerializerBuffer();

diff --git a/src/test/java/com/casper/sdk/model/deploy/DeployDataTests.java b/src/test/java/com/casper/sdk/model/deploy/DeployDataTests.java
@@ -112,6 +112,7 @@ void validateDeployDataMapping_5() throws IOException, JSONException {
 
         JSONAssert.assertEquals(inputJson, expectedJson, JSONCompareMode.NON_EXTENSIBLE);
     }
+
     @Test
     void validateDeployDataMapping_6() throws IOException, JSONException {
         final String inputJson = getPrettyJson(loadJsonFromFile("deploy-samples/deploy-v6.json"));
@@ -129,6 +130,23 @@ void validateDeployDataMapping_6() throws IOException, JSONException {
         JSONAssert.assertEquals(inputJson, expectedJson, JSONCompareMode.NON_EXTENSIBLE);
     }
 
+    @Test
+    void validateDeployDataMapping_7() throws IOException, JSONException {
+        final String inputJson = getPrettyJson(loadJsonFromFile("deploy-samples/deploy-v7.json"));
+
+        LOGGER.debug("Original JSON: {}", inputJson);
+
+        final Deploy dd = OBJECT_MAPPER.readValue(inputJson, Deploy.class);
+
+        assertNotNull(dd);
+
+        final String expectedJson = getPrettyJson(dd);
+
+        LOGGER.debug("Serialized JSON: {}", expectedJson);
+
+        JSONAssert.assertEquals(inputJson, expectedJson, JSONCompareMode.NON_EXTENSIBLE);
+    }
+
     @Test
     void validateDeployResultMapping() throws IOException, JSONException {
         final String inputJson = getPrettyJson(loadJsonFromFile("deploy-samples/deploy-result.json"));
@@ -143,4 +161,4 @@ void validateDeployResultMapping() throws IOException, JSONException {
 
         JSONAssert.assertEquals(inputJson, expectedJson, JSONCompareMode.NON_EXTENSIBLE);
     }
-}
+}