Skip to content

Commit 0e38cd7

Browse files
committed
Polishing
Issue: #910
1 parent 210979b commit 0e38cd7

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

junit-jupiter-engine/src/main/java/org/junit/jupiter/engine/descriptor/ClassTestDescriptor.java

-1
Original file line numberDiff line numberDiff line change
@@ -287,7 +287,6 @@ private AfterEachMethodAdapter synthesizeAfterEachMethodAdapter(Method method) {
287287
}
288288

289289
private void invokeMethodInExtensionContext(Method method, ExtensionContext context, ExtensionRegistry registry) {
290-
291290
Object testInstance = context.getTestInstance().orElseThrow(() -> new JUnitException(
292291
"Illegal state: test instance not present for method: " + method.toGenericString()));
293292

junit-jupiter-engine/src/main/java/org/junit/jupiter/engine/descriptor/TestTemplateTestDescriptor.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
import org.junit.jupiter.api.extension.TestTemplateInvocationContextProvider;
2323
import org.junit.jupiter.engine.execution.JupiterEngineExecutionContext;
2424
import org.junit.jupiter.engine.extension.ExtensionRegistry;
25+
import org.junit.platform.commons.JUnitException;
2526
import org.junit.platform.commons.meta.API;
2627
import org.junit.platform.commons.util.Preconditions;
2728
import org.junit.platform.engine.TestDescriptor;
@@ -59,7 +60,8 @@ public JupiterEngineExecutionContext prepare(JupiterEngineExecutionContext conte
5960
context.getExtensionRegistry());
6061

6162
// The test instance should be properly maintained by the enclosing class's ExtensionContext.
62-
Object testInstance = context.getExtensionContext().getTestInstance().orElse(null);
63+
Object testInstance = context.getExtensionContext().getTestInstance().orElseThrow(() -> new JUnitException(
64+
"Illegal state: test instance not present for method: " + getTestMethod().toGenericString()));
6365

6466
ExtensionContext extensionContext = new TestTemplateExtensionContext(context.getExtensionContext(),
6567
context.getExecutionListener(), this, testInstance);

0 commit comments

Comments
 (0)