This is an automated email from the ASF dual-hosted git repository.
papegaaij pushed a commit to branch csp-configurable
in repository https://gitbox.apache.org/repos/asf/wicket.git
The following commit(s) were added to refs/heads/csp-configurable by this push:
new 9dddf2e WICKET-6727: do not extend WicketTester but WicketTestCase
9dddf2e is described below
commit 9dddf2efe6762730ef5d484fd6dec37f3c44c012
Author: Emond Papegaaij <[email protected]>
AuthorDate: Fri Jan 17 09:24:21 2020 +0100
WICKET-6727: do not extend WicketTester but WicketTestCase
---
.../csp/CSPSettingRequestCycleListenerTest.java | 45 +++++++++++-----------
1 file changed, 23 insertions(+), 22 deletions(-)
diff --git
a/wicket-core/src/test/java/org/apache/wicket/csp/CSPSettingRequestCycleListenerTest.java
b/wicket-core/src/test/java/org/apache/wicket/csp/CSPSettingRequestCycleListenerTest.java
index 28b292c..d4893cd 100644
---
a/wicket-core/src/test/java/org/apache/wicket/csp/CSPSettingRequestCycleListenerTest.java
+++
b/wicket-core/src/test/java/org/apache/wicket/csp/CSPSettingRequestCycleListenerTest.java
@@ -38,13 +38,14 @@ import java.util.stream.Collectors;
import java.util.stream.Stream;
import org.apache.wicket.mock.MockHomePage;
+import org.apache.wicket.util.tester.WicketTestCase;
import org.apache.wicket.util.tester.WicketTester;
import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
@SuppressWarnings("deprecation")
-public class CSPSettingRequestCycleListenerTest extends WicketTester
+public class CSPSettingRequestCycleListenerTest extends WicketTestCase
{
private static String HEADER_CSP = "Content-Security-Policy";
@@ -62,7 +63,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testNullSrcInputIsRejected()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(DEFAULT_SRC,
(String) null);
});
@@ -72,7 +73,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testEmptySrcInputIsRejected()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(DEFAULT_SRC, "");
});
@@ -86,7 +87,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testInvalidSrcInputIsRejected()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(DEFAULT_SRC,
"abc?^()-_\'xyz");
});
@@ -100,7 +101,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testMultipleSrcInputWithNoneIsRejected1()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(DEFAULT_SRC, SELF,
NONE);
});
@@ -114,7 +115,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testMultipleSrcInputWithNoneIsRejected2()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(DEFAULT_SRC, NONE,
SELF);
});
@@ -128,7 +129,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testMultipleSrcInputWithStarIsRejected1()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
cspListener.blocking().addDirective(DEFAULT_SRC, SELF);
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(DEFAULT_SRC,
WILDCARD);
@@ -143,7 +144,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testMultipleSrcInputWithStarIsRejected2()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
cspListener.blocking().addDirective(DEFAULT_SRC, WILDCARD);
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(DEFAULT_SRC, SELF);
@@ -154,7 +155,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testWrongSrcInputIsRejected()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(DEFAULT_SRC,
ALLOW_FORMS);
});
@@ -164,7 +165,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testWrongSandboxInputIsRejected()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(SANDBOX, SELF);
});
@@ -174,7 +175,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testNullSandboxInputIsRejected()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(SANDBOX, (String)
null);
});
@@ -184,7 +185,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testEmptySandboxInputIsAccepted()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
cspListener.blocking().addDirective(SANDBOX,
CSPDirectiveSandboxValue.EMPTY);
}
@@ -192,7 +193,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testInvalidSandboxInputIsRejected()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(SANDBOX, "abcxyz");
});
@@ -202,7 +203,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testMultipleSandboxInputWithEmptyStringIsRejected1()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
cspListener.blocking().addDirective(SANDBOX, ALLOW_FORMS);
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(SANDBOX, EMPTY);
@@ -213,7 +214,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testMultipleSandboxInputWithEmptyStringIsRejected2()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
cspListener.blocking().addDirective(SANDBOX, EMPTY);
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(SANDBOX,
ALLOW_FORMS);
@@ -224,7 +225,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testNullReportUriInputIsRejected()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(REPORT_URI,
(String) null);
});
@@ -234,7 +235,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testEmptyReportUriInputIsRejected()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(REPORT_URI, "");
});
@@ -244,7 +245,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testInvalidReportUriInputIsRejected()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
Assertions.assertThrows(IllegalArgumentException.class, () -> {
cspListener.blocking().addDirective(REPORT_URI,
"abc?^()-_\'xyz");
});
@@ -254,7 +255,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testAllCSPSrcDefaultEnumsAreSetCorrectly() throws
NoSuchAlgorithmException
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
final int cspDirectiveCount = CSPDirective.values().length;
final int cspDirectiveSrcValueCount =
CSPDirectiveSrcValue.values().length;
@@ -284,7 +285,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testCSPReportUriDirectiveSetCorrectly()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
cspListener.blocking().addDirective(REPORT_URI,
"http://report.example.com";);
cspListener.reporting().addDirective(REPORT_URI,
"/example-report-uri");
@@ -300,7 +301,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testCSPSandboxDirectiveSetCorrectly()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
final int cspSandboxDirectiveValueCount =
CSPDirectiveSandboxValue.values().length;
for (int i = 0; i < cspSandboxDirectiveValueCount; i++)
{
@@ -329,7 +330,7 @@ public class CSPSettingRequestCycleListenerTest extends
WicketTester
public void testChildSrcDirectiveAlsoSetsFrameSrcDirective()
{
CSPSettingRequestCycleListener cspListener =
- new CSPSettingRequestCycleListener(getApplication());
+ new
CSPSettingRequestCycleListener(tester.getApplication());
cspListener.blocking().addDirective(CHILD_SRC, SELF);
cspListener.reporting().addDirective(CHILD_SRC, SELF);
StringBuffer headerErrors = checkHeaders(cspListener);
