在 java 测试中没有收到错误

Question

我的测试不应该通过。我应该在第 2 行收到一个错误，因为我在第 1 行输入了错误的值new Long(0)。 请告诉我我的错误在哪里。 谢谢。

    @Test
    public void getPersonListByOtdelIdTest() {            
        Long otdelId = new Long(454);
        ArgumentCaptor<Long> argumentOtdelId = ArgumentCaptor.forClass(Long.class);
        SessionFactory mockedSessionFactory = mock(SessionFactory.class);
        Session session = mock(Session.class);
        Query query = mock(Query.class);
        PersonDAOImpl personDAO = new PersonDAOImpl(mockedSessionFactory);
        when(mockedSessionFactory.getCurrentSession()). thenReturn(session);
        when(session.createQuery("FROM Person P where P.otdel.otdelId = :otdelId")).thenReturn(query);
 #1-->  when(query.setParameter("otdelId", new Long(0))).thenReturn(query);
        when(query.list()).thenReturn(persons);

        List<Person> expectedPersons =  personDAO.getPersonListByOtdelId(otdelId);

        verify(mockedSessionFactory).getCurrentSession();
        verify(session).createQuery("FROM Person P where P.otdel.otdelId = :otdelId");
        verify(query).setParameter(eq("otdelId"), argumentOtdelId.capture());
        verify(query).list();
 #2-->  assertEquals(otdelId, argumentOtdelId.getValue());
        assertTrue(expectedPersons.containsAll(persons));
    }

 public class PersonDAOImpl implements PersonDAO {
    public List<Person> getPersonListByOtdelId(Long otdelId) {
        Query query = sessionFactory.getCurrentSession().createQuery("FROM Person P where P.otdel.otdelId = :otdelId");
        query.setParameter("otdelId", otdelId);
        List<Person> listPersons = query.list();
        return listPersons;
    } 
 }

Answer 1

我不明白为什么您会期望 #2 的断言失败。您的测试通过454（作为Long）进入getPersonListByOtdelId()，因此将传递给query.setParameter()。 #1 中的when() 代码本质上是无操作的，因为没有使用这些值调用query.setParameter()，但是如果when() 中指定的调用从未发生过，Mockito 不会抱怨，并且下面的代码test 不检查返回值，所以也不例外。

在任何情况下，您都不需要ArgumentCaptor；您只需要让 Mockito 验证是否将正确的值传递给 setParameter()

事实上，很多验证调用是不需要的，你可以这样做：

@Test
public void getPersonListByOtdelIdTest() {
  Long otdelId = 454L; // or = Long.valueOf(454); don't use new Long
  SessionFactory mockedSessionFactory = mock(SessionFactory.class);
  Session session = mock(Session.class);
  Query query = mock(Query.class);

  when(mockedSessionFactory.getCurrentSession()).thenReturn(session);
  when(session.createQuery("FROM Person P where P.otdel.otdelId = :otdelId"))
      .thenReturn(query);
  when(query.setParameter("otdelId", otdelId)).thenReturn(query);
  when(query.list()).thenReturn(persons);

  PersonDAOImpl personDAO = new PersonDAOImpl(mockedSessionFactory);
  List<Person> result = personDAO.getPersonListByOtdelId(otdelId);

  verify(query).setParameter("otdelId", otdelId);
  assertEquals(result, persons);
}

您无需验证是否调用了getCurrentSession()，因为如果不是，被测代码将无法获得会话。您无需验证是否将正确的查询传递给createQuery()，因为被测代码使用了不同的查询，Mockito 不会返回模拟查询（当然，除非您使用RETURNS_MOCKS）。

话虽如此，我认为上述测试不是一个好的测试。测试几乎完全反映了代码，它并没有验证代码是否可以工作。换句话说，它是change detector test。

我不会使用模拟框架来测试PersonDaoImpl。相反，我会编写一个启动内存数据库的测试，使用的模式文件也用于在生产中创建实际表。

我会使用模拟来测试依赖于PersonDAO 的类。