Improve tests for BasicErrorController

This commit improves the tests for BasicErrorController by decoupling coverage for the include-message and include-binding-errors parameters to ensure the options operate properly independent of each other. See gh-21702
spring-projects · Jun 8, 2020 · 73aff08 · 73aff08
1 parent e2e52a1
commit 73aff08
Showing 1 changed file with 62 additions and 23 deletions.
diff --git a/...gframework/boot/autoconfigure/web/servlet/error/BasicErrorControllerIntegrationTests.java b/...gframework/boot/autoconfigure/web/servlet/error/BasicErrorControllerIntegrationTests.java
@@ -205,50 +205,89 @@ void testErrorForAnnotatedNoMessageExceptionWithMessage() {
 	}
 
 	@Test
-	void testBindingExceptionForMachineClientWithMessageAndErrorsParamTrue() {
-		load("--server.error.include-exception=true", "--server.error.include-message=on-param",
-				"--server.error.include-binding-errors=on-param");
-		bindingExceptionWithMessageAndErrors("?message=true&errors=true");
+	void testBindingExceptionForMachineClientWithErrorsParamTrue() {
+		load("--server.error.include-exception=true", "--server.error.include-binding-errors=on-param");
+		bindingExceptionWithErrors("?errors=true");
 	}
 
 	@Test
-	void testBindingExceptionForMachineClientWithMessageAndErrorsParamFalse() {
-		load("--server.error.include-exception=true", "--server.error.include-message=on-param",
-				"--server.error.include-binding-errors=on-param");
-		bindingExceptionWithoutMessageAndErrors("?message=false&errors=false");
+	void testBindingExceptionForMachineClientWithErrorsParamFalse() {
+		load("--server.error.include-exception=true", "--server.error.include-binding-errors=on-param");
+		bindingExceptionWithoutErrors("?errors=false");
 	}
 
 	@Test
-	void testBindingExceptionForMachineClientWithMessageAndErrorsParamAbsent() {
-		load("--server.error.include-exception=true", "--server.error.include-message=on-param",
-				"--server.error.include-binding-errors=on-param");
-		bindingExceptionWithoutMessageAndErrors("");
+	void testBindingExceptionForMachineClientWithErrorsParamAbsent() {
+		load("--server.error.include-exception=true", "--server.error.include-binding-errors=on-param");
+		bindingExceptionWithoutErrors("");
 	}
 
 	@Test
-	void testBindingExceptionForMachineClientAlwaysMessageAndErrors() {
-		load("--server.error.include-exception=true", "--server.error.include-message=always",
-				"--server.error.include-binding-errors=always");
-		bindingExceptionWithMessageAndErrors("?message=false&errors=false");
+	void testBindingExceptionForMachineClientAlwaysErrors() {
+		load("--server.error.include-exception=true", "--server.error.include-binding-errors=always");
+		bindingExceptionWithErrors("?errors=false");
 	}
 
 	@Test
-	void testBindingExceptionForMachineClientNeverMessageAndErrors() {
-		load("--server.error.include-exception=true", "--server.error.include-message=never",
-				"--server.error.include-binding-errors=never");
-		bindingExceptionWithoutMessageAndErrors("?message=true&errors=true");
+	void testBindingExceptionForMachineClientNeverErrors() {
+		load("--server.error.include-exception=true", "--server.error.include-binding-errors=never");
+		bindingExceptionWithoutErrors("?errors=true");
+	}
+
+	@Test
+	void testBindingExceptionForMachineClientWithMessageParamTrue() {
+		load("--server.error.include-exception=true", "--server.error.include-message=on-param");
+		bindingExceptionWithMessage("?message=true");
+	}
+
+	@Test
+	void testBindingExceptionForMachineClientWithMessageParamFalse() {
+		load("--server.error.include-exception=true", "--server.error.include-message=on-param");
+		bindingExceptionWithoutMessage("?message=false");
+	}
+
+	@Test
+	void testBindingExceptionForMachineClientWithMessageParamAbsent() {
+		load("--server.error.include-exception=true", "--server.error.include-message=on-param");
+		bindingExceptionWithoutMessage("");
+	}
+
+	@Test
+	void testBindingExceptionForMachineClientAlwaysMessage() {
+		load("--server.error.include-exception=true", "--server.error.include-message=always");
+		bindingExceptionWithMessage("?message=false");
+	}
+
+	@Test
+	void testBindingExceptionForMachineClientNeverMessage() {
+		load("--server.error.include-exception=true", "--server.error.include-message=never");
+		bindingExceptionWithoutMessage("?message=true");
+	}
+
+	@SuppressWarnings({ "rawtypes" })
+	private void bindingExceptionWithErrors(String param) {
+		ResponseEntity<Map> entity = new TestRestTemplate().getForEntity(createUrl("/bind" + param), Map.class);
+		assertErrorAttributes(entity.getBody(), "400", "Bad Request", BindException.class, "", "/bind");
+		assertThat(entity.getBody().containsKey("errors")).isTrue();
 	}
 
 	@SuppressWarnings({ "rawtypes" })
-	private void bindingExceptionWithMessageAndErrors(String param) {
+	private void bindingExceptionWithoutErrors(String param) {
+		ResponseEntity<Map> entity = new TestRestTemplate().getForEntity(createUrl("/bind" + param), Map.class);
+		assertErrorAttributes(entity.getBody(), "400", "Bad Request", BindException.class, "", "/bind");
+		assertThat(entity.getBody().containsKey("errors")).isFalse();
+	}
+
+	@SuppressWarnings({ "rawtypes" })
+	private void bindingExceptionWithMessage(String param) {
 		ResponseEntity<Map> entity = new TestRestTemplate().getForEntity(createUrl("/bind" + param), Map.class);
 		assertErrorAttributes(entity.getBody(), "400", "Bad Request", BindException.class,
 				"Validation failed for object='test'. Error count: 1", "/bind");
-		assertThat(entity.getBody().containsKey("errors")).isTrue();
+		assertThat(entity.getBody().containsKey("errors")).isFalse();
 	}
 
 	@SuppressWarnings({ "rawtypes" })
-	private void bindingExceptionWithoutMessageAndErrors(String param) {
+	private void bindingExceptionWithoutMessage(String param) {
 		ResponseEntity<Map> entity = new TestRestTemplate().getForEntity(createUrl("/bind" + param), Map.class);
 		assertErrorAttributes(entity.getBody(), "400", "Bad Request", BindException.class, "", "/bind");
 		assertThat(entity.getBody().containsKey("errors")).isFalse();