회원 서비스 개발 / 회원 서비스 테스트

* 인프런 김영한님의 스프링 입문 - 코드로 배우는 스프링 부트, 웹 MVC, DB 접근 기술 강의를 듣고 작성한 내용입니다.

  (강의 링크)

 

회원 서비스 개발

• 회원 리포지토리와 도메인 활용해서 실제 비즈니스 로직 작성

join

• 같은 이름이 있는 중복 회원 X

Optional<Member> result = memberRepository.findByName(member.getName());
result.ifPresent(m -> { //ifPresent Null이 아닌 어떤 값이 있으면 동작
    throw new IllegalStateException("이미 존재하는 회원입니다.");
});

findByName 후 결과는 Optional member이므로 바로 .ifPresent 가능

memberRepository.findByName(member.getName())
        .ifPresent(m -> { //ifPresent Null이 아닌 어떤 값이 있으면 동작
             throw new IllegalStateException("이미 존재하는 회원입니다.");
        });

이 부분이 특정한 기능 하는 메서드이므로 추출하여 따로 만들어주는 것이 좋음

* Ctrl + Alt + M 메서드 추출 단축키

public Long join(Member member){
    validateDuplicateMember(member); // 중복 회원 검증
    memberRepository.save(member);
    return member.getId();
}

private void validateDuplicateMember(Member member) {
    memberRepository.findByName(member.getName())
            .ifPresent(m -> { //ifPresent Null이 아닌 어떤 값이 있으면 동작
                 throw new IllegalStateException("이미 존재하는 회원입니다.");
            });
}

join에서는 중복 회원 검증 후 바로 저장하는 것이라고 쉽게 이해할 수 있음

MemberRepository의 경우 단순히 저장하고 꺼내오는 기능이 반복된다면 MemberService의 경우는 비즈니스에 가깝다

비즈니스와 매칭되는 이름을 써야 한다. → 비즈니스 의존적

MemberService.java

package hello.hellospring.service;

import hello.hellospring.domain.Member;
import hello.hellospring.repository.MemberRepository;
import hello.hellospring.repository.MemoryMemberRepository;

import java.util.List;
import java.util.Optional;

public class MemberService {
    private final MemberRepository memberRepository = new MemoryMemberRepository();

    /**
     *회원 가입
     */
    public Long join(Member member){
        validateDuplicateMember(member); // 중복 회원 검증
        memberRepository.save(member);
        return member.getId();
    }

    private void validateDuplicateMember(Member member) {
        memberRepository.findByName(member.getName())
                .ifPresent(m -> { //ifPresent Null이 아닌 어떤 값이 있으면 동작
                     throw new IllegalStateException("이미 존재하는 회원입니다.");
                });
    }

    /**
     * 전체 회원 조회
     */
    public List<Member> findMembers(){
        return memberRepository.findAll();
    }

    public Optional<Member> findOne(Long memberId){
        return memberRepository.findById(memberId);
    }
}

---

회원 서비스 테스트

편하게 테스트 만들기 → 테스트 하고 싶은 class에서 Ctrl + Shit + T → 자동으로 틀을 만들어줌

테스트의 경우 한글로 이름 만들어도 됨

 

• 테스트 추천하는 패턴 given when then 주석 깔기
• 뭔가가 주어져서(given)  이걸 실행했을 때(when) 결과가 이렇게 나와야 함(then) 

 

테스트는 정상 플로우보다 예외 처리가 더 중요

join의 핵심은 저장이 되는 것보다 중복 회원 검증이 잘 되는지!

 

try~catch 사용해서 중복_회원_예외 작성

public void 중복_회원_예외(){
    //given
    Member member1 = new Member();
    member1.setName("spring");

    Member member2 = new Member();
    member2.setName("spring"); // 이름이 똑같은 Member를 2개 만듦

    //when
    memberService.join(member1); // 첫 번째 join에서는 문제가 없음
    try{
        memberService.join(member2); // 두 번째 join에서는 중복이므로 문제 발생
        fail();
    } catch (IllegalStateException e){
        assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다.");
        // 이 부분은 예외가 터져서 정상적으로 성공한 것
    }

    //then
}

private void validateDuplicateMember(Member member) {
    memberRepository.findByName(member.getName())
            .ifPresent(m -> { //ifPresent Null이 아닌 어떤 값이 있으면 동작
                 throw new IllegalStateException("이미 존재하는 회원입니다.");
            });
}

try{
    memberService.join(member2); // 두 번째 join에서는 중복이므로 문제 발생
    fail();
} catch (IllegalStateException e){
    assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다.");
    // 이 부분은 예외가 터져서 정상적으로 성공한 것
}

MemberService의 validateDuplicateMember에서 중복 회원일 경우 "이미 존재하는 회원입니다." 메시지 출력하므로 테스트에서는 중복으로 인한 예외처리시 "이미 존재하는 회원입니다."라는 메시지 받으면 예외처리 성공한 것

try~catch 대신 assertThrows 사용하여 예외처리 작성

IllegalStateException 대신 NullPointerException 으로 바꾸면 오류 발생

clear 하기 위해서 MemoryMemberRepository 가져오고 @AfterEach 사용해서 메서드 끝나고 매번 clear

 

문제

MemberService에서 사용한 new MemoryMemberRepository와 테스트케이스에서 만든 MemoryMemberRepository가 다른 인스턴스 → 같은 리퍼지토리를 사용해서 테스트 진행하여야 함

 

해결 방법

public class MemberService {
    private final MemberRepository memberRepository;

    public MemberService(MemberRepository memberRepository) {
        this.memberRepository = memberRepository;
    } // MemberRepository를 외부에서 넣어주도록 수정

class MemberServiceTest {

    MemberService memberService;
    MemoryMemberRepository memberRepository;

    @BeforeEach // 각 테스트 실행 전 MemoryMemberRepository 만들고 MemberService에 넣는다
                // 같은 MemoryMemberRepository를 사용하게 된다 → Dependency Injection
    public void beforeEach(){
        memberRepository = new MemoryMemberRepository();
        memberService = new MemberService(memberRepository);
    }

@BeforeEach 사용해서 각 테스트 실행 전 MemoryMemberRepository 만들고 MemberService에 넣는다.

MemberServiceTest.java(~ 중복_회원_예외)

package hello.hellospring.service;

import hello.hellospring.domain.Member;
import hello.hellospring.repository.MemoryMemberRepository;
import org.assertj.core.api.Assertions;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;

import java.util.Optional;

import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
import static org.junit.jupiter.api.Assertions.*;

class MemberServiceTest {

    MemberService memberService;
    MemoryMemberRepository memberRepository;

    @BeforeEach // 각 테스트 실행 전 MemoryMemberRepository 만들고 MemberService에 넣는다
                // 같은 MemoryMemberRepository를 사용하게 된다 → Dependency Injection
    public void beforeEach(){
        memberRepository = new MemoryMemberRepository();
        memberService = new MemberService(memberRepository);
    }

    @AfterEach // 각 메서드 실행 끝날 때마다 호출 메모리 clear
    public void afterEach(){
        memberRepository.clearStore();
    }

    @Test
    void 회원가입() {
        //given
        Member member = new Member();
        member.setName("spring");

        //when
        Long saveId = memberService.join(member);

        //then 우리가 저장한 것이 리포지토리에 있는 게 맞아를 찾아보고 싶음
        Member findMember = memberService.findOne(saveId).get();
        assertThat(member.getName()).isEqualTo(findMember.getName());
    }

    @Test
    public void 중복_회원_예외(){
        //given
        Member member1 = new Member();
        member1.setName("spring");

        Member member2 = new Member();
        member2.setName("spring"); // 이름이 똑같은 Member를 2개 만듦

        //when
        memberService.join(member1); // 첫 번째 join에서는 문제가 없음
        IllegalStateException e = assertThrows(IllegalStateException.class, () -> memberService.join(member2));
        // 이 로직 실행 시 이 예외가 터져야 함
        assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다.");
/*
        try{
            memberService.join(member2); // 두 번째 join에서는 중복이므로 문제 발생
            fail();
        } catch (IllegalStateException e){
            assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다.");
            // 이 부분은 예외가 터져서 정상적으로 성공한 것
        }
*/
        //then
    }

    @Test
    void findMembers() {
    }

    @Test
    void findOne() {
    }
}

* 단축키 

Ctrl + Shift + F10: 이전 실행 내용 그대로 실행

Ctrl + Alt + V: 자동으로 리턴

Ctrl + Alt + M : 메서드 추출

Ctrl + Shit + T: 편하게 테스트 만들기 → 테스트 하고 싶은 class에서 단축키 누름

Alt + insert: constructor

 

try~catch 부분 공부해서 추가