This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/9.0.x by this push:
new 5aef58439c Fix IDE warnings. Reformat code.
5aef58439c is described below
commit 5aef58439cdfb9343047466bcf8861472a121435
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Wed Dec 11 14:22:06 2024 +0000
Fix IDE warnings. Reformat code.
---
.../TestDefaultServletRfc9110Section13.java | 23 +++---
...efaultServletRfc9110Section13Parameterized.java | 91 +++++++++++-----------
2 files changed, 56 insertions(+), 58 deletions(-)
diff --git
a/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13.java
b/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13.java
index c59957cb46..d132256803 100644
--- a/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13.java
+++ b/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13.java
@@ -422,7 +422,7 @@ public class TestDefaultServletRfc9110Section13 extends
TomcatBaseTest {
}
protected List<String> genETagCondtion(String strongETag, String weakETag,
IfPolicy policy) {
- List<String> headerValues = new ArrayList<String>();
+ List<String> headerValues = new ArrayList<>();
switch (policy) {
case ETAG_ALL:
headerValues.add("*");
@@ -442,10 +442,10 @@ public class TestDefaultServletRfc9110Section13 extends
TomcatBaseTest {
break;
case ETAG_NOT_IN:
if (weakETag != null && weakETag.length() > 8) {
- headerValues.add(weakETag.substring(0, 3) +
"XXXXX"+weakETag.substring(8));
+ headerValues.add(weakETag.substring(0, 3) + "XXXXX" +
weakETag.substring(8));
}
if (strongETag != null && strongETag.length() > 6) {
- headerValues.add(strongETag.substring(0, 1) +
"XXXXX"+strongETag.substring(6));
+ headerValues.add(strongETag.substring(0, 1) + "XXXXX" +
strongETag.substring(6));
}
break;
case ETAG_SYNTAX_INVALID:
@@ -459,7 +459,7 @@ public class TestDefaultServletRfc9110Section13 extends
TomcatBaseTest {
}
protected List<String> genDateCondtion(long lastModifiedTimestamp,
IfPolicy policy) {
- List<String> headerValues = new ArrayList<String>();
+ List<String> headerValues = new ArrayList<>();
if (lastModifiedTimestamp <= 0) {
return headerValues;
}
@@ -493,7 +493,7 @@ public class TestDefaultServletRfc9110Section13 extends
TomcatBaseTest {
if (policy == null) {
return;
}
- List<String> headerValues = new ArrayList<String>();
+ List<String> headerValues = new ArrayList<>();
String weakETag = resourceETag;
String strongETag = resourceETag;
if (resourceETag != null) {
@@ -610,8 +610,8 @@ public class TestDefaultServletRfc9110Section13 extends
TomcatBaseTest {
};
client.setPort(getPort());
StringBuffer curl = new StringBuffer();
- curl.append(task.m.name() + " " + task.uri + " HTTP/1.1" +
SimpleHttpClient.CRLF + "Host: localhost" + SimpleHttpClient.CRLF +
- "Connection: Close" + SimpleHttpClient.CRLF);
+ curl.append(task.m.name() + " " + task.uri + " HTTP/1.1" +
SimpleHttpClient.CRLF + "Host: localhost" +
+ SimpleHttpClient.CRLF + "Connection: Close" +
SimpleHttpClient.CRLF);
for (Entry<String,List<String>> e : requestHeaders.entrySet()) {
for (String v : e.getValue()) {
@@ -648,11 +648,10 @@ public class TestDefaultServletRfc9110Section13 extends
TomcatBaseTest {
test = p.test(sc);
}
String scExpectation = usePredicate ? "IntPredicate" :
Arrays.toString(scExpected);
- Assert.assertTrue(
- "Failure - sc expected:" + scExpectation + ", sc actual:" +
String.valueOf(sc) + ", " + message
- + ", task:" + task + ", \ntarget resource:(" + etag + "," +
FastHttpDateFormat.formatDate(lastModified)
- + "), \nreq headers: " + requestHeaders.toString() + ", \nresp
headers: " + responseHeaders.toString(),
- test);
+ Assert.assertTrue("Failure - sc expected:" + scExpectation + ", sc
actual:" + String.valueOf(sc) + ", " +
+ message + ", task:" + task + ", \ntarget resource:(" + etag +
"," +
+ FastHttpDateFormat.formatDate(lastModified) + "), \nreq
headers: " + requestHeaders.toString() +
+ ", \nresp headers: " + responseHeaders.toString(), test);
}
protected void testPreconditions(Task task, IfPolicy ifMatchHeader,
IfPolicy ifUnmodifiedSinceHeader,
diff --git
a/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
b/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
index 9f97c9a76d..d349a0b075 100644
---
a/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
+++
b/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
@@ -73,45 +73,45 @@ public class
TestDefaultServletRfc9110Section13Parameterized extends TomcatBaseT
public static Collection<Object[]> parameters() {
List<Object[]> parameterSets = new ArrayList<>();
// testPreconditions_rfc9110_13_2_2_1_head0
- parameterSets.add(new Object[] { true, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_ALL, null, null, null, null, false,
- null, new int[] { 200 } });
- parameterSets.add(new Object[] { true, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_EXACTLY, null, null, null, null,
- false, null, new int[] { 200 } });
- parameterSets.add(new Object[] { true, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_IN, null, null, null, null, false,
- null, new int[] { 200 } });
- parameterSets.add(new Object[] { true, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_NOT_IN, null, null, null, null,
- false, null, new int[] { 412 } });
- parameterSets.add(new Object[] { true, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_SYNTAX_INVALID, null, null, null,
- null, false, null, new int[] { 400 } });
-
- parameterSets.add(new Object[] { false, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_ALL, null, null, null, null, false,
- null, new int[] { 200 } });
- parameterSets.add(new Object[] { false, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_EXACTLY, null, null, null, null,
- false, null, new int[] { 412 } });
- parameterSets.add(new Object[] { false, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_IN, null, null, null, null, false,
- null, new int[] { 412 } });
- parameterSets.add(new Object[] { false, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_NOT_IN, null, null, null, null,
- false, null, new int[] { 412 } });
- parameterSets.add(new Object[] { false, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_SYNTAX_INVALID, null, null, null,
- null, false, null, new int[] { 400 } });
-
- parameterSets.add(new Object[] { true, Task.HEAD_INDEX_HTML, null,
IfPolicy.DATE_EQ, null, null, null, false,
- null, new int[] { 200 } });
- parameterSets.add(new Object[] { true, Task.HEAD_INDEX_HTML, null,
IfPolicy.DATE_LT, null, null, null, false,
- null, new int[] { 412 } });
- parameterSets.add(new Object[] { true, Task.HEAD_INDEX_HTML, null,
IfPolicy.DATE_GT, null, null, null, false,
- null, new int[] { 200 } });
- parameterSets.add(new Object[] { true, Task.HEAD_INDEX_HTML, null,
IfPolicy.DATE_MULTI_IN, null, null, null,
- false, null, new int[] { 200 } });
-
- parameterSets.add(new Object[] { false, Task.HEAD_INDEX_HTML, null,
IfPolicy.DATE_EQ, null, null, null, false,
- null, new int[] { 200 } });
- parameterSets.add(new Object[] { false, Task.HEAD_INDEX_HTML, null,
IfPolicy.DATE_LT, null, null, null, false,
- null, new int[] { 412 } });
- parameterSets.add(new Object[] { false, Task.HEAD_INDEX_HTML, null,
IfPolicy.DATE_GT, null, null, null, false,
- null, new int[] { 200 } });
- parameterSets.add(new Object[] { false, Task.HEAD_INDEX_HTML, null,
IfPolicy.DATE_MULTI_IN, null, null, null,
- false, null, new int[] { 200 } });
+ parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_ALL, null, null, null, null,
+ Boolean.FALSE, null, new int[] { 200 } });
+ parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_EXACTLY, null, null, null,
+ null, Boolean.FALSE, null, new int[] { 200 } });
+ parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_IN, null, null, null, null,
+ Boolean.FALSE, null, new int[] { 200 } });
+ parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_NOT_IN, null, null, null,
+ null, Boolean.FALSE, null, new int[] { 412 } });
+ parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_SYNTAX_INVALID, null, null,
+ null, null, Boolean.FALSE, null, new int[] { 400 } });
+
+ parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_ALL, null, null, null, null,
+ Boolean.FALSE, null, new int[] { 200 } });
+ parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_EXACTLY, null, null, null,
+ null, Boolean.FALSE, null, new int[] { 412 } });
+ parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_IN, null, null, null, null,
+ Boolean.FALSE, null, new int[] { 412 } });
+ parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_NOT_IN, null, null, null,
+ null, Boolean.FALSE, null, new int[] { 412 } });
+ parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
IfPolicy.ETAG_SYNTAX_INVALID, null, null,
+ null, null, Boolean.FALSE, null, new int[] { 400 } });
+
+ parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
null, IfPolicy.DATE_EQ, null, null, null,
+ Boolean.FALSE, null, new int[] { 200 } });
+ parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
null, IfPolicy.DATE_LT, null, null, null,
+ Boolean.FALSE, null, new int[] { 412 } });
+ parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
null, IfPolicy.DATE_GT, null, null, null,
+ Boolean.FALSE, null, new int[] { 200 } });
+ parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
null, IfPolicy.DATE_MULTI_IN, null, null,
+ null, Boolean.FALSE, null, new int[] { 200 } });
+
+ parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
null, IfPolicy.DATE_EQ, null, null, null,
+ Boolean.FALSE, null, new int[] { 200 } });
+ parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
null, IfPolicy.DATE_LT, null, null, null,
+ Boolean.FALSE, null, new int[] { 412 } });
+ parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
null, IfPolicy.DATE_GT, null, null, null,
+ Boolean.FALSE, null, new int[] { 200 } });
+ parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
null, IfPolicy.DATE_MULTI_IN, null, null,
+ null, Boolean.FALSE, null, new int[] { 200 } });
return parameterSets;
@@ -204,7 +204,7 @@ public class
TestDefaultServletRfc9110Section13Parameterized extends TomcatBaseT
}
protected List<String> genETagCondtion(String strongETag, String weakETag,
IfPolicy policy) {
- List<String> headerValues = new ArrayList<String>();
+ List<String> headerValues = new ArrayList<>();
switch (policy) {
case ETAG_ALL:
headerValues.add("*");
@@ -241,7 +241,7 @@ public class
TestDefaultServletRfc9110Section13Parameterized extends TomcatBaseT
}
protected List<String> genDateCondtion(long lastModifiedTimestamp,
IfPolicy policy) {
- List<String> headerValues = new ArrayList<String>();
+ List<String> headerValues = new ArrayList<>();
if (lastModifiedTimestamp <= 0) {
return headerValues;
}
@@ -275,7 +275,7 @@ public class
TestDefaultServletRfc9110Section13Parameterized extends TomcatBaseT
if (policy == null) {
return;
}
- List<String> headerValues = new ArrayList<String>();
+ List<String> headerValues = new ArrayList<>();
String weakETag = resourceETag;
String strongETag = resourceETag;
if (resourceETag != null) {
@@ -423,10 +423,9 @@ public class
TestDefaultServletRfc9110Section13Parameterized extends TomcatBaseT
test = p.test(sc);
}
String scExpectation = usePredicate ? "IntPredicate" :
Arrays.toString(scExpected);
- Assert.assertTrue(
- "Failure - sc expected:" + scExpectation + ", sc actual:" +
String.valueOf(sc)
- + ", task:" + task + ", \ntarget resource:(" + etag + "," +
FastHttpDateFormat.formatDate(lastModified)
- + "), \nreq headers: " + requestHeaders.toString() + ", \nresp
headers: " + responseHeaders.toString(),
+ Assert.assertTrue("Failure - sc expected:" + scExpectation + ", sc
actual:" + String.valueOf(sc) + ", task:" +
+ task + ", \ntarget resource:(" + etag + "," +
FastHttpDateFormat.formatDate(lastModified) +
+ "), \nreq headers: " + requestHeaders.toString() + ", \nresp
headers: " + responseHeaders.toString(),
test);
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
