Skip to content

Commit d20e632

Browse files
committed
TimedImagePreviewStrategy: use NOPLogger for unit tests to reduce output to console.
No functional changes.
1 parent 3e14652 commit d20e632

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

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

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,10 @@ public ImageService getImageService() {
9494
return new ImageServiceImpl(
9595
LoggerFactory.getLogger(ImageServiceImpl.class),
9696
strategiesConfig.getImagePersistenceStrategy(),
97-
new TimedImagePreviewStrategy(new ThumbnailatorImagePreviewStrategy()),
97+
new TimedImagePreviewStrategy(
98+
LoggerFactory.getLogger(TimedImagePreviewStrategy.class),
99+
new ThumbnailatorImagePreviewStrategy()
100+
),
98101
daoConfig.getImageDao()
99102
);
100103
}

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

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,13 @@
2121
import org.apache.commons.lang3.time.StopWatch;
2222

2323
import org.slf4j.Logger;
24-
import org.slf4j.LoggerFactory;
2524

2625
import lombok.RequiredArgsConstructor;
2726

2827
@RequiredArgsConstructor
2928
public class TimedImagePreviewStrategy implements ImagePreviewStrategy {
30-
private static final Logger LOG = LoggerFactory.getLogger(TimedImagePreviewStrategy.class);
3129

30+
private final Logger log;
3231
private final ImagePreviewStrategy strategy;
3332

3433
@Override
@@ -46,7 +45,7 @@ public byte[] createPreview(byte[] image) {
4645
byte[] result = strategy.createPreview(image);
4746
timer.stop();
4847

49-
LOG.debug(
48+
log.debug(
5049
"Image preview has been generated in {} msecs: {} -> {} bytes",
5150
timer.getTime(),
5251
// let's hope that it won't throw IllegalStateException :)

0 commit comments

Comments
 (0)