JsonInclude.Include.NON_NULL for all json attributes

Author: brokkoli71

src/main/java/dev/zarr/zarrjava/v2/Node.java

@@ -1,5 +1,6 @@
 package dev.zarr.zarrjava.v2;
 
+import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.databind.ObjectWriter;
 import com.fasterxml.jackson.datatype.jdk8.Jdk8Module;
@@ -19,6 +20,7 @@ static ObjectMapper makeObjectMapper() {
     ObjectMapper objectMapper = new ObjectMapper();
     objectMapper.registerModule(new Jdk8Module());
     objectMapper.registerSubtypes(CodecRegistry.getNamedTypes());
+    objectMapper.setDefaultPropertyInclusion(JsonInclude.Include.NON_NULL);
     return objectMapper;
   }
 

src/main/java/dev/zarr/zarrjava/v3/Node.java

@@ -1,5 +1,6 @@
 package dev.zarr.zarrjava.v3;
 
+import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.databind.ObjectWriter;
 import com.fasterxml.jackson.datatype.jdk8.Jdk8Module;
@@ -21,6 +22,7 @@ static ObjectMapper makeObjectMapper() {
     ObjectMapper objectMapper = new ObjectMapper();
     objectMapper.registerModule(new Jdk8Module());
     objectMapper.registerSubtypes(CodecRegistry.getNamedTypes());
+    objectMapper.setDefaultPropertyInclusion(JsonInclude.Include.NON_NULL);
     return objectMapper;
   }
 

src/main/java/dev/zarr/zarrjava/v3/codec/core/BytesCodec.java

@@ -2,7 +2,6 @@
 
 import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonIgnore;
-import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import dev.zarr.zarrjava.ZarrException;
 import dev.zarr.zarrjava.v3.codec.Codec;
@@ -12,7 +11,6 @@
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
-@JsonInclude(JsonInclude.Include.NON_NULL)
 public class BytesCodec extends dev.zarr.zarrjava.core.codec.core.BytesCodec implements Codec {
 
   @JsonIgnore

src/test/java/dev/zarr/zarrjava/ZarrV3Test.java

@@ -585,7 +585,7 @@ public void testCodecWithoutConfiguration() throws ZarrException, IOException {
             .withCodecs(CodecBuilder::withBytes)
             .build()
         );
-        Assertions.assertTrue(storeHandle.resolve("zarr.json").exists());
+        Assertions.assertTrue(storeHandle.resolve(ZARR_JSON).exists());
         Codec bytesCodec = array.metadata().codecs[0];
         Assertions.assertInstanceOf(BytesCodec.class, bytesCodec);
         Assertions.assertNull(((BytesCodec) bytesCodec).configuration);