Fire events for empty containers to meet expectations of IDEs

Since empty container and engine descriptors are no longer pruned they
show up in the test tree as "atomic tests". As such, IDEs expect
events when they started/finished.

Author: marcphilipp

junit-platform-runner/src/main/java/org/junit/platform/runner/JUnitPlatformRunnerListener.java

@@ -58,8 +58,8 @@ private void fireTestIgnored(TestIdentifier testIdentifier) {
 
 	@Override
 	public void executionStarted(TestIdentifier testIdentifier) {
-		if (testIdentifier.isTest()) {
-			Description description = findJUnit4Description(testIdentifier);
+		Description description = findJUnit4Description(testIdentifier);
+		if (description.isTest()) {
 			this.notifier.fireTestStarted(description);
 		}
 	}
@@ -74,7 +74,7 @@ public void executionFinished(TestIdentifier testIdentifier, TestExecutionResult
 		else if (status == FAILED) {
 			this.notifier.fireTestFailure(toFailure(testExecutionResult, description));
 		}
-		if (testIdentifier.isTest()) {
+		if (description.isTest()) {
 			this.notifier.fireTestFinished(description);
 		}
 	}