Suppress warnings we cannot get rid of at the moment

junit-team · Oct 30, 2015 · 3d7f8db · 3d7f8db
1 parent a0ff89a
commit 3d7f8db
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/junit5-engine/src/main/java/org/junit/gen5/engine/junit5/JUnit5TestEngine.java b/junit5-engine/src/main/java/org/junit/gen5/engine/junit5/JUnit5TestEngine.java
@@ -59,14 +59,22 @@ public List<TestDescriptor> discoverTests(TestPlanSpecification specification) {
 		testDescriptors.add(root);
 
 		for (TestPlanSpecificationElement element : specification) {
-			TestDescriptorResolver testDescriptorResolver = testDescriptorResolverRegistry.forType(element.getClass());
-			TestDescriptor descriptor = testDescriptorResolver.resolve(root, element);
-			testDescriptors.add(descriptor);
-			testDescriptors.addAll(testDescriptorResolver.resolveChildren(descriptor, element));
+			testDescriptors.addAll(resolveElement(testDescriptorResolverRegistry, root, element));
 		}
 		return new ArrayList<>(testDescriptors);
 	}
 
+	@SuppressWarnings({ "rawtypes", "unchecked" })
+	private Set<TestDescriptor> resolveElement(TestDescriptorResolverRegistry testDescriptorResolverRegistry,
+			EngineDescriptor root, TestPlanSpecificationElement element) {
+		Set<TestDescriptor> testDescriptors = new LinkedHashSet<>();
+		TestDescriptorResolver testDescriptorResolver = testDescriptorResolverRegistry.forType(element.getClass());
+		TestDescriptor descriptor = testDescriptorResolver.resolve(root, element);
+		testDescriptors.add(descriptor);
+		testDescriptors.addAll(testDescriptorResolver.resolveChildren(descriptor, element));
+		return testDescriptors;
+	}
+
 	@Override
 	public boolean supports(TestDescriptor testDescriptor) {
 		// TODO super class for Java test descriptors?