SeriesSalesServiceImpl.add(): add more checks for non-nullable fields.

No functional changes.

Author: php-coder

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

@@ -47,6 +47,8 @@ public class SeriesSalesServiceImpl implements SeriesSalesService {
 	@PreAuthorize(HasAuthority.ADD_SERIES_SALES)
 	public void add(AddSeriesSalesDto dto, Integer seriesId, Integer userId) {
 		Validate.isTrue(dto != null, "DTO must be non null");
+		Validate.isTrue(dto.getSellerId() != null, "Seller id must be non null");
+		Validate.isTrue(dto.getPrice() != null, "Price must be non null");
 		Validate.isTrue(dto.getCurrency() != null, "Currency must be non null");
 		Validate.isTrue(seriesId != null, "Series id must be non null");
 		Validate.isTrue(userId != null, "Current user id must be non null");

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

@@ -36,6 +36,8 @@ class SeriesSalesServiceImplTest extends Specification {
 
 	def setup() {
 		form = new AddSeriesSalesForm()
+		form.setSellerId(7)
+		form.setPrice(new BigDecimal('14'))
 		form.setCurrency(Currency.EUR)
 	}
 
@@ -50,9 +52,29 @@ class SeriesSalesServiceImplTest extends Specification {
 			thrown IllegalArgumentException
 	}
 
+	def 'add() should throw exception when seller id is null'() {
+		given:
+			form.setSellerId(null)
+		when:
+			service.add(form, 123, 456)
+		then:
+			thrown IllegalArgumentException
+	}
+	
+	def 'add() should throw exception when price is null'() {
+		given:
+			form.setPrice(null)
+		when:
+			service.add(form, 123, 456)
+		then:
+			thrown IllegalArgumentException
+	}
+	
 	def 'add() should throw exception when currency is null'() {
+		given:
+			form.setCurrency(null)
 		when:
-			service.add(new AddSeriesSalesForm(), 123, 456)
+			service.add(form, 123, 456)
 		then:
 			thrown IllegalArgumentException
 	}