EventServiceTest: try to speed up a test

We do not need to _always_ wait 500 ms, but rather can try waiting for
a notification from the other thread. Given the overhead of queuing
things onto the EDT, it turns out that the test thread does end up
waiting before the EDT notifies. So in practice it should cut down on
how long the test takes to run.

Suggested by Johannes Schindelin.

Author: ctrueden

src/test/java/org/scijava/event/EventServiceTest.java

@@ -89,7 +89,10 @@ public void testEventHandlerDependencies() throws InterruptedException {
 		final ServiceNeedingAnEventService snaeService =
 			context.getService(ServiceNeedingAnEventService.class);
 		assertNotNull(eventService);
-		Thread.sleep(500); // NB: ServicesLoadedEvent is published asynchronously.
+		// NB: ServicesLoadedEvent is published asynchronously.
+		synchronized (snaeService) {
+			snaeService.wait(500);
+		}
 		assertTrue(snaeService.isContextCreated());
 	}
 
@@ -127,6 +130,9 @@ public void onEvent(
 			@SuppressWarnings("unused") final ServicesLoadedEvent evt)
 		{
 			contextCreated = true;
+			synchronized (this) {
+				notifyAll();
+			}
 		}
 	}