Polish contribution

See gh-23756
spring-projects · Oct 23, 2019 · 82f64f6 · 82f64f6
1 parent e73344f
commit 82f64f6
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/...test/src/main/java/org/springframework/test/web/servlet/result/RequestResultMatchers.java b/...test/src/main/java/org/springframework/test/web/servlet/result/RequestResultMatchers.java
@@ -153,6 +153,7 @@ public <T> ResultMatcher sessionAttribute(String name, Object value) {
 
 	/**
 	 * Assert the given session attributes do not exist.
+	 * @since 5.2.1
 	 */
 	public <T> ResultMatcher sessionAttributeDoesNotExist(String... names) {
 		return result -> {

diff --git a/...rk/test/web/servlet/samples/standalone/resultmatchers/SessionAttributeAssertionTests.java b/...rk/test/web/servlet/samples/standalone/resultmatchers/SessionAttributeAssertionTests.java
@@ -28,6 +28,7 @@
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.SessionAttributes;
 
+import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
 import static org.hamcrest.Matchers.equalTo;
 import static org.hamcrest.Matchers.notNullValue;
 import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
@@ -68,7 +69,13 @@ public void testSessionAttributeMatcher() throws Exception {
 	@Test
 	public void testSessionAttributeDoesNotExist() throws Exception {
 		this.mockMvc.perform(get("/"))
-			.andExpect(request().sessionAttributeDoesNotExist("myAttr1", "myAttr2"));
+			.andExpect(request().sessionAttributeDoesNotExist("bogus", "enigma"));
+
+		assertThatExceptionOfType(AssertionError.class)
+			.isThrownBy(() ->
+				this.mockMvc.perform(get("/"))
+					.andExpect(request().sessionAttributeDoesNotExist("locale")))
+			.withMessage("Session attribute 'locale' exists");
 	}