Сравнение объектов в тестах

[funnyDevGirl]

Теперь для сравнения объектов в тестах не требуется работать напрямую с json, а сравнивать java-объекты.

Было:

        var request = get("/api/posts/" + testPost.getId()).with(jwt());
        var result = mockMvc.perform(request)
                .andExpect(status().isOk())
                .andReturn();
        var body = result.getResponse().getContentAsString();
        assertThatJson(body).and(
                v -> v.node("slug").isEqualTo(testPost.getSlug()),
                v -> v.node("name").isEqualTo(testPost.getName()),
                v -> v.node("body").isEqualTo(testPost.getBody()));

Стало:

        String body = response.getContentAsString();

        PostDTO postDTO = om.readValue(body, PostDTO.class);
        PostDTO testPostDTO = postMapper.map(testPost);

        assertThat(postDTO.getName()).isEqualTo(testPostDTO.getName());
        assertThat(postDTO.getSlug()).isEqualTo(testPostDTO.getSlug());
        assertThat(postDTO.getBody()).isEqualTo(testPostDTO.getBody());

[funnyDevGirl]

Ранее при проверке получения списка сущностей из контроллера проверялся только тип данных - список.

    @Test
    public void testIndex() throws Exception {
        postRepository.save(testPost);
        var result = mockMvc.perform(get("/api/posts").with(token))
                .andExpect(status().isOk())
                .andReturn();
        var body = result.getResponse().getContentAsString();
        assertThatJson(body).isArray();
    }

Теперь проверяется не только тип данных, но и содержимое этого списка:

    @Test
    public void testIndex() throws Exception {
        var response = mockMvc.perform(get("/api/posts").with(token))
                .andExpect(status().isOk())
                .andReturn()
                .getResponse();

        String body = response.getContentAsString();

        List<PostDTO> postDTOS = om.readValue(body, new TypeReference<>() {});

        List<Post> actual = postDTOS.stream().map(postMapper::map).toList();
        List<Post> expected = postRepository.findAll();

        Assertions.assertThat(actual).containsAll(expected);
    }

[Malcom1986]

Привет! Мы тестируем апи и сейчас напрямую проверяем этот апи. То есть мы говорим, что должны получить в ответе JSON, что у него должны быть такие-то ключи с такими-то значениями.
При использовании же маппера, если я накосячу в маппере, такой тест будет проходить. Так как и фактическое и эталонное значение будут преобразованы одним и тем же маппером с косяком. Например, в ответе не будет приходить слаг поста и такой тест все равно пройдет. А сейчас, напрямую проверяя полученный json, мы получим читаемый вывод, что такой ноды slug не существует

[s-chepurnov]

Все тесты возвращены в исходное состояние, но тесты testIndex() содержат преобразование json в java объекты.
В тесте testIndex() тестируется метод получения всех объектов, это достигается за счёт сравнения двух списков - один получается из БД, а второй из контроллера.
Для сравнения списков объектов в классах доменных моделей необходимо реализовать методы equals() и hashCode().
Студенты забывают про эти методы и у них не работает сравнение, это заставляет их вспомнить что необходимо реализовывать equals() и hashCode().
Это хороший пример со сравнением двух списков объектов, т.к. подавляющее большинство студентов к концу курса так и не понимают зачем необходимо переопределять эти методы.

[s-chepurnov]

Актуальная версия тестов testIndex() в этом Pull Request:

    @Test
    public void testIndex() throws Exception {
        var response = mockMvc.perform(get("/api/posts").with(token))
                .andExpect(status().isOk())
                .andReturn()
                .getResponse();

        String body = response.getContentAsString();

        List<PostDTO> postDTOS = om.readValue(body, new TypeReference<>() {});

        List<Post> actual = postDTOS.stream().map(postMapper::map).toList();
        List<Post> expected = postRepository.findAll();

        Assertions.assertThat(actual).containsAll(expected);
    }

[Malcom1986]

Давайте тут матчер поменяем на специальный для итерируемых объектов (списки, объекты) https://assertj.github.io/doc/#assertj-core-group-contains

[Malcom1986]

и тут

[funnyDevGirl]

Malcom1986 Максим, добрый день! Скорректировала во всех тестовых классах. Прошу посмотреть.
Спасибо.

[Malcom1986]

Мержим, спасибо