ensure that failed place startups don't prevent server startup (#622)

src/main/java/emissary/admin/Startup.java

@@ -515,9 +515,11 @@ public boolean verifyNoInvisiblePlacesStarted() {
         try {
             IDirectoryPlace dirPlace = DirectoryPlace.lookup();
             List<DirectoryEntry> dirEntries = dirPlace.getEntries();
-            for (DirectoryEntry currentDir : dirEntries) {
-                // add place names of active directories
-                activeDirPlaces.add(currentDir.getLocalPlace().getPlaceName());
+            for (DirectoryEntry entry : dirEntries) {
+                // add place names of active places. getLocalPlace() returns null for any place that failed to start
+                if (entry.getLocalPlace() != null) {
+                    activeDirPlaces.add(entry.getLocalPlace().getPlaceName());
+                }
             }
 
             // remove DirectoryPlace from activeDirPlaces. DirectoryPlace is started up automatically in order to

src/test/java/emissary/admin/StartupTest.java

@@ -1,8 +1,10 @@
 package emissary.admin;
 
 import emissary.core.Namespace;
+import emissary.directory.DirectoryEntry;
 import emissary.directory.DirectoryPlace;
 import emissary.directory.EmissaryNode;
+import emissary.directory.IDirectoryPlace;
 import emissary.pickup.file.FilePickUpClient;
 import emissary.pickup.file.FilePickUpPlace;
 import emissary.place.CoordinationPlace;
@@ -12,6 +14,8 @@
 import emissary.util.io.ResourceReader;
 
 import org.junit.jupiter.api.Test;
+import org.mockito.MockedStatic;
+import org.mockito.Mockito;
 
 import java.io.IOException;
 import java.io.InputStream;
@@ -22,7 +26,9 @@
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertIterableEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.spy;
+import static org.mockito.Mockito.when;
 
 class StartupTest extends UnitTest {
     @Test
@@ -80,6 +86,29 @@ void testInvisPlaceStart() throws IOException {
         dirTeardown();
     }
 
+    @Test
+    void verifyNoInvisiblePlacesStartedHandlesNullLocalPlace() throws IOException {
+        // setup node, startup, and DirectoryPlace
+        EmissaryNode node = new EmissaryNode();
+        Startup startup = new Startup(node.getNodeConfigurator(), node);
+
+        try (MockedStatic<DirectoryPlace> dirPlace = Mockito.mockStatic(DirectoryPlace.class)) {
+
+            DirectoryEntry entry = mock(DirectoryEntry.class);
+            when(entry.getLocalPlace()).thenReturn(null);
+
+            List<DirectoryEntry> dirEntries = new ArrayList<>();
+            dirEntries.add(0, entry);
+
+            IDirectoryPlace directoryPlace = mock(IDirectoryPlace.class);
+            when(directoryPlace.getEntries()).thenReturn(dirEntries);
+
+            dirPlace.when(DirectoryPlace::lookup).thenReturn(directoryPlace);
+
+            assertTrue(startup.verifyNoInvisiblePlacesStarted());
+        }
+    }
+
     private DirectoryPlace master = null;
     private DirectoryPlace client = null;