From 0e097640972835e71a625cdee7a9359089cbec78 Mon Sep 17 00:00:00 2001
From: Marc Philipp <mail@marcphilipp.de>
Date: Tue, 3 Nov 2015 10:46:05 +0100
Subject: [PATCH] Only add TestDescriptor for engine if non-empty

Empty Engine nodes confuse Eclipse so much, it
never finishes its progress bar.
---
 .../src/main/java/org/junit/gen5/launcher/Launcher.java   | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/junit-launcher/src/main/java/org/junit/gen5/launcher/Launcher.java b/junit-launcher/src/main/java/org/junit/gen5/launcher/Launcher.java
index e36937da299..01012d7bff7 100644
--- a/junit-launcher/src/main/java/org/junit/gen5/launcher/Launcher.java
+++ b/junit-launcher/src/main/java/org/junit/gen5/launcher/Launcher.java
@@ -12,6 +12,7 @@
 
 import static org.junit.gen5.launcher.TestEngineRegistry.lookupAllTestEngines;
 
+import java.util.Collection;
 import java.util.List;
 
 import org.junit.gen5.engine.EngineDescriptor;
@@ -38,8 +39,11 @@ public TestPlan discover(TestPlanSpecification specification) {
 		TestPlan testPlan = new TestPlan();
 		for (TestEngine testEngine : lookupAllTestEngines()) {
 			TestDescriptor engineDescriptor = new EngineDescriptor(testEngine);
-			testPlan.addTestDescriptor(engineDescriptor);
-			testPlan.addTestDescriptors(testEngine.discoverTests(specification, engineDescriptor));
+			Collection<TestDescriptor> testDescriptors = testEngine.discoverTests(specification, engineDescriptor);
+			if (!testDescriptors.isEmpty()) {
+				testPlan.addTestDescriptor(engineDescriptor);
+				testPlan.addTestDescriptors(testDescriptors);
+			}
 		}
 		return testPlan;
 	}