Mockito单元测试使用ArgumentCaptor时saveAndFlush空指针问题咨询

Q1 代码片段必要性判断 #

• LabelRequest实例化并设置名称的代码是必须的：调用labelService.create()必须传入请求参数，业务逻辑中会读取request.getName()给Label属性赋值，缺少这部分会导致逻辑无法正常执行，也无法后续校验赋值逻辑的正确性。
• 手动实例化Label并调用setName()的代码是多余的：业务代码的create()方法会自行创建新的Label实例传入私有方法，不会使用你测试类中创建的这个Label对象。如果你是用这个Label作为saveAndFlush的打桩返回值，只需要给它设置uuid属性即可，不需要设置name。

Q2 空指针异常原因 #

你的认知存在错误，当前场景必须对saveAndFlush进行Mock打桩：

你通过@Mock注入的LabelRepository是Mock代理对象，所有非void方法默认返回null。如果不对saveAndFlush做打桩配置，方法调用后会直接返回null，赋值给saved变量后调用saved.getUuid()自然触发空指针异常。

你当前写的打桩代码when(labelRepository.saveAndFlush(any())).thenReturn(label)是正确且必要的，你只需要给返回的Label补全uuid属性即可解决空指针问题：

final Label label = new Label();
label.setUuid(UUID.randomUUID().toString()); // 新增这行配置返回对象的uuid

补充优化点 #

• 你已经使用ArgumentCaptor捕获持久化的Label实例，可以直接校验捕获到的实例name是否和请求参数的name一致，不需要提前手动给测试Label设置name。
• 测试方法定义public void test_create缺少方法括号，属于语法错误需要修正为public void test_create()。

内容的提问来源于stack exchange，提问作者user17186249