Skip to content

Commit 0079c25

Browse files
committed
SeriesSalesImportServiceImpl.getParsedData(): add unit teststs.
Fix #835
1 parent 57a79a8 commit 0079c25

File tree

3 files changed

+29
-1
lines changed

3 files changed

+29
-1
lines changed

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@ public void saveParsedData(Integer requestId, SeriesSalesParsedDataDbDto data) {
4545
seriesSalesImportDao.addParsedData(requestId, data);
4646
}
4747

48-
// @todo #695 SeriesSalesImportServiceImpl.getParsedData(): add unit tests
4948
@Override
5049
@Transactional(readOnly = true)
5150
public SeriesSaleParsedDataDto getParsedData(Integer requestId) {

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

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ import org.slf4j.helpers.NOPLogger
2222
import spock.lang.Specification
2323

2424
import ru.mystamps.web.dao.SeriesSalesImportDao
25+
import ru.mystamps.web.dao.dto.SeriesSaleParsedDataDto
2526
import ru.mystamps.web.dao.dto.SeriesSalesParsedDataDbDto
2627
import ru.mystamps.web.tests.Random
2728

@@ -66,4 +67,28 @@ class SeriesSalesImportServiceImplTest extends Specification {
6667
1 * seriesSalesImportDao.addParsedData(expectedRequestId, expectedParsedData)
6768
}
6869

70+
//
71+
// Tests for getParsedData()
72+
//
73+
74+
def 'getParsedData() should throw exception when request id is null'() {
75+
when:
76+
service.getParsedData(null)
77+
then:
78+
IllegalArgumentException ex = thrown()
79+
ex.message == 'Request id must be non null'
80+
}
81+
82+
def 'getParsedData() should invoke dao and return its result'() {
83+
given:
84+
Integer expectedRequestId = Random.id()
85+
SeriesSaleParsedDataDto expectedResult = TestObjects.createSeriesSaleParsedDataDto()
86+
when:
87+
SeriesSaleParsedDataDto result = service.getParsedData(expectedRequestId)
88+
then:
89+
1 * seriesSalesImportDao.findParsedDataByRequestId(expectedRequestId) >> expectedResult
90+
and:
91+
result == expectedResult
92+
}
93+
6994
}

src/test/java/ru/mystamps/web/service/TestObjects.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -296,4 +296,8 @@ public static SeriesSalesParsedDataDbDto createSeriesSalesParsedDataDbDto() {
296296
return dto;
297297
}
298298

299+
public static SeriesSaleParsedDataDto createSeriesSaleParsedDataDto() {
300+
return new SeriesSaleParsedDataDto(Random.id(), Random.price(), Random.currency());
301+
}
302+
299303
}

0 commit comments

Comments
 (0)