Polish contribution

See gh-23769

Author: sbrannen

spring-test/src/main/java/org/springframework/mock/web/MockCookie.java

@@ -46,7 +46,7 @@ public class MockCookie extends Cookie {
 
 
 	/**
-	 * Constructor with the cookie name and value.
+	 * Construct a new {@link MockCookie} with the supplied name and value.
 	 * @param name the name
 	 * @param value the value
 	 * @see Cookie#Cookie(String, String)
@@ -56,33 +56,37 @@ public MockCookie(String name, String value) {
 	}
 
 	/**
-	 * Add the "Expires" attribute to the cookie.
+	 * Set the "Expires" attribute for this cookie.
+	 * @since 5.1.11
 	 */
 	public void setExpires(@Nullable ZonedDateTime expires) {
 		this.expires = expires;
 	}
 
 	/**
-	 * Return the "Expires" attribute, or {@code null} if not set.
+	 * Get the "Expires" attribute for this cookie.
+	 * @since 5.1.11
+	 * @return the "Expires" attribute for this cookie, or {@code null} if not set
 	 */
 	@Nullable
 	public ZonedDateTime getExpires() {
 		return this.expires;
 	}
 
 	/**
-	 * Add the "SameSite" attribute to the cookie.
+	 * Set the "SameSite" attribute for this cookie.
 	 * <p>This limits the scope of the cookie such that it will only be attached
-	 * to same site requests if {@code "Strict"} or cross-site requests if
-	 * {@code "Lax"}.
+	 * to same-site requests if the supplied value is {@code "Strict"} or cross-site
+	 * requests if the supplied value is {@code "Lax"}.
 	 * @see <a href="https://tools.ietf.org/html/draft-ietf-httpbis-rfc6265bis#section-4.1.2.7">RFC6265 bis</a>
 	 */
 	public void setSameSite(@Nullable String sameSite) {
 		this.sameSite = sameSite;
 	}
 
 	/**
-	 * Return the "SameSite" attribute, or {@code null} if not set.
+	 * Get the "SameSite" attribute for this cookie.
+	 * @return the "SameSite" attribute for this cookie, or {@code null} if not set
 	 */
 	@Nullable
 	public String getSameSite() {
@@ -91,7 +95,7 @@ public String getSameSite() {
 
 
 	/**
-	 * Factory method that parses the value of a "Set-Cookie" header.
+	 * Factory method that parses the value of the supplied "Set-Cookie" header.
 	 * @param setCookieHeader the "Set-Cookie" value; never {@code null} or empty
 	 * @return the created cookie
 	 */

spring-test/src/test/java/org/springframework/mock/web/MockHttpServletResponseTests.java

@@ -339,6 +339,18 @@ public void setCookieHeader() {
 		assertPrimarySessionCookie("999");
 	}
 
+	/**
+	 * @since 5.1.11
+	 */
+	@Test
+	public void setCookieHeaderWithExpiresAttribute() {
+		String cookieValue = "SESSION=123; Path=/; Max-Age=100; Expires=Tue, 8 Oct 2019 19:50:00 GMT; Secure; " +
+				"HttpOnly; SameSite=Lax";
+		response.setHeader(HttpHeaders.SET_COOKIE, cookieValue);
+		assertNumCookies(1);
+		assertEquals(cookieValue, response.getHeader(HttpHeaders.SET_COOKIE));
+	}
+
 	@Test
 	public void addCookieHeader() {
 		response.addHeader(HttpHeaders.SET_COOKIE, "SESSION=123; Path=/; Secure; HttpOnly; SameSite=Lax");
@@ -352,8 +364,11 @@ public void addCookieHeader() {
 		assertCookieValues("123", "999");
 	}
 
+	/**
+	 * @since 5.1.11
+	 */
 	@Test
-	public void addCookieHeaderWithExpires() {
+	public void addCookieHeaderWithExpiresAttribute() {
 		String cookieValue = "SESSION=123; Path=/; Max-Age=100; Expires=Tue, 8 Oct 2019 19:50:00 GMT; Secure; " +
 				"HttpOnly; SameSite=Lax";
 		response.addHeader(HttpHeaders.SET_COOKIE, cookieValue);

spring-web/src/test/java/org/springframework/mock/web/test/MockCookie.java

@@ -46,7 +46,7 @@ public class MockCookie extends Cookie {
 
 
 	/**
-	 * Constructor with the cookie name and value.
+	 * Construct a new {@link MockCookie} with the supplied name and value.
 	 * @param name the name
 	 * @param value the value
 	 * @see Cookie#Cookie(String, String)
@@ -56,33 +56,37 @@ public MockCookie(String name, String value) {
 	}
 
 	/**
-	 * Add the "Expires" attribute to the cookie.
+	 * Set the "Expires" attribute for this cookie.
+	 * @since 5.1.11
 	 */
 	public void setExpires(@Nullable ZonedDateTime expires) {
 		this.expires = expires;
 	}
 
 	/**
-	 * Return the "Expires" attribute, or {@code null} if not set.
+	 * Get the "Expires" attribute for this cookie.
+	 * @since 5.1.11
+	 * @return the "Expires" attribute for this cookie, or {@code null} if not set
 	 */
 	@Nullable
 	public ZonedDateTime getExpires() {
 		return this.expires;
 	}
 
 	/**
-	 * Add the "SameSite" attribute to the cookie.
+	 * Set the "SameSite" attribute for this cookie.
 	 * <p>This limits the scope of the cookie such that it will only be attached
-	 * to same site requests if {@code "Strict"} or cross-site requests if
-	 * {@code "Lax"}.
+	 * to same-site requests if the supplied value is {@code "Strict"} or cross-site
+	 * requests if the supplied value is {@code "Lax"}.
 	 * @see <a href="https://tools.ietf.org/html/draft-ietf-httpbis-rfc6265bis#section-4.1.2.7">RFC6265 bis</a>
 	 */
 	public void setSameSite(@Nullable String sameSite) {
 		this.sameSite = sameSite;
 	}
 
 	/**
-	 * Return the "SameSite" attribute, or {@code null} if not set.
+	 * Get the "SameSite" attribute for this cookie.
+	 * @return the "SameSite" attribute for this cookie, or {@code null} if not set
 	 */
 	@Nullable
 	public String getSameSite() {
@@ -91,7 +95,7 @@ public String getSameSite() {
 
 
 	/**
-	 * Factory method that parses the value of a "Set-Cookie" header.
+	 * Factory method that parses the value of the supplied "Set-Cookie" header.
 	 * @param setCookieHeader the "Set-Cookie" value; never {@code null} or empty
 	 * @return the created cookie
 	 */