Merge pull request #1502 from lesserwhirls/nullAttrDap4

Dap4 null valued attributes

Author: lesserwhirls

dap4/build.gradle.kts

@@ -22,6 +22,8 @@ dependencies {
 
   testImplementation(project(":cdm-test-utils"))
 
+  testImplementation(libs.google.truth)
+
   testCompileOnly(libs.junit4)
 
   testRuntimeOnly(libs.junit5.platformLauncher)

dap4/src/main/java/dap4/core/dmr/DMRPrinter.java

@@ -1,6 +1,6 @@
 /*
- * Copyright 2012, UCAR/Unidata.
- * See the LICENSE file for more information.
+ * Copyright (c) 2012-2025 University Corporation for Atmospheric Research/Unidata
+ * See LICENSE for license information.
  */
 
 package dap4.core.dmr;
@@ -531,10 +531,14 @@ void printAttribute(DapAttribute attr) throws IOException {
         printer.marginPrintln(cs);
       }
     } else {
-      for (int i = 0; i < svec.length; i++) {
-        String s = Escape.entityEscape(svec[i], null);
-        String cs = String.format("<Value value=\"%s\"/>", s);
-        printer.marginPrintln(cs);
+      if (svec.length == 0) {
+        printer.marginPrintln("<Value/>");
+      } else {
+        for (String string : svec) {
+          String s = Escape.entityEscape(string, null);
+          String cs = String.format("<Value value=\"%s\"/>", s);
+          printer.marginPrintln(cs);
+        }
       }
     }
     printer.outdent();

dap4/src/test/java/dap4/core/dmr/TestDMRPrinterEdgeCases.java

@@ -0,0 +1,28 @@
+/*
+ * Copyright (c) 2025 University Corporation for Atmospheric Research/Unidata
+ * See LICENSE for license information.
+ */
+
+package dap4.core.dmr;
+
+import static com.google.common.truth.Truth.assertThat;
+
+import dap4.core.util.IndentWriter;
+import java.io.IOException;
+import java.io.StringWriter;
+import org.junit.Test;
+
+public class TestDMRPrinterEdgeCases {
+  @Test
+  public void testNullValueAttr() throws IOException {
+    DapAttribute attr = new DapAttribute("name", DapType.STRING);
+    attr.setValues(new String[] {});
+    DMRPrinter dmrPrinter = new DMRPrinter();
+    StringWriter sw = new StringWriter();
+    dmrPrinter.printer = new IndentWriter(sw);
+    dmrPrinter.printAttribute(attr);
+    String encodedAttribute = sw.toString();
+    assertThat(encodedAttribute).isNotEmpty();
+    assertThat(encodedAttribute).ignoringCase().contains("<value/>");
+  }
+}