通过单元测试测试服务方法？ [关闭]

Question

我没有测试经验，并尝试通过单元测试来测试方法。我看过的所有示例都通过使用模拟值来执行操作。我知道，我还将在我的项目中使用带有mockito 的模拟值。这是我要测试的服务方法：

ProductServiceImpl：

public List<ProductDTO> findAllByCategoryUuid(UUID categoryUuid) {

    // code omitted

    return result;
}

这是我的单元测试类：

ProductServiceImplTest：

// ? @InjectMocks
@Autowired 
ProductServiceImpl productService;

@Mock
ProductRepository productRepository;
  

@Test
public void testFindAllByCategoryUuid() {

    UUID categoryUuid = UUID.randomUUID();

    final List<Product> productList = new ArrayList<>();
    for (int i = 0; i < size; i++) {
        // create product by setting "categoryUuid" and add to productList
    }
    productRepository.saveAll(productList);


    when(productService.findAllByCategoryUuid(categoryUuid)
                .thenReturn(productList);
}

我的问题：

1. 为了测试服务方法，上述方法是否正确？我认为我不应该在服务方法内部处理，只需通过categoryUuid 并检查该方法的结果进行测试？这是真的吗？

2. 在测试类中，我使用@Autowired 访问服务方法，但我不确定是否应该@Mock。有错吗？

任何帮助将不胜感激。

更新：我还使用 DiffBlue 插件创建单元测试，它会生成如下所示的测试方法。但我认为它似乎是作为测试存储库方法而不是服务方法。不是吗？

@Test
public void testFindAllByCategoryUuid() {
    when(this.productRepository.findAllByCategoryUuid((UUID) any()))
        .thenReturn(new ArrayList<Product>());
    assertTrue(this.productService.findAllByCategoryUuid(UUID.randomUUID())
        .isEmpty());
    verify(this.productRepository).findAllByCategoryUuid((UUID) any());
    // ...
}

Answer 1

我不是专家，但我会尽力回答你的问题

对您的方法进行单元测试的一般方法应该是针对所有可能的输入集测试输出。 在您的具体情况下，您可以测试

• 输入：现有 UUID 输出：NonNull 列表。
• 输入：不存在的 UUID 输出：空列表。
• 输入：null：空列表。

现在您在这里所做的是正确的，您需要自动装配您正在为其编写测试用例的类并模拟该类中的依赖项。 唯一的问题是

when(productService.findAllByCategoryUuid(categoryUuid)
                .thenReturn(productList);

应该是

when(productRepository.findAllByCategoryUuid(categoryUuid)
                .thenReturn(productList);   

您在这里模拟 productRepository.findAllByCategoryUuid，因为您的目标是测试服务类中的方法。

在此之后，只需为上述所有条件添加适当的断言语句。

此外，每当针对某些代码记录错误时，我通常都会遵循一条规则，我尝试在我的 Junit 中使用 assert 来覆盖输入和输出情况，以便每次我都会测试所有可能的输入和输出场景。

Answer 2

使用 Mockito 编写 Junit 测试时要记住的重要事项

1. 所有类级别@Runwith()
2. 测试类应该使用@InjectMocks
3. 所有测试都应使用@Test 注释
4. 应使用@Mock 模拟任何外部服务
5. 应模拟任何对 DB 或其他服务的调用，并应相应地返回值。
6. 您应该有断言来测试您的结果。

我会这样写：

@RunWith(MockitoJUnitRunner.class)
public class ProductServiceImplTest {

@InjectMocks
ProductServiceImpl productService;

@Mock
ProductRepository productRepository;

@Test
public void testFindAllByCategoryUuid() {

    UUID categoryUuid = UUID.randomUUID();

    final List<Product> productList = new ArrayList<>();
    for (int i = 0; i < size; i++) {
        // create product by setting "categoryUuid" and add to productList
    }
    
    when(productRepository.saveAll(ArgumentMatchers.any()).thenReturn(productList); 
// Or below might work for newer version of test cases when we get Null Pointer Exp using older version of Junit test cases

//doReturn(productList).when(productRepository).saveAll(any(List.class));
    
    List<ProductDTO> response = productService.findAllByCategoryUuid(categoryUuid);
    
    Assert.assertNotNull(response);
    Assert.assertEquals("Object Value", response.getXXX());
}

Answer 3

针对服务层编写单元测试有缺点：

1. 您违反了被测方法的封装。如果它因为您开始调用不同的类/方法而改变 - 测试将中断。即使该方法可能工作正常。
2. 因为您打算使用模拟，部分测试将只是检查您的模拟是否已设置为通过测试。所以基本上你将测试测试逻辑。

将逻辑向下移动通常更有效率，例如到模型。这些类可以在没有模拟的情况下进行单元测试。然后你可以编写一个更高级别的测试（包括 DB）来检查一切是否一起工作。

阅读：

• Anemic architecture - enemy of testing
• Building Test Pyramid to optimize automated testing