테스트코드 오류

학습하는 분들께 도움이 되고, 더 좋은 답변을 드릴 수 있도록 질문전에 다음을 꼭 확인해주세요.

1. 강의 내용과 관련된 질문을 남겨주세요.
2. 인프런의 질문 게시판과 자주 하는 질문(링크)을 먼저 확인해주세요.
(자주 하는 질문 링크: https://bit.ly/3fX6ygx)
3. 질문 잘하기 메뉴얼(링크)을 먼저 읽어주세요.
(질문 잘하기 메뉴얼 링크: https://bit.ly/2UfeqCG)

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

[질문 내용]
여기에 질문 내용을 남겨주세요.

강의 듣기 전에 혼자서 update 테스트 코드를 짜봤는데 오류가 나더라고요. 어느 부분이 잘못된건지 알 수 있을까요?

void updateItem(){
    //given
    Item item1=new Item("itemA",10000,10);
    itemRepository.save(item1);
    //when
    Item item2=new Item("itemB",20000,12);
    itemRepository.update(item1.getId(),item2);
    //then
    Item updateItem=itemRepository.findById(item2.getId());
    assertThat(updateItem.getItemName()).isEqualTo("itemB");
    assertThat(updateItem.getPrice()).isEqualTo(20000);
    assertThat(updateItem.getQuantity()).isEqualTo(12);
}

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

별도의 구현 코드를 올리시지 않아 강의에서 작성한 코드에서 테스트 코드만 새로 작성하신것으로 이해하였습니다.

Item updateItem=itemRepository.findById(item2.getId());

코드를 보면 item2에서 데이터를 조회하고 있는데요, 실제 save()가 발생하여 id가 생성된 데이터는 item1뿐입니다. Item2는 변경하려는 정보만을 담고 있는(id는 Null) 데이터여서 위의 코드에서 NullPointerException이 발생할 것이고 아래와 같이 item1의 id로 변경하고 update 되었는지 조회하는 검증 코드를 사용하시면 될 것 같습니다.

@Test
void updateItem(){
    //given
    Item item1=new Item("itemA",10000,10);
    itemRepository.save(item1);
    //when
    Item item2=new Item("itemB",20000,12);
    itemRepository.update(item1.getId(),item2);
    //then
    Item updateItem=itemRepository.findById(item1.getId()); // 여기를 수정해주세요
    assertThat(updateItem.getItemName()).isEqualTo("itemB");
    assertThat(updateItem.getPrice()).isEqualTo(20000);
    assertThat(updateItem.getQuantity()).isEqualTo(12);
}

감사합니다.

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