itemService 테스트 코드 질문입니다.

[질문 템플릿]
1. 강의 내용과 관련된 질문인가요? (예/아니오)
2. 인프런의 질문 게시판과 자주 하는 질문에 없는 내용인가요? (예/아니오)
3. 질문 잘하기 메뉴얼을 읽어보셨나요? (예/아니오)

[질문 내용]

ItemService 테스트 코드입니다.

@RunWith(SpringRunner.class)
@SpringBootTest
@Transactional
public class ItemServiceTest {

    @Autowired ItemRepository itemRepository;
    @Autowired ItemService itemService;

    @Test
    public void 상품저장() throws Exception {
        //given
        Item item = getItem("JPA book", 10000, 100);
        itemService.save(item);

        //when
        Item findItem = itemService.findById(item.getId());

        //then
//        assertThat(item).isEqualTo(findItem);
        assertEquals(item, findItem);
    }

    @Test
    public void 상품조회() throws Exception {

        //given
        Item item1 = getItem("JPA book1", 10000, 100);
        Item item2 = getItem("JPA book2", 20000, 100);
        itemService.save(item1);
        itemService.save(item2);

        //when
        List<Item> findAll = itemService.findAll();
        Item findItem1 = itemService.findById(item1.getId());

        //then
        assertThat(findAll.size()).isEqualTo(2); //전체조회
        assertThat(item1).isEqualTo(findItem1); //단건조회
    }

    @Test
    public void 상품변경() throws Exception {

        //given
        

        //when
        

        //then


    }


    private static Item getItem(String name, int price, int stockQuantity) {
        Item item = new Book();
        item.setName(name);
        item.setPrice(price);
        item.setStockQuantity(stockQuantity);
        return item;
    }
}

상품 저장과 조회에 대한 테스트 코드를 작성해보았는데 혹시 수정해야 할 부분이 있는지 궁금합니다.
상품 저장 시 item.getId() == null 이 아닐 때 merge 하도록 되어있는 부분에 대해서도 테스트를 해야할 것 같은데, 어떻게 테스트할지에 대해 방향성을 알려주시면 감사하겠습니다.

안녕하세요. jungbeen님, 공식 서포터즈 y2gcoder입니다.

대체로 잘 작성하셨다고 생각합니다!

몇가지 추가로 해보셨으면 하는 것들에 대해 말씀드리고 싶습니다 🙂 개인적인 의견일 뿐이니 참고만 해주십쇼!

단건 조회와 목록 조회의 테스트 코드는 나누는 게 어떨까요?
실패 케이스에 대한 테스트 코드가 없습니다! 특히 단건 조회와 목록 조회를 나눌 때, 단건 조회에서 상품을 찾지 못했을 때 발생하는 실패 케이스에 대한 테스트 코드가 있으면 좋을 것 같습니다
저장 후 비교할 때는 객체 자체를 비교하는 것도 좋으나 엔티티라면 id 값을 검증하거나, 저장할 때 넣었던 데이터 값들을 검증하는 쪽으로 테스트 코드를 짜보시는 게 어떨까요?

추가로 해당 테스트 코드는 서비스 단의 서비스 로직을 대상으로 하는 테스트 코드입니다. 말씀해주셨던 코드는 사실 서비스 단에서 테스트할 코드는 아니라고 생각합니다 🙂 상품 변경 에 대한 테스트 코드를 말씀하시는 거라면 저는 상품을 일단 저장하고 변경 필드 하나 당 테스트 코드를 만들어 이전 필드 값과 이후 필드값을 비교해보는 식으로 테스트 코드를 작성할 것 같습니다 !

감사합니다.

인프런 커뮤니티 질문&답변