最近,我们在系统开发过程中遇到了一些冲突。我们发现,我们团队中有3种不同的测试方法,我们需要确定哪种方法最好,并检查是否有比这更好的方法。
首先,让我们面对一些事实:
-我们系统中有3个数据层(DTO,域对象,表)
-我们正在使用通过mapstruct生成的映射器将每个图层的对象映射到另一个
-我们正在使用Mockito
-我们正在对每个图层进行单元测试
现在发生冲突:假设我们要测试ExampleService
,它使用ExampleModelMapper
将ExampleModel
映射到ExampleModelDto
,并执行一些需要测试的附加业务逻辑。我们可以通过三种不同的方式来验证返回数据的正确性:
a)我们可以手动将返回对象的每个字段与预期结果进行比较:
assertThat(returnedDto)
.isNotNull()
.hasFieldOrPropertyWithValue("id", expectedEntity.getId())
.hasFieldOrPropertyWithValue("address", expectedEntity.getAddress())
.hasFieldOrPropertyWithValue("orderId", expectedEntity.getOrderId())
.hasFieldOrPropertyWithValue("creationTimestamp", expectedEntity.getCreationTimestamp())
.hasFieldOrPropertyWithValue("price", expectedEntity.getPrice())
.hasFieldOrPropertyWithValue("successCallbackUrl", expectedEntity.getSuccessCallbackUrl())
.hasFieldOrPropertyWithValue("failureCallbackUrl", expectedEntity.getFailureCallbackUrl())
b)我们可以使用真实的映射器(与普通逻辑相同)来比较两个对象:
assertThat(returnedDto).isEqualToComparingFieldByFieldRecursivly(mapper.mapToDto(expectedEntity)))
c)最后,我们可以模拟映射器及其响应:
final Entity entity = randomEntity();
final Dto dto = new Dto(entity.getId(), entity.getName(), entity.getOtherField());
when(mapper.mapToDto(entity)).thenReturn(dto);
我们希望在保持弹性和抗变化性的同时尽可能地进行测试。我们还希望遵守DRY原则。
我们很高兴听到每种方法的任何建议,评论,优点和缺点。我们也愿意看到其他解决方案。
问候。
答案 0 :(得分:1)
要测试ExampleService
,我认为模拟mapper及其响应是一个好主意,将行为与Mapper测试和MapperImpl
测试分开。
但是,您需要对Mapper
实例进行单元测试,我更喜欢使用模拟数据进行测试,也可以使用夹具进行测试。
要测试Mapper中引入的业务逻辑(映射规则),可以针对MapperImpl
类进行测试。
答案 1 :(得分:1)
在这里我建议有两种选择。
如果要进行单元测试,则在服务(其他依赖项以及OFC)中模拟您的映射器,并仅测试服务逻辑。为映射器编写一个单独的单元测试套件。我在这里创建了一个代码示例:https://github.com/jannis-baratheon/stackoverflow/tree/master/52255896-proper-way-of-using-and-testing-generated-mapper。
示例摘录:
服务类别:
public class AService {
private final ARepository repository;
private final EntityMapper mapper;
public AService(ARepository repository, EntityMapper mapper) {
this.repository = repository;
this.mapper = mapper;
}
public ADto getResource(int id) {
AnEntity entity = repository.getEntity(id);
return mapper.toDto(entity);
}
}
映射器:
import org.mapstruct.Mapper;
@Mapper
public interface EntityMapper {
ADto toDto(AnEntity entity);
}
服务单元测试:
import static org.assertj.core.api.Assertions.assertThat;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;
import org.junit.Before;
import org.junit.Test;
public class AServiceTest {
private EntityMapper mapperMock;
private ARepository repositoryMock;
private AService sut;
@Before
public void setup() {
repositoryMock = mock(ARepository.class);
mapperMock = mock(EntityMapper.class);
sut = new AService(repositoryMock, mapperMock);
}
@Test
public void shouldReturnResource() {
// given
AnEntity mockEntity = mock(AnEntity.class);
ADto mockDto = mock(ADto.class);
when(repositoryMock.getEntity(42))
.thenReturn(mockEntity);
when(mapperMock.toDto(mockEntity))
.thenReturn(mockDto);
// when
ADto resource = sut.getResource(42);
// then
assertThat(resource)
.isSameAs(mockDto);
}
}
映射器单元测试:
import static org.assertj.core.api.Assertions.assertThat;
import org.junit.Before;
import org.junit.Test;
public class EntityMapperTest {
private EntityMapperImpl sut;
@Before
public void setup() {
sut = new EntityMapperImpl();
}
@Test
public void shouldMapEntityToDto() {
// given
AnEntity entity = new AnEntity();
entity.setId(42);
// when
ADto aDto = sut.toDto(entity);
// then
assertThat(aDto)
.hasFieldOrPropertyWithValue("id", 42);
}
}
第二个选项是进行集成测试,在该测试中,您需要向服务中注入真实的映射器。我强烈建议不要在集成测试中花太多精力来验证映射逻辑。很可能会变得凌乱。只需对映射进行烟雾测试,并分别为映射器编写单元测试。
总结:
我通常选择第二个选项,在其中我用MockMvc
测试主应用程序路径,并为较小的单元编写完整的单元测试。