Add UI event processing after command, close editors after test

Fixes: #666

Author: trancexpress

ui/org.eclipse.pde.genericeditor.extension.tests/src/org/eclipse/pde/genericeditor/extension/tests/AbstractTargetEditorTest.java

@@ -41,6 +41,8 @@
 import org.eclipse.jface.text.contentassist.ICompletionProposal;
 import org.eclipse.pde.internal.genericeditor.target.extension.autocomplete.TargetDefinitionContentAssist;
 import org.eclipse.ui.IEditorPart;
+import org.eclipse.ui.IWorkbenchPage;
+import org.eclipse.ui.IWorkbenchWindow;
 import org.eclipse.ui.PlatformUI;
 import org.eclipse.ui.ide.IDE;
 import org.junit.After;
@@ -87,6 +89,14 @@ protected String getLocationForSite(String name) {
 
 	@After
 	public void tearDown() throws Exception {
+		// Close any editors we opened
+		IWorkbenchWindow[] workbenchPages = PlatformUI.getWorkbench().getWorkbenchWindows();
+		for (IWorkbenchWindow workbenchPage : workbenchPages) {
+			IWorkbenchPage page = workbenchPage.getActivePage();
+			if (page != null) {
+				page.closeAllEditors(false);
+			}
+		}
 		if (tempFile != null) {
 			tempFile.delete();
 		}

ui/org.eclipse.pde.genericeditor.extension.tests/src/org/eclipse/pde/genericeditor/extension/tests/TestUtils.java

@@ -0,0 +1,15 @@
+package org.eclipse.pde.genericeditor.extension.tests;
+
+import org.eclipse.swt.widgets.Display;
+
+public class TestUtils {
+
+	public static void processUIEvents() {
+		Display display = Display.getCurrent();
+		if (display != null && !display.isDisposed()) {
+			while (display.readAndDispatch()) {
+				// Keep pumping events until the queue is empty
+			}
+		}
+	}
+}

ui/org.eclipse.pde.genericeditor.extension.tests/src/org/eclipse/pde/genericeditor/extension/tests/UpdateUnitVersionsCommandTests.java

@@ -82,6 +82,7 @@ private void confirmVersionUpdates(Map<String, String> expected) throws Exceptio
 		@SuppressWarnings("unchecked")
 		String updatedText = ((CompletableFuture<String>) response).get();
 		assertNotNull(updatedText);
+		TestUtils.processUIEvents();
 
 		Map<String, String> actual = getVersionsForIdsFromTargetFile(updatedText);