#1920 - Support for JSR-303 @SiZe annotation in property metadata.

Using the JSR-303 @SiZe annotation is now reflected in the property being considered for input type range as well as exposed min and max values.

Author: odrotbohm

src/main/java/org/springframework/hateoas/mediatype/PropertyUtils.java

@@ -34,6 +34,7 @@
 import javax.validation.constraints.NotBlank;
 import javax.validation.constraints.NotNull;
 import javax.validation.constraints.Pattern;
+import javax.validation.constraints.Size;
 
 import org.reactivestreams.Publisher;
 import org.springframework.beans.BeanUtils;
@@ -558,6 +559,7 @@ private static class Jsr303AwarePropertyMetadata extends DefaultPropertyMetadata
 
 			Map<Class<? extends Annotation>, String> typeMap = new HashMap<>();
 			typeMap.put(Email.class, "email");
+			typeMap.put(Size.class, "range");
 
 			if (URL_ANNOTATION != null) {
 				typeMap.put(URL_ANNOTATION, "url");
@@ -617,8 +619,9 @@ public Optional<String> getPattern() {
 		@Override
 		public Number getMin() {
 
-			return Optional.ofNullable(RANGE_ANNOTATION) //
-					.flatMap(it -> getAnnotationAttribute(it, "min", Number.class)) //
+			return getAnnotationAttribute(Size.class, "min", Number.class) //
+					.or(() -> Optional.ofNullable(RANGE_ANNOTATION)
+							.flatMap(it -> getAnnotationAttribute(it, "min", Number.class))) //
 					.or(() -> getAnnotationAttribute(Min.class, "value", Number.class)) //
 					.or(() -> parsePropertyAnnotationValue(DecimalMin.class)) //
 					.orElse(null);
@@ -632,8 +635,9 @@ public Number getMin() {
 		@Override
 		public Number getMax() {
 
-			return Optional.ofNullable(RANGE_ANNOTATION) //
-					.flatMap(it -> getAnnotationAttribute(it, "max", Number.class)) //
+			return getAnnotationAttribute(Size.class, "max", Number.class) //
+					.or(() -> Optional.ofNullable(RANGE_ANNOTATION)
+							.flatMap(it -> getAnnotationAttribute(it, "max", Number.class))) //
 					.or(() -> getAnnotationAttribute(Max.class, "value", Number.class)) //
 					.or(() -> parsePropertyAnnotationValue(DecimalMax.class)) //
 					.orElse(null);

src/test/java/org/springframework/hateoas/mediatype/PropertyUtilsTest.java

@@ -34,6 +34,7 @@
 import javax.validation.constraints.NotBlank;
 import javax.validation.constraints.NotNull;
 import javax.validation.constraints.Pattern;
+import javax.validation.constraints.Size;
 
 import org.hibernate.validator.constraints.Range;
 import org.hibernate.validator.constraints.URL;
@@ -187,7 +188,8 @@ Stream<DynamicTest> exposesInputTypeForProperties() {
 				InputTypes.of("url", HtmlInputType.URL), //
 				InputTypes.of("stringUrl", HtmlInputType.URL), //
 				InputTypes.of("email", HtmlInputType.EMAIL), //
-				InputTypes.of("ranged", HtmlInputType.RANGE));
+				InputTypes.of("ranged", HtmlInputType.RANGE),
+				InputTypes.of("sized", HtmlInputType.RANGE));
 
 		InputPayloadMetadata metadata = PropertyUtils.getExposedProperties(InputTypeSample.class);
 
@@ -225,6 +227,18 @@ void considersJsr303NotBlankAnnotation() {
 		});
 	}
 
+	@Test // #1920
+	void exposesMinAndMaxFromJsr303AtSizeAnnotation() {
+
+		InputPayloadMetadata metadata = PropertyUtils.getExposedProperties(Jsr303SamplePayload.class);
+		Optional<PropertyMetadata> property = metadata.stream().filter(it -> it.getName().equals("sized")).findFirst();
+
+		assertThat(property).hasValueSatisfying(it -> {
+			assertThat(it.getMin()).isEqualTo(41);
+			assertThat(it.getMax()).isEqualTo(4711);
+		});
+	}
+
 	@Data
 	@AllArgsConstructor
 	@JsonIgnoreProperties({ "ignoreThisProperty" })
@@ -271,6 +285,7 @@ static class Jsr303SamplePayload {
 		@Pattern(regexp = "\\w") String pattern;
 		@NotBlank @Pattern(regexp = "\\w") String nonBlankPattern;
 		TypeAnnotated annotated;
+		@Size(min = 41, max = 4711) int sized;
 	}
 
 	@Pattern(regexp = "regex")
@@ -316,6 +331,8 @@ static class InputTypeSample {
 		@URL String stringUrl;
 
 		@Range int ranged;
+
+		@Size int sized;
 	}
 
 	@Value