Add moar tests

.github/workflows/build.yml

@@ -6,6 +6,7 @@ on:
     branches:
       - v2
       - v3
+      - develop
 
 jobs:
   build:

.github/workflows/pr-auto-approve.yml

@@ -1,13 +1,12 @@
 name: PR Auto-Approver
 
-# Auto-approve on projects that have a "secret" variable saying this is okay. This lets us set up a restricted PR
-# process on projects with just a single active developer. It prevents the developer from pushing directly to the
-# main branch (so enforces a particular workflow that our other GitHub Actions expect).
 on:
   pull_request:
     branches:
+      - main
       - v2
       - v3
+      - develop
 
 jobs:
   approve-pr:

src/main/java/info/freelibrary/iiif/presentation/v3/AbstractCanvas.java

@@ -4,23 +4,18 @@
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collections;
-import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
 import java.util.Optional;
-import java.util.Set;
 
 import com.fasterxml.jackson.annotation.JsonGetter;
 import com.fasterxml.jackson.annotation.JsonIgnore;
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.annotation.JsonInclude.Include;
 import com.fasterxml.jackson.annotation.JsonPropertyOrder;
 import com.fasterxml.jackson.annotation.JsonSetter;
-import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.JavaType;
-import com.fasterxml.jackson.databind.ser.impl.SimpleBeanPropertyFilter;
-import com.fasterxml.jackson.databind.ser.impl.SimpleFilterProvider;
 import com.fasterxml.jackson.databind.type.TypeFactory;
 
 import info.freelibrary.util.Constants;
@@ -44,7 +39,6 @@
 import info.freelibrary.iiif.presentation.v3.utils.JSON;
 import info.freelibrary.iiif.presentation.v3.utils.JsonKeys;
 import info.freelibrary.iiif.presentation.v3.utils.MessageCodes;
-import info.freelibrary.iiif.presentation.v3.utils.json.JsonParsingException;
 
 /**
  * A virtual container that represents a page or view and has content resources associated with it or with parts of it.
@@ -68,9 +62,6 @@ abstract class AbstractCanvas<T extends AbstractCanvas<T>> extends NavigableReso
     /** A temporal constant. */
     private static final String TEMPORAL = "temporal";
 
-    /** A Jackson serialization filter name for width and height. */
-    private static final String WIDTH_HEIGHT_FILTER = "JPv3WidthHeightFilter";
-
     /** A zero (non-existent) duration. */
     private static final float ZERO_DURATION = 0.0f;
 
@@ -142,6 +133,27 @@ protected AbstractCanvas(final String aID, final Label aLabel) {
         super(ResourceTypes.CANVAS, aID, aLabel, CanvasBehavior.class);
     }
 
+    @Override
+    public boolean equals(final Object aObject) {
+        if (this == aObject) {
+            return true;
+        }
+
+        if (aObject == null || getClass() != aObject.getClass()) {
+            return false;
+        }
+
+        if (aObject instanceof final AbstractCanvas<?> other) {
+            return Objects.equals(myDuration, other.myDuration) && Objects.equals(myHeight, other.myHeight) &&
+                    Objects.equals(myWidth, other.myWidth) &&
+                    Objects.equals(myOtherAnnotations, other.myOtherAnnotations) &&
+                    Objects.equals(myPaintingPageList, other.myPaintingPageList) &&
+                    Objects.equals(mySupplementingPageList, other.mySupplementingPageList) && super.equals(other);
+        }
+
+        return false;
+    }
+
     /**
      * Gets the duration of the canvas.
      *
@@ -228,6 +240,12 @@ public int getWidth() {
         return myWidth;
     }
 
+    @Override
+    public int hashCode() {
+        return Objects.hash(super.hashCode(), myDuration, myHeight, myWidth, myOtherAnnotations, myPaintingPageList,
+                mySupplementingPageList);
+    }
+
     /**
      * Sets the canvas' behaviors.
      *
@@ -252,11 +270,16 @@ public T setBehaviors(final Behavior... aBehaviorArray) {
     @JsonSetter(JsonKeys.BEHAVIOR)
     @SuppressWarnings({ JDK.UNCHECKED })
     public T setBehaviors(final List<Behavior> aBehaviorList) {
+        final T canvas;
+
         if (aBehaviorList instanceof final BehaviorList behaviorList) {
             behaviorList.checkType(CanvasBehavior.class, getClass());
+            canvas = super.setBehaviors(behaviorList);
+        } else {
+            canvas = super.setBehaviors(new BehaviorList(CanvasBehavior.class, aBehaviorList));
         }
 
-        return super.setBehaviors(aBehaviorList);
+        return canvas;
     }
 
     /**
@@ -406,37 +429,6 @@ public T setWidthHeight(final int aWidth, final int aHeight) {
         return (T) this;
     }
 
-    /**
-     * Converts the canvas to its string/JSON representation.
-     *
-     * @return A string representation of the canvas
-     */
-    @Override
-    public String toString() {
-        final SimpleFilterProvider filterProvider = new SimpleFilterProvider();
-        final Set<String> filtered = new HashSet<>();
-
-        // Don't write duration if it's zero
-        if (myDuration == ZERO_DURATION) {
-            filtered.add(JsonKeys.DURATION);
-        }
-
-        // Don't write width and height if they're both zero
-        if (myHeight == 0 && myWidth == 0) {
-            filtered.add(JsonKeys.HEIGHT);
-            filtered.add(JsonKeys.WIDTH);
-        }
-
-        // These are the things we filter when we serialize to JSON
-        filterProvider.addFilter(WIDTH_HEIGHT_FILTER, SimpleBeanPropertyFilter.serializeAllExcept(filtered));
-
-        try {
-            return JSON.getWriter(filterProvider).writeValueAsString(this);
-        } catch (final JsonProcessingException details) {
-            throw new JsonParsingException(details);
-        }
-    }
-
     /**
      * Gets the manifest context. The manifest can either have a single context or an array of contexts (Cf.
      * https://iiif.io/api/presentation/3.0/#46-linked-data-context-and-extensions)

src/main/java/info/freelibrary/iiif/presentation/v3/AbstractCanvasAnnotation.java

@@ -19,6 +19,7 @@
 import com.fasterxml.jackson.annotation.JsonPropertyOrder;
 import com.fasterxml.jackson.annotation.JsonSetter;
 
+import info.freelibrary.util.ListUtils;
 import info.freelibrary.util.Logger;
 import info.freelibrary.util.LoggerFactory;
 import info.freelibrary.util.warnings.JDK;
@@ -129,6 +130,26 @@ public boolean bodyHasChoice() {
         return myBodyHasChoice;
     }
 
+    @Override
+    public boolean equals(final Object aObject) {
+        if (this == aObject) {
+            return true;
+        }
+
+        if (aObject == null || getClass() != aObject.getClass()) {
+            return false;
+        }
+
+        if (aObject instanceof final AbstractCanvasAnnotation<?> other) {
+            return Objects.equals(myBodyHasChoice, other.myBodyHasChoice) &&
+                    Objects.equals(myMotivation, other.myMotivation) &&
+                    ListUtils.equals(myResources, other.myResources) && Objects.equals(myTarget, other.myTarget) &&
+                    Objects.equals(myTimeMode, other.myTimeMode) && super.equals(other);
+        }
+
+        return false;
+    }
+
     /**
      * Gets the content resources associated with this annotation.
      *
@@ -171,6 +192,11 @@ public Optional<TimeMode> getTimeMode() {
         return Optional.ofNullable(myTimeMode);
     }
 
+    @Override
+    public int hashCode() {
+        return Objects.hash(super.hashCode(), myBodyHasChoice, myMotivation, myResources, myTarget, myTimeMode);
+    }
+
     /**
      * Sets the annotation resource's behaviors. The supplied behaviors are checked for compatibility with the resource.
      *
@@ -193,11 +219,16 @@ public A setBehaviors(final Behavior... aBehaviorArray) {
     @Override
     @JsonSetter(JsonKeys.BEHAVIOR)
     public A setBehaviors(final List<Behavior> aBehaviorList) {
+        final A canvasAnno;
+
         if (aBehaviorList instanceof final BehaviorList behaviorList) {
             behaviorList.checkType(ResourceBehavior.class, getClass());
+            canvasAnno = super.setBehaviors(behaviorList);
+        } else {
+            canvasAnno = super.setBehaviors(new BehaviorList(ResourceBehavior.class, aBehaviorList));
         }
 
-        return super.setBehaviors(aBehaviorList);
+        return canvasAnno;
     }
 
     /**

src/main/java/info/freelibrary/iiif/presentation/v3/AbstractContentResource.java

@@ -19,6 +19,7 @@
 import com.fasterxml.jackson.databind.annotation.JsonDeserialize;
 import com.fasterxml.jackson.databind.annotation.JsonSerialize;
 
+import info.freelibrary.util.ListUtils;
 import info.freelibrary.util.warnings.JDK;
 
 import info.freelibrary.iiif.presentation.v3.annotations.WebAnnotation;
@@ -73,6 +74,24 @@ protected AbstractContentResource(final String aType, final String aID,
                 : MediaType.parse(aID).orElse(null);
     }
 
+    @Override
+    public boolean equals(final Object aObject) {
+        if (this == aObject) {
+            return true;
+        }
+
+        if (aObject == null || getClass() != aObject.getClass()) {
+            return false;
+        }
+
+        if (aObject instanceof final AbstractContentResource<?> other) {
+            return Objects.equals(myFormat, other.myFormat) && ListUtils.equals(myAnnotations, other.myAnnotations) &&
+                    ListUtils.equals(myLanguages, other.myLanguages) && super.equals(other);
+        }
+
+        return false;
+    }
+
     /**
      * Gets the content resource's annotations.
      *
@@ -113,6 +132,11 @@ public List<String> getLanguages() {
         return myLanguages;
     }
 
+    @Override
+    public int hashCode() {
+        return Objects.hash(super.hashCode(), myFormat, myAnnotations, myLanguages);
+    }
+
     /**
      * Sets the content resource's annotation pages from an array.
      *

src/main/java/info/freelibrary/iiif/presentation/v3/AbstractResource.java

@@ -19,6 +19,7 @@
 import com.fasterxml.jackson.databind.annotation.JsonDeserialize;
 
 import info.freelibrary.util.I18nRuntimeException;
+import info.freelibrary.util.ListUtils;
 import info.freelibrary.util.Logger;
 import info.freelibrary.util.LoggerFactory;
 import info.freelibrary.util.warnings.JDK;
@@ -154,6 +155,32 @@ protected AbstractResource(final String aType, final String aID, final Label aLa
         myLabel = Objects.requireNonNull(aLabel);
     }
 
+    @Override
+    public boolean equals(final Object aObject) {
+        if (this == aObject) {
+            return true;
+        }
+
+        if (aObject == null || getClass() != aObject.getClass()) {
+            return false;
+        }
+
+        if (aObject instanceof final AbstractResource<?> other) {
+            return Objects.equals(myType, other.myType) && Objects.equals(myBehaviorClass, other.myBehaviorClass) &&
+                    ListUtils.equals(myBehaviors, other.myBehaviors) &&
+                    ListUtils.equals(myHomepages, other.myHomepages) && Objects.equals(myID, other.myID) &&
+                    Objects.equals(myLabel, other.myLabel) && ListUtils.equals(myMetadata, other.myMetadata) &&
+                    ListUtils.equals(myPartOfs, other.myPartOfs) && ListUtils.equals(myProviders, other.myProviders) &&
+                    ListUtils.equals(myRenderings, other.myRenderings) &&
+                    Objects.equals(myRequiredStatement, other.myRequiredStatement) &&
+                    Objects.equals(myRights, other.myRights) && Objects.equals(mySeeAlsoRefs, other.mySeeAlsoRefs) &&
+                    ListUtils.equals(myServices, other.myServices) && Objects.equals(mySummary, other.mySummary) &&
+                    ListUtils.equals(myThumbnails, other.myThumbnails);
+        }
+
+        return false;
+    }
+
     /**
      * Gets the resource's behaviors.
      *
@@ -347,6 +374,13 @@ public String getType() {
         return myType;
     }
 
+    @Override
+    public int hashCode() {
+        return Objects.hash(myType, myBehaviorClass, myBehaviors, myHomepages, myID, myLabel, myMetadata, myPartOfs,
+                myProviders, myRenderings, myRequiredStatement, myRights, mySeeAlsoRefs, myServices, mySummary,
+                myThumbnails);
+    }
+
     @Override
     @JsonIgnore
     public T setBehaviors(final Behavior... aBehaviorArray) {

src/main/java/info/freelibrary/iiif/presentation/v3/AccompanyingCanvas.java

@@ -3,16 +3,12 @@
 
 import java.util.List;
 
-import com.fasterxml.jackson.core.JsonProcessingException;
-
 import info.freelibrary.util.warnings.Eclipse;
 import info.freelibrary.util.warnings.PMD;
 
 import info.freelibrary.iiif.presentation.v3.ids.Minter;
 import info.freelibrary.iiif.presentation.v3.properties.Label;
 import info.freelibrary.iiif.presentation.v3.properties.selectors.MediaFragmentSelector;
-import info.freelibrary.iiif.presentation.v3.utils.JSON;
-import info.freelibrary.iiif.presentation.v3.utils.json.JsonParsingException;
 
 /**
  * A single canvas that provides additional content that can be used while rendering the resource. Examples include: 1)
@@ -219,20 +215,4 @@ public final AccompanyingCanvas supplementWith(final String aCanvasRegion,
     protected Object getJsonContext() {
         return null;
     }
-
-    /**
-     * Returns an AccompanyingCanvas from its JSON representation.
-     *
-     * @param aJsonString A JSON serialization of an accompanying canvas
-     * @throws JsonParsingException If there is trouble reading an accompanying canvas from the supplied string
-     * @return The accompanying canvas
-     */
-    static AccompanyingCanvas fromJSON(final String aJsonString) {
-        try {
-            return JSON.getReader(AccompanyingCanvas.class).readValue(aJsonString);
-        } catch (final JsonProcessingException details) {
-            throw new JsonParsingException(details);
-        }
-    }
-
 }