CollectionServiceImpl: use NOPLogger for unit tests to reduce output to console.

No functional changes.

Author: php-coder

src/main/java/ru/mystamps/web/config/ServicesConfig.java

@@ -67,7 +67,10 @@ public CategoryService getCategoryService() {
 
 	@Bean
 	public CollectionService getCollectionService() {
-		return new CollectionServiceImpl(daoConfig.getCollectionDao());
+		return new CollectionServiceImpl(
+			LoggerFactory.getLogger(CollectionServiceImpl.class),
+			daoConfig.getCollectionDao()
+		);
 	}
 
 	@Bean

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

@@ -24,7 +24,6 @@
 import org.apache.commons.lang3.Validate;
 
 import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import org.springframework.transaction.annotation.Transactional;
 
@@ -41,8 +40,8 @@
 
 @RequiredArgsConstructor
 public class CollectionServiceImpl implements CollectionService {
-	private static final Logger LOG = LoggerFactory.getLogger(CollectionServiceImpl.class);
 
+	private final Logger log;
 	private final CollectionDao collectionDao;
 
 	@Override
@@ -64,7 +63,7 @@ public void createCollection(Integer ownerId, String ownerLogin) {
 
 		Integer id = collectionDao.add(collection);
 
-		LOG.info("Collection #{} has been created ({})", id, collection);
+		log.info("Collection #{} has been created ({})", id, collection);
 	}
 
 	@Override
@@ -77,7 +76,7 @@ public void addToCollection(Integer userId, Integer seriesId) {
 		collectionDao.addSeriesToUserCollection(userId, seriesId);
 		collectionDao.markAsModified(userId, new Date());
 
-		LOG.info("Series #{} has been added to collection of user #{}", seriesId, userId);
+		log.info("Series #{} has been added to collection of user #{}", seriesId, userId);
 	}
 
 	@Override
@@ -90,7 +89,7 @@ public void removeFromCollection(Integer userId, Integer seriesId) {
 		collectionDao.removeSeriesFromUserCollection(userId, seriesId);
 		collectionDao.markAsModified(userId, new Date());
 
-		LOG.info("Series #{} has been removed from collection of user #{}", seriesId, userId);
+		log.info("Series #{} has been removed from collection of user #{}", seriesId, userId);
 	}
 
 	@Override

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

@@ -20,6 +20,8 @@ package ru.mystamps.web.service
 import spock.lang.Specification
 import spock.lang.Unroll
 
+import org.slf4j.helpers.NOPLogger
+
 import ru.mystamps.web.dao.CollectionDao
 import ru.mystamps.web.dao.dto.AddCollectionDbDto
 import ru.mystamps.web.dao.dto.CollectionInfoDto
@@ -34,7 +36,7 @@ class CollectionServiceImplTest extends Specification {
 	private CollectionService service
 
 	def setup() {
-		service = new CollectionServiceImpl(collectionDao)
+		service = new CollectionServiceImpl(NOPLogger.NOP_LOGGER, collectionDao)
 	}
 
 	//