From 593dc77b6d88ba4f04e0dd4f7e90cfd6b28b9e3f Mon Sep 17 00:00:00 2001 From: Jonah Graham Date: Sat, 15 Apr 2017 03:30:36 +0100 Subject: [PATCH] Bug 515307: Run test order of magnitude faster The break inserter test was running all display events for 200ms after each simulated keystroke, but all sideeffects were happening synchronously or being placed on the event queue synchronously. So speed-up is achieved by running event queue until empty instead of effectively sleeping for 200ms Change-Id: Icffd4d5526176e596cf8ce3ac716a4ae88d51159 Signed-off-by: Jonah Graham --- .../eclipse/cdt/ui/tests/text/BracketInserterTest.java | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/BracketInserterTest.java b/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/BracketInserterTest.java index bd2b2adda06..b8f58f44c89 100644 --- a/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/BracketInserterTest.java +++ b/core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/text/BracketInserterTest.java @@ -489,13 +489,9 @@ public class BracketInserterTest extends TestCase { event.keyCode= keyCode; event.stateMask= stateMask; fAccessor.invoke("handleKeyDown", new Object[] {event}); - - new DisplayHelper() { - @Override - protected boolean condition() { - return false; - } - }.waitForCondition(EditorTestHelper.getActiveDisplay(), 200); + + // run event loop once until all events are handled + DisplayHelper.runEventLoop(EditorTestHelper.getActiveDisplay(), 0); } private int getCaret() {