SeriesInfoExtractorServiceImpl.extractQuantity(): respect MAX_STAMPS_IN_SERIES.

Fix #789

Author: php-coder

src/main/java/ru/mystamps/web/service/SeriesInfoExtractorServiceImpl.java

@@ -227,7 +227,6 @@ protected Integer extractReleaseYear(String fragment) {
 		return null;
 	}
 
-	// @todo #781 SeriesInfoExtractorServiceImpl.extractQuantity() respect MAX_STAMPS_IN_SERIES
 	protected Integer extractQuantity(String fragment) {
 		if (StringUtils.isBlank(fragment)) {
 			return null;
@@ -237,9 +236,11 @@ protected Integer extractQuantity(String fragment) {
 
 		Matcher matcher = NUMBER_OF_STAMPS_REGEXP.matcher(fragment);
 		if (matcher.find()) {
-			String quantity = matcher.group("quantity");
-			log.debug("Quantity is {}", quantity);
-			return Integer.valueOf(quantity);
+			Integer quantity = Integer.valueOf(matcher.group("quantity"));
+			if (quantity <= ValidationRules.MAX_STAMPS_IN_SERIES) {
+				log.debug("Quantity is {}", quantity);
+				return quantity;
+			}
 		}
 
 		log.debug("Could not extract quantity from a fragment");

src/test/groovy/ru/mystamps/web/service/SeriesInfoExtractorServiceImplTest.groovy

@@ -22,6 +22,7 @@ import static io.qala.datagen.RandomShortApi.nullOrBlank
 import static io.qala.datagen.RandomValue.between
 
 import static ru.mystamps.web.service.SeriesInfoExtractorServiceImpl.MAX_SUPPORTED_RELEASE_YEAR
+import static ru.mystamps.web.validation.ValidationRules.MAX_STAMPS_IN_SERIES
 
 import java.time.Year
 
@@ -334,8 +335,9 @@ class SeriesInfoExtractorServiceImplTest extends Specification {
 		expect:
 			service.extractQuantity(fragment) == null
 		where:
-			fragment  | _
-			'0 марок' | _
+			fragment                              | _
+			'0 марок'                             | _
+			(MAX_STAMPS_IN_SERIES + 1) + ' марок' | _
 	}
 
 	@Unroll