CategoryService.findIdsByNames(): change argument type.

Follow-up to 1c0d64f commit.

No functional changes.

Author: php-coder

src/main/java/ru/mystamps/web/dao/CategoryDao.java

@@ -19,7 +19,6 @@
 
 import java.util.Date;
 import java.util.List;
-import java.util.Set;
 
 import ru.mystamps.web.dao.dto.AddCategoryDbDto;
 import ru.mystamps.web.dao.dto.EntityWithParentDto;
@@ -36,7 +35,7 @@ public interface CategoryDao {
 	long countAddedSince(Date date);
 	long countUntranslatedNamesSince(Date date);
 	List<Object[]> getStatisticsOf(Integer collectionId, String lang);
-	List<Integer> findIdsByNames(Set<String> names);
+	List<Integer> findIdsByNames(List<String> names);
 	List<Integer> findIdsByNamePattern(String pattern);
 	List<LinkEntityDto> findAllAsLinkEntities(String lang);
 	LinkEntityDto findOneAsLinkEntity(String slug, String lang);

src/main/java/ru/mystamps/web/dao/impl/JdbcCategoryDao.java

@@ -22,7 +22,6 @@
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
-import java.util.Set;
 
 import org.apache.commons.lang3.Validate;
 
@@ -190,7 +189,7 @@ public List<Object[]> getStatisticsOf(Integer collectionId, String lang) {
 	}
 
 	@Override
-	public List<Integer> findIdsByNames(Set<String> names) {
+	public List<Integer> findIdsByNames(List<String> names) {
 		return jdbcTemplate.query(
 			findIdsByNamesSql,
 			Collections.singletonMap("names", names),

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

@@ -19,7 +19,6 @@
 
 import java.util.Date;
 import java.util.List;
-import java.util.Set;
 
 import ru.mystamps.web.dao.dto.EntityWithParentDto;
 import ru.mystamps.web.dao.dto.LinkEntityDto;
@@ -28,7 +27,7 @@
 @SuppressWarnings("PMD.TooManyMethods")
 public interface CategoryService {
 	String add(AddCategoryDto dto, Integer userId);
-	List<Integer> findIdsByNames(Set<String> names);
+	List<Integer> findIdsByNames(List<String> names);
 	List<Integer> findIdsWhenNameStartsWith(String name);
 	List<LinkEntityDto> findAllAsLinkEntities(String lang);
 	List<EntityWithParentDto> findCategoriesWithParents(String lang);

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

@@ -21,7 +21,6 @@
 import java.util.Date;
 import java.util.List;
 import java.util.Locale;
-import java.util.Set;
 
 import org.apache.commons.lang3.StringUtils;
 import org.apache.commons.lang3.Validate;
@@ -84,7 +83,7 @@ public String add(AddCategoryDto dto, Integer userId) {
 
 	@Override
 	@Transactional(readOnly = true)
-	public List<Integer> findIdsByNames(Set<String> names) {
+	public List<Integer> findIdsByNames(List<String> names) {
 		if (names == null || names.isEmpty()) {
 			return Collections.emptyList();
 		}

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

@@ -83,9 +83,9 @@ protected List<Integer> extractCategory(String fragment) {
 		log.debug("Determining category from a fragment: '{}'", fragment);
 
 		String[] candidates = StringUtils.split(fragment, "\n\t ,");
-		Set<String> uniqueCandidates = Arrays.stream(candidates)
+		List<String> uniqueCandidates = Arrays.stream(candidates)
 			.distinct()
-			.collect(Collectors.toSet());
+			.collect(Collectors.toList());
 
 		log.debug("Possible candidates: {}", uniqueCandidates);
 

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

@@ -147,14 +147,14 @@ class CategoryServiceImplTest extends Specification {
 
 	def 'findIdsByNames() should return empty result when no names are specified'() {
 		given:
-			Set<String> names = nullOr(Collections.emptySet())
+			List<String> names = nullOr(Collections.emptyList())
 		expect:
 			service.findIdsByNames(names) == []
 	}
 
 	def 'findIdsByNames() should invoke dao, pass argument and return result from dao'() {
 		given:
-			Set<String> expectedNames = Random.setOfStrings()
+			List<String> expectedNames = Random.listOfStrings()
 			List<Integer> expectedResult = Random.listOfIntegers()
 		when:
 			List<Integer> result = service.findIdsByNames(expectedNames)

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

@@ -60,7 +60,7 @@ class SeriesInfoExtractorServiceImplTest extends Specification {
 	def 'extractCategory() should try to search by category names'() {
 		given:
 			String fragment = 'Lorem ipsum   dolor\tsit\namet,'
-			Set<String> expectedCandidates = [ 'Lorem', 'ipsum', 'dolor', 'sit', 'amet' ]
+			List<String> expectedCandidates = [ 'Lorem', 'ipsum', 'dolor', 'sit', 'amet' ]
 		and:
 			List<Integer> expectedResult = Random.listOfIntegers()
 		when:
@@ -74,7 +74,7 @@ class SeriesInfoExtractorServiceImplTest extends Specification {
 	def 'extractCategory() should deduplicate candidates'() {
 		given:
 			String fragment = 'foo bar foo'
-			Set<String> expectedCandidates = [ 'foo', 'bar' ]
+			List<String> expectedCandidates = [ 'foo', 'bar' ]
 		when:
 			service.extractCategory(fragment)
 		then:
@@ -88,7 +88,7 @@ class SeriesInfoExtractorServiceImplTest extends Specification {
 			List<Integer> result = service.extractCategory('foo1 foo2')
 		then:
 			// in order to search by prefix, we shouldn't find anything by name
-			1 * categoryService.findIdsByNames(_ as Set<String>) >> Collections.emptyList()
+			1 * categoryService.findIdsByNames(_ as List<String>) >> Collections.emptyList()
 		and:
 			// the first lookup will find nothing
 			1 * categoryService.findIdsWhenNameStartsWith('foo1') >> Collections.emptyList()
@@ -103,7 +103,7 @@ class SeriesInfoExtractorServiceImplTest extends Specification {
 		when:
 			List<Integer> result = service.extractCategory('foo')
 		then:
-			1 * categoryService.findIdsByNames(_ as Set<String>) >> Collections.emptyList()
+			1 * categoryService.findIdsByNames(_ as List<String>) >> Collections.emptyList()
 		and:
 			1 * categoryService.findIdsWhenNameStartsWith(_ as String) >> Collections.emptyList()
 		and:

src/test/java/ru/mystamps/web/tests/Random.java

@@ -177,6 +177,13 @@ public static String catalogNumber() {
 		return String.valueOf(between(1, maxCatalogNumber).integer());
 	}
 
+	public static List<String> listOfStrings() {
+		final int minSize = 1;
+		final int maxSize = 3;
+		int size = integer(minSize, maxSize);
+		return sampleMultiple(size, "foo", "bar", "baz");
+	}
+	
 	public static Set<String> setOfStrings() {
 		final int minSize = 1;
 		final int maxSize = 3;