SeriesImportService: rename add() to addRequest().

Prerequisite for #700

No functional changes.

Author: php-coder

src/main/java/ru/mystamps/web/controller/SeriesImportController.java

@@ -80,7 +80,7 @@ public String processRequestImportForm(
 			return null;
 		}
 
-		Integer requestId = seriesImportService.add(form, currentUserId);
+		Integer requestId = seriesImportService.addRequest(form, currentUserId);
 
 		ImportRequestCreated requestCreated =
 			new ImportRequestCreated(this, requestId, form.getUrl());

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

@@ -23,7 +23,7 @@
 import ru.mystamps.web.service.dto.RequestImportDto;
 
 public interface SeriesImportService {
-	Integer add(RequestImportDto dto, Integer userId);
+	Integer addRequest(RequestImportDto dto, Integer userId);
 	void changeStatus(Integer requestId, String oldStatus, String newStatus);
 	ImportRequestDto findById(Integer requestId);
 	void saveDownloadedContent(Integer requestId, String content);

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

@@ -59,7 +59,7 @@ public class SeriesImportServiceImpl implements SeriesImportService {
 	@Transactional
 	@PreAuthorize(HasAuthority.IMPORT_SERIES)
 	@SuppressWarnings({ "PMD.NPathComplexity", "PMD.ModifiedCyclomaticComplexity" })
-	public Integer add(RequestImportDto dto, Integer userId) {
+	public Integer addRequest(RequestImportDto dto, Integer userId) {
 		Validate.isTrue(dto != null, "DTO must be non null");
 		Validate.isTrue(userId != null, "Current user id must be non null");
 

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

@@ -49,33 +49,33 @@ class SeriesImportServiceImplTest extends Specification {
 	}
 
 	//
-	// Tests for add()
+	// Tests for addRequest()
 	//
 
-	def 'add() should throw exception if dto is null'() {
+	def 'addRequest() should throw exception if dto is null'() {
 		when:
-			service.add(null, Random.userId())
+			service.addRequest(null, Random.userId())
 		then:
 			thrown IllegalArgumentException
 	}
 
-	def 'add() should throw exception if user id is null'() {
+	def 'addRequest() should throw exception if user id is null'() {
 		when:
-			service.add(form, null)
+			service.addRequest(form, null)
 		then:
 			thrown IllegalArgumentException
 	}
 
 	@SuppressWarnings(['ClosureAsLastMethodParameter', 'UnnecessaryReturnKeyword'])
-	def 'add() should pass dto to dao and return its result'() {
+	def 'addRequest() should pass dto to dao and return its result'() {
 		given:
 			String expectedUrl = Random.url()
 			form.setUrl(expectedUrl)
 		and:
 			Integer expectedUserId = Random.userId()
 			Integer expectedResult = Random.id()
 		when:
-			Integer result = service.add(form, expectedUserId)
+			Integer result = service.addRequest(form, expectedUserId)
 		then:
 			1 * seriesImportDao.add({ ImportSeriesDbDto request ->
 				assert request.url == expectedUrl