JUnit5+Mockito测试getUserByEmail返回Null致测试失败求助

测试getUserByEmail方法时返回null，核心问题在于Mock对象的行为未正确模拟，且测试逻辑存在错误：

1. 测试类的核心错误 #

• 你模拟了userRepository.save()的返回，但getUserByEmail实际调用的是userRepository.findAll()，mock的save行为对查询无任何作用
• 直接调用userRepository.save(userdetails2)不会将数据存入mock仓库，mock对象不会自动维护数据状态
• 断言逻辑错误：userService.getUserByEmail返回Optional<UserDetails>，直接与字符串比较会导致类型不匹配，且未正确提取Optional中的值

2. 修复后的测试类代码 #

@SpringBootTest
class UserServiceTest {
    
    @MockBean
    private UsersRepository userRepository;
    
    @Autowired
    private UserService userService;

    @Test
    void check_email_existInDB_thenReturn_User() {
        // given - 准备测试数据并模拟findAll的返回
        UserDetails userdetails2 = new UserDetails(101L, "Anthony Ji", "anthony@gmail.com", "password2");   
        List<UserDetails> userList = Collections.singletonList(userdetails2);
        when(userRepository.findAll()).thenReturn(userList);
        
        // when - 执行测试方法
        Optional<UserDetails> result = userService.getUserByEmail("anthony@gmail.com");
        
        // then - 验证结果
        assertTrue(result.isPresent());
        assertEquals(userdetails2.getEmail(), result.get().getEmail());
    }
}

3. 额外优化建议 #

优化Repository查询逻辑 #

当前getUserByEmail通过findAll()全表查询再过滤，性能极低，建议在UsersRepository中添加专用查询方法：

public interface UsersRepository extends JpaRepository<UserDetails, Long>{
    Optional<UserDetails> findByEmailIgnoreCase(String email);
}

修改UserServiceImpl的实现：

@Override
public Optional<UserDetails> getUserByEmail(String email) {
    return userRepository.findByEmailIgnoreCase(email);
}

对应的测试可改为模拟findByEmailIgnoreCase的返回：

when(userRepository.findByEmailIgnoreCase("anthony@gmail.com")).thenReturn(Optional.of(userdetails2));

测试注解简化 #

@SpringBootTest已集成Mockito支持，无需额外添加@ExtendWith(MockitoExtension.class)

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