使用模拟的测试错误：想要但未调用：

Question

我尝试为我的弹簧启动控制器编写集成测试。这是我要测试的控制器方法：

    @GetMapping("/")
    public String getAllBooks(Model model, @RequestParam("page") Optional<Integer> page, @RequestParam("size") Optional<Integer> size) {
        int currentPage = page.orElse(FIRST_PAGE);
        int pageSize = size.orElse(DEFAULT_PAGE_SIZE);

        Page<Book> bookPage = bookService.getAllBooks(PageRequest.of(currentPage - 1, pageSize));
        Set<String> categories = bookService.findAllCategories();

        model.addAttribute("categories", categories);
        model.addAttribute("currentPage", currentPage);
        model.addAttribute("books", bookPage);

        int totalPages = bookPage.getTotalPages();
        if (totalPages > 0) {
            List<Integer> pageNumbers = IntStream.rangeClosed(1, totalPages)
                    .boxed()
                    .collect(Collectors.toList());
            model.addAttribute("pageNumbers", pageNumbers);
        }

        return "library";
    }

这是我的课程，带有测试：

@SpringBootTest
@AutoConfigureMockMvc
public class BookControllerTests {

    @Autowired
    private MockMvc mockMvc;
    @Mock
    BookService bookService;
    @Mock
    BookRepository bookRepository;

    private final String category = "forTest";

    final Book book = new Book("test1", 9999, "testAuthor1", 1, "test1");

    @BeforeEach
    public void initEach() {
        bookService.addNewBook(book, category, "not used");
    }

    @AfterEach
    public void deleteEach() {
        bookRepository.deleteAll();
    }

    @Test
    void getAllBooksTest() throws Exception {
        mockMvc.perform(get("/")
                        .param("page", "1")
                        .param("size", "20"))
                .andExpect(status().isOk());

        ArgumentCaptor<Pageable> pageableCaptor = ArgumentCaptor.forClass(Pageable.class);

        verify(bookRepository).findAll(pageableCaptor.capture());
        PageRequest pageable = (PageRequest) pageableCaptor.getValue();

        assertThat(pageable).hasPageNumber(1);
        assertThat(pageable).hasPageSize(20);
    }
}

我编写了自定义essertthat方法，它看起来像：

class PageableAssert extends AbstractAssert<PageableAssert, Pageable> {

    PageableAssert(Pageable pageable) {
        super(pageable, PageableAssert.class);
    }

    static PageableAssert assertThat(Pageable actual) {
        return new PageableAssert(actual);
    }

    void hasPageSize(int expectedPageSize) {
        if (!Objects.equals(actual.getPageSize(), expectedPageSize)) {
            failWithMessage("expected page size to be <%s> but was <%s>", expectedPageSize, actual.getPageSize());
        }
    }

    void hasPageNumber(int expectedPageNumber) {
        if (!Objects.equals(actual.getPageNumber(), expectedPageNumber)) {
            failWithMessage("expected page number to be <%s> but was <%s>", expectedPageNumber, actual.getPageNumber());
        }
    }

    PageableAssert hasSort(String field, Sort.Direction direction) {

        Sort.Order actualOrder = actual.getSort().getOrderFor(field);

        if (actualOrder == null) {
            failWithMessage("expected sort for field <%s> to be <%s> but was null", field, direction);
        } else if (actualOrder.getDirection() != direction) {
            failWithMessage("expected sort for field <%s> to be <%s> but was <%s>", field, direction, actualOrder.getDirection());
        }

        return this;
    }
}

但是，正如我正确理解我的问题时，我出现在这一行中： 验证（bookrepository）.findall（pageablecaptor.capture（））; 因此，我的自定义断言不会影响这种错误的发生，看起来像： 想要但不被调用： bookrepository.findall（）; - ＆gt;在kpi.diploma.ovcharenko.controller.bookcontrollertests.getallbookstest（bookcontrollertests.java:58） 实际上，与此模拟的相互作用为零。 您能帮我解决这个问题吗，因为我完全不知道如何解决它。

Answer 1

在您的测试中，您正在验证bookrepository.findall被调用，但是在您正在测试的方法中，BookRepository甚至没有引用。

您要验证的是bookservice.getallbooks和bookservice.findallcategories正在调用。

如果您想验证是否正确调用您的BookRepository Inconcations，请围绕BookService方法编写测试，因为我认为这是进行这些调用的地方。

另外，我应该补充一点，由于您正在测试的方法使用这些模拟的响应，因此您需要创建一个wher（） - ＆gt; then（）语句以确保这些调用返回一些数据，否则您将获得一个NULL指针异常到达此行时：int totalpages = bookpage.getTotalPages（）;

类似，

Page<Book> bookPage =  Mockito.mock(Page.class);
when(bookService.getAllBooks(any()).thenReturn(bookPage);