Some work towards improving NullPointerExceptions to provide more con…

…text when missing data is the cause.
david-waltermire · Feb 22, 2024 · 368a999 · 368a999
1 parent 79470ac
commit 368a999
Show file tree

Hide file tree

Showing 7 changed files with 91 additions and 21 deletions.
diff --git a/.../java/gov/nist/secauto/metaschema/core/datatype/markup/flexmark/AbstractMarkupWriter.java b/.../java/gov/nist/secauto/metaschema/core/datatype/markup/flexmark/AbstractMarkupWriter.java
@@ -303,7 +303,7 @@ public void writeLink(
     }
 
     if (!node.getTitle().isBlank()) {
-      String title = ObjectUtils.requireNonNull(node.getTitle().unescape());
+      String title = ObjectUtils.notNull(node.getTitle().unescape());
       attributes.put("title", title);
     }
 

diff --git a/core/src/main/java/gov/nist/secauto/metaschema/core/model/xml/impl/ModelFactory.java b/core/src/main/java/gov/nist/secauto/metaschema/core/model/xml/impl/ModelFactory.java
@@ -131,8 +131,14 @@ private static Map<String, IAllowedValue> toAllowedValues(
     Map<String, IAllowedValue> allowedValues // NOPMD - intentional
         = new LinkedHashMap<>(xmlObject.sizeOfEnumArray());
     for (AllowedValueType xmlEnum : xmlObject.getEnumList()) {
+      String value = xmlEnum.getValue();
+      if (value == null) {
+        throw new IllegalStateException(String.format("Null value found in allowed value enumeration: %s",
+            xmlObject.xmlText()));
+      }
+
       IAllowedValue allowedValue = IAllowedValue.of(
-          ObjectUtils.requireNonNull(xmlEnum.getValue()),
+          value,
           MarkupStringConverter.toMarkupString(xmlEnum));
       allowedValues.put(allowedValue.getValue(), allowedValue);
     }

diff --git a/databind/src/main/java/gov/nist/secauto/metaschema/databind/DefaultBindingContext.java b/databind/src/main/java/gov/nist/secauto/metaschema/databind/DefaultBindingContext.java
@@ -145,6 +145,10 @@ public IBindingMatcher registerBindingMatcher(@NonNull IBoundDefinitionModelAsse
   @Override
   public IBindingMatcher registerBindingMatcher(@NonNull Class<?> clazz) {
     IBoundDefinitionModelComplex definition = getBoundDefinitionForClass(clazz);
+    if (definition == null) {
+      throw new IllegalArgumentException(String.format("Unable to find bound definition for class '%s'.",
+          clazz.getName()));
+    }
 
     try {
       IBoundDefinitionModelAssembly assemblyDefinition = IBoundDefinitionModelAssembly.class.cast(definition);
@@ -318,7 +322,8 @@ public <CLASS> CLASS deepCopy(@NonNull CLASS other, Object parentInstance) throw
     if (definition == null) {
       throw new IllegalStateException(String.format("Class '%s' is not bound", other.getClass().getName()));
     }
-    @SuppressWarnings("unchecked") CLASS retval = (CLASS) definition.deepCopyItem(other, parentInstance);
+    @SuppressWarnings("unchecked")
+    CLASS retval = (CLASS) definition.deepCopyItem(other, parentInstance);
     return retval;
   }
 }
diff --git a/...bind/src/main/java/gov/nist/secauto/metaschema/databind/io/json/MetaschemaJsonReader.java b/...bind/src/main/java/gov/nist/secauto/metaschema/databind/io/json/MetaschemaJsonReader.java
@@ -394,7 +394,14 @@ public Object readChoiceGroupItem(Object parentItem, IBoundInstanceModelChoiceGr
       JsonParser parser = getReader();
       ObjectNode node = parser.readValueAsTree();
 
-      JsonNode descriminatorNode = ObjectUtils.requireNonNull(node.get(instance.getJsonDiscriminatorProperty()));
+      String descriminatorProperty = instance.getJsonDiscriminatorProperty();
+      JsonNode descriminatorNode = node.get(descriminatorProperty);
+      if (descriminatorNode == null) {
+        throw new IllegalArgumentException(String.format(
+            "Unable to find descriminator property '%s' for object at '%s'.",
+            descriminatorProperty,
+            JsonUtil.toString(parser)));
+      }
       String discriminator = ObjectUtils.requireNonNull(descriminatorNode.asText());
 
       IBoundInstanceModelGroupedNamed actualInstance = instance.getGroupedModelInstance(discriminator);
@@ -432,7 +439,8 @@ private JsonKeyBodyHandler(
       @Override
       public void accept(IBoundDefinitionModelComplex definition, Object parentItem, IJsonProblemHandler problemHandler)
           throws IOException {
-        @SuppressWarnings("resource") JsonParser parser = getReader();
+        @SuppressWarnings("resource")
+        JsonParser parser = getReader();
         JsonUtil.assertCurrent(parser, JsonToken.FIELD_NAME);
 
         // the field will be the JSON key
@@ -469,7 +477,8 @@ public void accept(
           Object parentItem,
           IJsonProblemHandler problemHandler)
           throws IOException {
-        @SuppressWarnings("resource") JsonParser parser = getReader();
+        @SuppressWarnings("resource")
+        JsonParser parser = getReader();
 
         // advance past the start object
         JsonUtil.assertAndAdvance(parser, JsonToken.START_OBJECT);
@@ -593,7 +602,8 @@ public boolean handleUnknownProperty(IBoundDefinitionModelComplex definition, Ob
         if (foundJsonValueKey) {
           retval = delegate.handleUnknownProperty(definition, parentItem, fieldName, reader);
         } else {
-          @SuppressWarnings("resource") JsonParser parser = getReader();
+          @SuppressWarnings("resource")
+          JsonParser parser = getReader();
           // handle JSON value key
           String key = ObjectUtils.notNull(parser.getCurrentName());
           Object keyValue = jsonValueKyeFlag.getJavaTypeAdapter().parse(key);
@@ -693,7 +703,8 @@ public List<?> readList() throws IOException {
 
       IBoundInstanceModel instance = getCollectionInfo().getInstance();
 
-      @SuppressWarnings("PMD.UseConcurrentHashMap") Map<String, Object> items = new LinkedHashMap<>();
+      @SuppressWarnings("PMD.UseConcurrentHashMap")
+      Map<String, Object> items = new LinkedHashMap<>();
 
       // A map value is always wrapped in a START_OBJECT, since fields are used for
       // the keys
@@ -711,7 +722,12 @@ public List<?> readList() throws IOException {
         IBoundInstanceFlag jsonKey = instance.getItemJsonKey(item);
         assert jsonKey != null;
 
-        String key = ObjectUtils.requireNonNull(jsonKey.getValue(item)).toString();
+        Object keyValue = jsonKey.getValue(item);
+        if (keyValue == null) {
+          throw new IOException(String.format("Null JSON key value for definition '%s'",
+              jsonKey.getContainingDefinition().toCoordinates()));
+        }
+        String key = jsonKey.getJavaTypeAdapter().asString(keyValue);
         items.put(key, item);
 
         // the next item will be a FIELD_NAME, or we will encounter an END_OBJECT if all

diff --git a/...bind/src/main/java/gov/nist/secauto/metaschema/databind/io/json/MetaschemaJsonWriter.java b/...bind/src/main/java/gov/nist/secauto/metaschema/databind/io/json/MetaschemaJsonWriter.java
@@ -29,7 +29,6 @@
 import com.fasterxml.jackson.core.JsonGenerator;
 
 import gov.nist.secauto.metaschema.core.model.JsonGroupAsBehavior;
-import gov.nist.secauto.metaschema.core.util.ObjectUtils;
 import gov.nist.secauto.metaschema.databind.model.IBoundDefinitionModelAssembly;
 import gov.nist.secauto.metaschema.databind.model.IBoundDefinitionModelFieldComplex;
 import gov.nist.secauto.metaschema.databind.model.IBoundFieldValue;
@@ -128,8 +127,13 @@ public void writeItemFieldValue(Object parentItem, IBoundFieldValue fieldValue)
 
       String valueKeyName;
       if (jsonValueKey != null) {
+        Object keyValue = jsonValueKey.getValue(parentItem);
+        if (keyValue == null) {
+          throw new IOException(String.format("Null JSON value key value for definition '%s'",
+              jsonValueKey.getContainingDefinition().toCoordinates()));
+        }
         // this is the JSON value key case
-        valueKeyName = ObjectUtils.requireNonNull(jsonValueKey.getValue(parentItem)).toString();
+        valueKeyName = jsonValueKey.getJavaTypeAdapter().asString(keyValue);
       } else {
         valueKeyName = fieldValue.getParentFieldDefinition().getEffectiveJsonValueKeyName();
       }
@@ -253,8 +257,9 @@ public void writeChoiceGroupItem(Object item, IBoundInstanceModelChoiceGroup ins
       actualInstance.writeItem(item, this);
     }
 
-    private void writeScalarItem(Object item, IFeatureScalarItemValueHandler handler) throws IOException {
-      handler.getJavaTypeAdapter().writeJsonValue(ObjectUtils.requireNonNull(item), writer);
+    private void writeScalarItem(@NonNull Object item, @NonNull IFeatureScalarItemValueHandler handler)
+        throws IOException {
+      handler.getJavaTypeAdapter().writeJsonValue(item, writer);
     }
 
     private <T extends IBoundInstanceModelGroupedNamed> void writeDiscriminatorProperty(
@@ -297,8 +302,14 @@ private <T extends IFeatureComplexItemValueHandler & IBoundInstanceModel> void w
 
       IBoundInstanceFlag jsonKey = handler.getItemJsonKey(parentItem);
       if (jsonKey != null) {
+        Object keyValue = jsonKey.getValue(parentItem);
+        if (keyValue == null) {
+          throw new IOException(String.format("Null JSON key value for definition '%s'",
+              jsonKey.getContainingDefinition().toCoordinates()));
+        }
+
         // the field will be the JSON key value
-        String key = jsonKey.getJavaTypeAdapter().asString(ObjectUtils.requireNonNull(jsonKey.getValue(parentItem)));
+        String key = jsonKey.getJavaTypeAdapter().asString(keyValue);
         writer.writeFieldName(key);
 
         // next the value will be a start object
@@ -323,8 +334,14 @@ private <T extends IFeatureComplexItemValueHandler & IBoundInstanceModelGroupedN
       IBoundInstanceModelChoiceGroup choiceGroup = handler.getParentContainer();
       IBoundInstanceFlag jsonKey = choiceGroup.getItemJsonKey(parentItem);
       if (jsonKey != null) {
+        Object keyValue = jsonKey.getValue(parentItem);
+        if (keyValue == null) {
+          throw new IOException(String.format("Null JSON key value for definition '%s'",
+              jsonKey.getContainingDefinition().toCoordinates()));
+        }
+
         // the field will be the JSON key value
-        String key = jsonKey.getJavaTypeAdapter().asString(ObjectUtils.requireNonNull(jsonKey.getValue(parentItem)));
+        String key = jsonKey.getJavaTypeAdapter().asString(keyValue);
         writer.writeFieldName(key);
 
         // next the value will be a start object

diff --git a/...c/main/java/gov/nist/secauto/metaschema/databind/model/metaschema/impl/BindingModule.java b/...c/main/java/gov/nist/secauto/metaschema/databind/model/metaschema/impl/BindingModule.java
@@ -183,12 +183,20 @@ public URI getLocation() {
 
   @Override
   public MarkupLine getName() {
-    return ObjectUtils.requireNonNull(getBinding().getSchemaName());
+    MarkupLine retval = getBinding().getSchemaName();
+    if (retval == null) {
+      throw new IllegalStateException(String.format("The schema name is NULL for module '%s'.", getLocation()));
+    }
+    return retval;
   }
 
   @Override
   public String getVersion() {
-    return ObjectUtils.requireNonNull(getBinding().getSchemaVersion());
+    String retval = getBinding().getSchemaVersion();
+    if (retval == null) {
+      throw new IllegalStateException(String.format("The schema version is NULL for module '%s'.", getLocation()));
+    }
+    return retval;
   }
 
   @Override
@@ -198,17 +206,30 @@ public MarkupMultiline getRemarks() {
 
   @Override
   public String getShortName() {
-    return ObjectUtils.requireNonNull(getBinding().getShortName());
+    String retval = getBinding().getShortName();
+    if (retval == null) {
+      throw new IllegalStateException(String.format("The schema short name is NULL for module '%s'.", getLocation()));
+    }
+    return retval;
   }
 
   @Override
   public URI getXmlNamespace() {
-    return ObjectUtils.requireNonNull(getBinding().getNamespace());
+    URI retval = getBinding().getNamespace();
+    if (retval == null) {
+      throw new IllegalStateException(
+          String.format("The XML schema namespace is NULL for module '%s'.", getLocation()));
+    }
+    return retval;
   }
 
   @Override
   public URI getJsonBaseUri() {
-    return ObjectUtils.requireNonNull(getBinding().getJsonBaseUri());
+    URI retval = getBinding().getJsonBaseUri();
+    if (retval == null) {
+      throw new IllegalStateException(String.format("The JSON schema URI is NULL for module '%s'.", getLocation()));
+    }
+    return retval;
   }
 
   @NonNull

diff --git a/...c/test/java/gov/nist/secauto/metaschema/databind/model/AbstractBoundModelTestSupport.java b/...c/test/java/gov/nist/secauto/metaschema/databind/model/AbstractBoundModelTestSupport.java
@@ -76,7 +76,12 @@ protected IBindingContext getBindingContext() {
 
   @NonNull
   protected IBoundDefinitionModel registerClassBinding(@NonNull Class<?> clazz) {
-    return ObjectUtils.requireNonNull(getBindingContext().getBoundDefinitionForClass(clazz));
+    IBoundDefinitionModel definition = getBindingContext().getBoundDefinitionForClass(clazz);
+    if (definition == null) {
+      throw new IllegalArgumentException(String.format("Unable to find bound definition for class '%s'.",
+          clazz.getName()));
+    }
+    return definition;
   }
 
   @NonNull