This is an automated email from the ASF dual-hosted git repository. pedrosans pushed a commit to branch wicket-10.x in repository https://gitbox.apache.org/repos/asf/wicket.git
commit fedc172255b5dbd9c87c7d1ca26bed4f3b6012f3 Author: Schäfer, H.H. (Hans Hosea) <[email protected]> AuthorDate: Tue Sep 8 21:18:19 2026 +0200 Simplified test. --- .../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 ad3edefdcb..71a9fc62e3 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 @@ -58,6 +58,8 @@ import static org.junit.jupiter.api.Assertions.assertFalse; import static org.mockito.ArgumentMatchers.anyString; import static org.mockito.Mockito.*; +import java.util.Optional; + class CSPHeaderWriterTest extends WicketTestCase { @@ -124,7 +126,7 @@ class CSPHeaderWriterTest extends WicketTestCase } @Test - void addCspDirectiveToStatelessPageEvenIfWrappedInMultipleIRequestHandlerDelegates() + void addCspDirectiveToPageEvenIfWrappedInMultipleIRequestHandlerDelegates() { tester.getApplication().mount(new MountedMapper("withdelegate/page", NoopMockPage.class) { @@ -132,14 +134,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);
