This is an automated email from the ASF dual-hosted git repository. pedrosans pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/wicket.git
commit c106a81350c405dc0b31b1ecebae1457f2c9daab Author: Schäfer, H.H. (Hans Hosea) <[email protected]> AuthorDate: Tue Sep 8 21:18:19 2026 +0200 Simplified test. (cherry picked from commit fedc172255b5dbd9c87c7d1ca26bed4f3b6012f3) --- .../java/org/apache/wicket/csp/CSPHeaderWriterTest.java | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/wicket-core-tests/src/test/java/org/apache/wicket/csp/CSPHeaderWriterTest.java b/wicket-core-tests/src/test/java/org/apache/wicket/csp/CSPHeaderWriterTest.java index a2034c8d12..d4127adca1 100644 --- a/wicket-core-tests/src/test/java/org/apache/wicket/csp/CSPHeaderWriterTest.java +++ b/wicket-core-tests/src/test/java/org/apache/wicket/csp/CSPHeaderWriterTest.java @@ -45,6 +45,8 @@ import static org.apache.wicket.csp.CSPDirectiveSrcValue.SELF; import static org.assertj.core.api.Assertions.assertThat; import static org.junit.jupiter.api.Assertions.*; +import java.util.Optional; + class CSPHeaderWriterTest extends WicketTestCase { @@ -127,7 +129,7 @@ class CSPHeaderWriterTest extends WicketTestCase } @Test - void addCspDirectiveToStatelessPageEvenIfWrappedInMultipleIRequestHandlerDelegates() + void addCspDirectiveToPageEvenIfWrappedInMultipleIRequestHandlerDelegates() { tester.getApplication().mount(new MountedMapper("withdelegate/page", NoopMockPage.class) { @@ -135,14 +137,10 @@ class CSPHeaderWriterTest extends WicketTestCase public IRequestHandler mapRequest(final Request request) { final IRequestHandler requestHandler = super.mapRequest(request); - final IRequestHandler result; - if (requestHandler instanceof RenderPageRequestHandler renderPageRequestHandler && NoopMockPage.class.equals(renderPageRequestHandler.getPageClass())) - { - result = new NoopIRequestHandlerDelegate(new NoopIRequestHandlerDelegate(renderPageRequestHandler)); - } else { - result = requestHandler; - } - return result; + return Optional.ofNullable(requestHandler) + .map(NoopIRequestHandlerDelegate::new) + .map(NoopIRequestHandlerDelegate::new) // wrap twice + .orElse(null); } }); tester.startPage(NoopMockPage.class);
