0

我得说我是 Mockito 和 TDD 的新手.. 我正在努力寻找使用它的理由.. 尤其是当我发现以下问题时.. 我想测试一个“RegisterUser”方法我的 UserService 类 .. 我正在使用 spring mvc 3.0.6、Hibernate、Maven 等等。

@Override
public void registerUser(User user, UserRoles userRole) throws DataAccessException, UserExistingException {
    checkExistingUser(user);
    user.addRole(new Role(userRole));
    String password = encryptPassword(user);
    userRepository.makePersistent(user);
    sendWelcomeEmail(user.getFirstname(), user.getUsername(), password, user.getEmail());
}

private void checkExistingUser(User user) throws UserExistingException {
    List<User> users = userRepository.findByCriteria(Restrictions.or(Restrictions.eq("username", user.getUsername()), Restrictions.eq("email", user.getEmail())));
    if (!CollectionUtils.isEmpty(users)) {
        User userFound = users.get(0);
        List<String> fields = new ArrayList<String>();
        if (userFound.getUsername().equals(user.getUsername())) {
            fields.add("username");
        }
        if (userFound.getEmail().equals(user.getEmail())) {
            fields.add("email");
        }
        throw new UserExistingException(fields);
    }
}

现在要测试这些东西，我需要模拟“userRepository.findByCriteria ..”，然后我尝试了以下 junit 测试

@Test(expected = UserExistingException.class)
public void registerExistingUserTest() throws DataAccessException, UserExistingException {
    User user = new User();
    user.setUsername("gfalco77");
    user.setEmail("Giuseppe.falco@gmail.com");
    List<User> users = new ArrayList<User>();
    users.add(user);


    Mockito.when(userRepository.findByCriteria(Restrictions.or(Restrictions.eq("username", user.getUsername()), Restrictions.eq("email", user.getEmail())))).thenReturn(users);
    userService.registerUser(user, UserRoles.ROLE_USER);
    Mockito.verify(userRepository).makePersistent(user);
}

但似乎“用户”列表总是空的。在上一篇文章中，我读到限制不是同一个对象，也许我必须使用匹配器。但是如何？如果我使用匹配器..它仍然有效创建一个不同的测试吗？

4

1 回答 1

3

这不是您问题的直接答案，但您的测试很难编写，因为您的服务类将业务代码与数据访问代码混合在一起。findByCriteria调用应该被封装到更高级别的方法UserRepository中

List<User> findByNameOrEmail(String name, String email)

您只需在测试中模拟这个简单的方法，并进行实际的存储库单元测试，测试findByNameOrEmail在测试数据库上是否按预期工作。

如果您 UserRepository 仅公开诸如findByCriteriaor之类的通用方法findByQuery，那么它不再是真正的存储库，因为创建条件或查询是调用者的责任，而不是存储库的责任。这样做并没有比直接在业务服务中使用 Hibernate 会话增加多少。

于 2011-12-12T23:01:13.397 回答