Check null fields too

Author: labkey-jeckels

src/org/labkey/test/tests/SampleTypeTest.java

@@ -48,6 +48,7 @@
 import org.labkey.test.params.FieldDefinition;
 import org.labkey.test.params.FieldDefinition.ColumnType;
 import org.labkey.test.params.FieldDefinition.LookupInfo;
+import org.labkey.test.params.FieldInfo;
 import org.labkey.test.params.experiment.SampleTypeDefinition;
 import org.labkey.test.util.DataRegionExportHelper;
 import org.labkey.test.util.DataRegionTable;
@@ -249,10 +250,13 @@ public void testCustomProperties()
         final String sampleTypeName = "SampleTypeCustomProps" + DOMAIN_TRICKY_CHARACTERS;
         FieldInfo stringCol1 = new FieldInfo(TestDataGenerator.randomFieldName("StringColPlain", "\""), ColumnType.String);
         FieldInfo stringCol2 = new FieldInfo(TestDataGenerator.randomFieldName("StringCol%"), ColumnType.String);
+        // Used to make sure the details page shows properties with null values
+        FieldInfo stringCol3 = new FieldInfo(TestDataGenerator.randomFieldName("StringColNull"), ColumnType.String);
         FieldInfo calcCol = new FieldInfo(TestDataGenerator.randomFieldName("CalcCol"), ColumnType.Calculation);
         final List<FieldDefinition> fields = List.of(
                 stringCol1.getFieldDefinition(),
                 stringCol2.getFieldDefinition(),
+                stringCol3.getFieldDefinition(),
                 calcCol.getFieldDefinition().setValueExpression("\"" + stringCol1.getName() + "\" || 'Concat'")
         );
 
@@ -273,9 +277,10 @@ public void testCustomProperties()
         var dataRegion = DataRegionTable.DataRegion(getDriver()).withName("Material").waitFor();
         checker().verifyEquals("Row data does not contain expected custom properties", "PlainValue", dataRegion.getDataAsText(0, stringCol1.getLabel()));
         checker().verifyEquals("Row data does not contain expected custom properties", "PercentValue", dataRegion.getDataAsText(0, stringCol2.getLabel()));
+        checker().verifyEquals("Row data does not contain expected custom properties", " ", dataRegion.getDataAsText(0, stringCol3.getLabel()));
         checker().verifyEquals("Row data does not contain expected custom properties", "PlainValueConcat", dataRegion.getDataAsText(0, calcCol.getLabel()));
         clickAndWait(Locator.linkWithText("CustomPropsSample"));
-        assertTextPresent(stringCol1.getLabel(), stringCol2.getLabel(), calcCol.getLabel(), "PlainValue", "PercentValue", "PlainValueConcat");
+        assertTextPresent(stringCol1.getLabel(), stringCol2.getLabel(), stringCol3.getLabel(), calcCol.getLabel(), "PlainValue", "PercentValue", "PlainValueConcat");
     }
 
         // Issue 47280: LKSM: Trailing/Leading whitespace in Source name won't resolve when deriving samples