Polish MockHttpServletRequest

sbrannen · sbrannen · commit 58b4286216da · 2023-07-05T13:59:47.000+02:00
diff --git a/spring-test/src/main/java/org/springframework/mock/web/MockHttpServletRequest.java b/spring-test/src/main/java/org/springframework/mock/web/MockHttpServletRequest.java
@@ -1159,7 +1159,8 @@ public String getHeader(String name) {
 	@Override
 	public Enumeration<String> getHeaders(String name) {
 		HeaderValueHolder header = this.headers.get(name);
-		return Collections.enumeration(header != null ? header.getStringValues() : new LinkedList<>());
+		return (header != null ? Collections.enumeration(header.getStringValues()) :
+				Collections.emptyEnumeration());
 	}
 
 	@Override
diff --git a/spring-web/src/testFixtures/java/org/springframework/web/testfixture/servlet/MockHttpServletRequest.java b/spring-web/src/testFixtures/java/org/springframework/web/testfixture/servlet/MockHttpServletRequest.java
@@ -1159,7 +1159,8 @@ public String getHeader(String name) {
 	@Override
 	public Enumeration<String> getHeaders(String name) {
 		HeaderValueHolder header = this.headers.get(name);
-		return Collections.enumeration(header != null ? header.getStringValues() : new LinkedList<>());
+		return (header != null ? Collections.enumeration(header.getStringValues()) :
+				Collections.emptyEnumeration());
 	}
 
 	@Override

Original file line number	Diff line number	Diff line change
`@@ -1159,7 +1159,8 @@ public String getHeader(String name) {`
`1159`	`1159`	`@Override`
`1160`	`1160`	`public Enumeration<String> getHeaders(String name) {`
`1161`	`1161`	`HeaderValueHolder header = this.headers.get(name);`
`1162`		`- return Collections.enumeration(header != null ? header.getStringValues() : new LinkedList<>());`
	`1162`	`+ return (header != null ? Collections.enumeration(header.getStringValues()) :`
	`1163`	`+ Collections.emptyEnumeration());`
`1163`	`1164`	`}`
`1164`	`1165`
`1165`	`1166`	`@Override`