QuarkusUnitTest#interceptBeforeEachMethod() throws NPE if a test expects an exception and declares BeforeEach

[mkouba]

Describe the bug

The method body currently looks like:

    @Override
    public void interceptBeforeEachMethod(Invocation<Void> invocation, ReflectiveInvocationContext<Method> invocationContext,
            ExtensionContext extensionContext) throws Throwable {
        runExtensionMethod(invocationContext, extensionContext, true);
        invocation.skip();
    }

Unlike other interception methods that reflect the assertException, e.g.:

@Override
    public void interceptAfterEachMethod(Invocation<Void> invocation, ReflectiveInvocationContext<Method> invocationContext,
            ExtensionContext extensionContext) throws Throwable {
        if (assertException == null) {
            runExtensionMethod(invocationContext, extensionContext, true);
            invocation.skip();
        } else {
            invocation.proceed();
        }
    }

I have no idea if it's intentional or just an oversight.

Expected behavior

No response

Actual behavior

No response

How to Reproduce?

No response

Output of uname -a or ver

No response

Output of java -version

No response

Quarkus version or git rev

No response

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

No response

[quarkus-bot]

/cc @geoand (testing)

[geoand]

Good catch!

Are you planning on fixing it?

[mkouba]

Good catch!

Are you planning on fixing it?

If you believe that it was not intentional then yes, ofc ;-)

[geoand]

I am pretty sure it was just an oversight