This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.1.x by this push:
new e585746651 Reduce duplication
e585746651 is described below
commit e5857466517144481a6fa2452dcb675da47073ee
Author: Mark Thomas <[email protected]>
AuthorDate: Thu Dec 12 11:22:44 2024 +0000
Reduce duplication
---
...efaultServletRfc9110Section13Parameterized.java | 68 ++++++++--------------
1 file changed, 25 insertions(+), 43 deletions(-)
diff --git
a/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
b/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
index 9f9b7926b6..e3948e5ab9 100644
---
a/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
+++
b/test/org/apache/catalina/servlets/TestDefaultServletRfc9110Section13Parameterized.java
@@ -69,49 +69,31 @@ public class
TestDefaultServletRfc9110Section13Parameterized extends TomcatBaseT
@Parameterized.Parameters(name = "{index} resource-strong [{0}],
matchHeader [{1}]")
public static Collection<Object[]> parameters() {
List<Object[]> parameterSets = new ArrayList<>();
- // testPreconditions_rfc9110_13_2_2_1_head0
- parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
EtagPrecondition.ALL, null, null, null,
- null, Boolean.FALSE, SC_200 });
- parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
EtagPrecondition.EXACTLY, null, null, null,
- null, Boolean.FALSE, SC_200 });
- parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
EtagPrecondition.IN, null, null, null,
- null, Boolean.FALSE, SC_200 });
- parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
EtagPrecondition.NOT_IN, null, null, null,
- null, Boolean.FALSE, SC_412 });
- parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
EtagPrecondition.INVALID_ALL_PLUS_OTHER,
- null, null, null, null, Boolean.FALSE, SC_400 });
-
- parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
EtagPrecondition.ALL, null, null, null,
- null, Boolean.FALSE, SC_200 });
- parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
EtagPrecondition.EXACTLY, null, null,
- null, null, Boolean.FALSE, SC_412 });
- parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
EtagPrecondition.IN, null, null, null,
- null, Boolean.FALSE, SC_412 });
- parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
EtagPrecondition.NOT_IN, null, null, null,
- null, Boolean.FALSE, SC_412 });
- parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
EtagPrecondition.INVALID_ALL_PLUS_OTHER,
- null, null, null, null, Boolean.FALSE, SC_400 });
-
- parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
null, DatePrecondition.EQ, null, null,
- null, Boolean.FALSE, SC_200 });
- parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
null, DatePrecondition.LT, null, null,
- null, Boolean.FALSE, SC_412 });
- parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
null, DatePrecondition.GT, null, null,
- null, Boolean.FALSE, SC_200 });
- parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
null, DatePrecondition.MULTI_IN, null,
- null, null, Boolean.FALSE, SC_200 });
- parameterSets.add(new Object[] { Boolean.TRUE, Task.HEAD_INDEX_HTML,
null, DatePrecondition.LT, null,
- DatePrecondition.GT, null, Boolean.FALSE, SC_412 });
-
- parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
null, DatePrecondition.EQ, null, null,
- null, Boolean.FALSE, SC_200 });
- parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
null, DatePrecondition.LT, null, null,
- null, Boolean.FALSE, SC_412 });
- parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
null, DatePrecondition.GT, null, null,
- null, Boolean.FALSE, SC_200 });
- parameterSets.add(new Object[] { Boolean.FALSE, Task.HEAD_INDEX_HTML,
null, DatePrecondition.MULTI_IN, null,
- null, null, Boolean.FALSE, SC_200 });
-
+ for (Boolean useStrongEtag : booleans) {
+ // RFC 9110, Section 13.2.2, Step 1, HEAD
+ parameterSets.add(new Object[] { useStrongEtag,
Task.HEAD_INDEX_HTML, EtagPrecondition.ALL, null, null, null,
+ null, Boolean.FALSE, SC_200 });
+ parameterSets.add(new Object[] { useStrongEtag,
Task.HEAD_INDEX_HTML, EtagPrecondition.EXACTLY, null, null, null,
+ null, Boolean.FALSE, useStrongEtag.booleanValue() ? SC_200
: SC_412});
+ parameterSets.add(new Object[] { useStrongEtag,
Task.HEAD_INDEX_HTML, EtagPrecondition.IN, null, null, null,
+ null, Boolean.FALSE, useStrongEtag.booleanValue() ? SC_200
: SC_412 });
+ parameterSets.add(new Object[] { useStrongEtag,
Task.HEAD_INDEX_HTML, EtagPrecondition.NOT_IN, null, null, null,
+ null, Boolean.FALSE, SC_412 });
+ parameterSets.add(new Object[] { useStrongEtag,
Task.HEAD_INDEX_HTML, EtagPrecondition.INVALID_ALL_PLUS_OTHER,
+ null, null, null, null, Boolean.FALSE, SC_400 });
+
+ // RFC 9110, Section 13.2.2, Step 2, HEAD
+ parameterSets.add(new Object[] { useStrongEtag,
Task.HEAD_INDEX_HTML, null, DatePrecondition.EQ, null, null,
+ null, Boolean.FALSE, SC_200 });
+ parameterSets.add(new Object[] { useStrongEtag,
Task.HEAD_INDEX_HTML, null, DatePrecondition.LT, null, null,
+ null, Boolean.FALSE, SC_412 });
+ parameterSets.add(new Object[] { useStrongEtag,
Task.HEAD_INDEX_HTML, null, DatePrecondition.GT, null, null,
+ null, Boolean.FALSE, SC_200 });
+ parameterSets.add(new Object[] { useStrongEtag,
Task.HEAD_INDEX_HTML, null, DatePrecondition.MULTI_IN, null,
+ null, null, Boolean.FALSE, SC_200 });
+ parameterSets.add(new Object[] { useStrongEtag,
Task.HEAD_INDEX_HTML, null, DatePrecondition.LT, null,
+ DatePrecondition.GT, null, Boolean.FALSE, SC_412 });
+ }
return parameterSets;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
