Add more informative errors for null ElementCache

[labkey-tchad]

Rationale

Some components are able to end up with a null element cache. Hopefully, adding some null checks to Component.elementCache will help narrow down what is happening.

java.lang.NullPointerException: Cannot read field "editableGrid" because the return value of "org.labkey.test.components.biologics.PlateEditor.elementCache()" is null
	at org.labkey.test.components.biologics.PlateEditor.getEditableGrid(PlateEditor.java:240)
	at org.labkey.test.components.biologics.PlateEditor.lambda$4(PlateEditor.java:251)
	at org.labkey.test.util.Timer.waitFor(Timer.java:91)
	at org.labkey.test.WebDriverWrapper.waitFor(WebDriverWrapper.java:2315)
	at org.labkey.test.components.biologics.PlateEditor.doAndWaitForGridColumnUpdate(PlateEditor.java:247)
	at org.labkey.test.pages.biologics.plates.BasePlateSetPage.doAndWaitForGridColumnUpdate(BasePlateSetPage.java:279)
	at org.labkey.test.components.biologics.CustomizePlateViewDialog.clickUpdateGrid(CustomizePlateViewDialog.java:29)
	at org.labkey.test.tests.biologics.plates.PlateSamplesTest.testAssayImportWithTransformScript(PlateSamplesTest.java:938)

Related Pull Requests

• https://github.com/LabKey/limsModules/pull/1558

Changes

• Add null checks to Component.elementCache
• Add @NotNull annotations to overridden SearchContext methods

[labkey-chrisj]

Sounds all right to me.
Hope these help!

[labkey-tchad]

Retargeting to 25.7: #2600