Skip to content

Commit 776d581

Browse files
committed
test: add unit test for hidden images to SeriesServiceImpl.findFullInfoById()
Fix #1385
1 parent 79c7e14 commit 776d581

File tree

2 files changed

+17
-1
lines changed

2 files changed

+17
-1
lines changed

src/main/java/ru/mystamps/web/feature/series/SeriesServiceImpl.java

-1
Original file line numberDiff line numberDiff line change
@@ -335,7 +335,6 @@ public SeriesDto findFullInfoById(
335335

336336
List<Integer> imageIds = imageService.findBySeriesId(seriesId, false);
337337

338-
// @todo #1356 SeriesServiceImpl.findFullInfoById(): add unit test for hidden images
339338
List<Integer> hiddenImageIds = Collections.emptyList();
340339
if (userCanSeeHiddenImages) {
341340
hiddenImageIds = imageService.findBySeriesId(seriesId, true);

src/test/groovy/ru/mystamps/web/feature/series/SeriesServiceImplTest.groovy

+17
Original file line numberDiff line numberDiff line change
@@ -800,6 +800,23 @@ class SeriesServiceImplTest extends Specification {
800800

801801
result.zagorski?.numbers == expectedZagorskiNumbers
802802
result.zagorski?.price == expectedInfo.zagorskiPrice
803+
804+
result.hiddenImageIds?.isEmpty()
805+
}
806+
807+
def 'findFullInfoById() should load hidden images when user can see them'() {
808+
given:
809+
Integer expectedSeriesId = Random.id()
810+
List<Integer> expectedImageIds = Random.listOfIntegers()
811+
and:
812+
seriesDao.findByIdAsSeriesFullInfo(expectedSeriesId, _ as Integer, _ as String) >>
813+
TestObjects.createSeriesFullInfoDto()
814+
when:
815+
SeriesDto result = service.findFullInfoById(expectedSeriesId, Random.userId(), Random.lang(), true)
816+
then:
817+
1 * imageService.findBySeriesId(expectedSeriesId, true) >> expectedImageIds
818+
and:
819+
result?.hiddenImageIds == expectedImageIds
803820
}
804821

805822
//

0 commit comments

Comments
 (0)