Polish tests

Author: sbrannen

spring-web/src/test/java/org/springframework/web/method/annotation/ExceptionHandlerMethodResolverTests.java

@@ -208,54 +208,47 @@ public void handle() {
 	@Controller
 	static class MediaTypeController {
 
-		@ExceptionHandler(exception = {IllegalArgumentException.class}, produces = "application/json")
+		@ExceptionHandler(exception = IllegalArgumentException.class, produces = "application/json")
 		public void handleJson() {
-
 		}
 
-		@ExceptionHandler(exception = {IllegalArgumentException.class}, produces = {"text/html", "*/*"})
+		@ExceptionHandler(exception = IllegalArgumentException.class, produces = {"text/html", "*/*"})
 		public void handleHtml() {
-
 		}
 
 	}
 
 	@Controller
 	static class AmbiguousMediaTypeController {
 
-		@ExceptionHandler(exception = {IllegalArgumentException.class}, produces = "application/json")
+		@ExceptionHandler(exception = IllegalArgumentException.class, produces = "application/json")
 		public void handleJson() {
-
 		}
 
-		@ExceptionHandler(exception = {IllegalArgumentException.class}, produces = "application/json")
+		@ExceptionHandler(exception = IllegalArgumentException.class, produces = "application/json")
 		public void handleJsonToo() {
-
 		}
 
 	}
 
 	@Controller
 	static class MixedController {
 
-		@ExceptionHandler(exception = {IllegalArgumentException.class}, produces = "application/json")
+		@ExceptionHandler(exception = IllegalArgumentException.class, produces = "application/json")
 		public void handleJson() {
-
 		}
 
 		@ExceptionHandler(IllegalArgumentException.class)
 		public void handleOther() {
-
 		}
 
 	}
 
 	@Controller
 	static class InvalidMediaTypeController {
 
-		@ExceptionHandler(exception = {IllegalArgumentException.class}, produces = "invalid-mediatype")
+		@ExceptionHandler(exception = IllegalArgumentException.class, produces = "invalid-mediatype")
 		public void handle() {
-
 		}
 	}
 

spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/ExceptionHandlerExceptionResolverTests.java

@@ -544,7 +544,7 @@ static class IoExceptionController {
 
 		public void handle() {}
 
-		@ExceptionHandler(value = IOException.class)
+		@ExceptionHandler(IOException.class)
 		public void handleException() {
 		}
 	}

spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/method/annotation/ResponseEntityExceptionHandlerTests.java

@@ -97,22 +97,21 @@ class ResponseEntityExceptionHandlerTests {
 	private WebRequest request = new ServletWebRequest(this.servletRequest, this.servletResponse);
 
 
-	@SuppressWarnings("unchecked")
 	@Test
 	void supportsAllDefaultHandlerExceptionResolverExceptionTypes() throws Exception {
-
 		ExceptionHandler annotation = ResponseEntityExceptionHandler.class
 				.getMethod("handleException", Exception.class, WebRequest.class)
 				.getAnnotation(ExceptionHandler.class);
+		Class<?>[] exceptionTypes = annotation.value();
 
 		Arrays.stream(DefaultHandlerExceptionResolver.class.getDeclaredMethods())
 				.filter(method -> method.getName().startsWith("handle") && (method.getParameterCount() == 4))
 				.filter(method -> !method.getName().equals("handleErrorResponse"))
 				.filter(method -> !method.getName().equals("handleDisconnectedClientException"))
 				.map(method -> method.getParameterTypes()[0])
-				.forEach(exceptionType -> assertThat(annotation.value())
+				.forEach(exceptionType -> assertThat(exceptionTypes)
 						.as("@ExceptionHandler is missing declaration for " + exceptionType.getName())
-						.contains((Class<Exception>) exceptionType));
+						.contains(exceptionType));
 	}
 
 	@Test