我的方法的JUnit测试

Question

大家好，我需要为我的方法编写单元测试。我有点麻烦，因为我是JUnit的新手。我需要为这个方法写一个测试。这是我的方法

@Override
public Long countSellingOrdersInQueue(String principal) {
    List<String> states = Arrays.asList(PENDING.name(), REGULARIZED.name());
    return orderRepository.countByArticleUserIdAndStateIn(principal, states);
}

我试过了，但我被挡了，这是我的结果

附言：测试通过了，但我不明白我的测试是不是真的

@MockBean
private OrderRepository orderRepository;

private String principal ;

@Test
public void countSellingOrdersInQueueTest(){
    orderService.countSellingOrdersInQueue(principal);
    List<String> states = Arrays.asList(PENDING.name(), REGULARIZED.name());
    orderRepository.countByUserIdAndStateIn(principal,states);
}

Answer 1

测试通过，因为您没有任何用于检查结果的断言。你只需要调用那些毫无例外地执行的方法。

简单的测试示例：

    @Test
    public void test() {
        assertEquals(true, true);
    }

在你的情况下，测试将是这样的：

    @Test
    public void countSellingOrdersInQueueTest(){
        orderService.countSellingOrdersInQueue(principal);
        List<String> states = Arrays.asList(PENDING.name(), REGULARIZED.name());
        orderRepository.countByUserIdAndStateIn(principal,states);
        assertEquals(10, orderRepository.countByUserIdAndStateIn(principal,states));//10 replace to expectetion count
        //add some more checks
    }

Answer 2

在您的情况下，这只是单元测试，您不需要使用@MockBean，因为它加载了上下文。单元测试的目的是更快地运行，使用@MockBean，将加载上下文并花费时间来完成测试。下面是关于何时使用@Mock和何时使用@MockBean的建议。

正如马克西姆所说，测试中没有断言。这就是测试没有失败的原因。

在编写测试时，需要记住的事情很少。

• 测试被认为是代码的文档，它应该更具可读性，这样可以让其他人理解代码
• 如前所述，单元测试是为了提供更快的反馈
• 在测试中使用AAA（安排、行动、断言）结构。更多信息请点击这里

下面是代码：

public class OrderServiceTest {

    @InjectMocks
    private OrderService orderService;

    @Mock
    private OrderRepository orderRepository;

    @Before
    public void setUp() throws Exception {
        initMocks(this);
    }

    @Test
    public void countSellingOrdersInQueueTest(){
        when(orderRepository.countByArticleUserIdAndStateIn(any(), any())).thenReturn(1L);
        String principal = "dummyString";

        Long actualCount = orderService.countSellingOrdersInQueue(principal);

        List<String> expectedStates = Arrays.asList("State 1", "State 2");
        assertThat(actualCount, is(1L));
        verify(orderRepository).countByArticleUserIdAndStateIn(principal, expectedStates);
    }
}