[JPA Self Study] AccountController Refactoring

2023-08-25
spring-self-study

SpringBootJpa 개발 | 5 DAY - 1

하다가 막히는 부분이나 알아야 할 사항, 혹은 버전의 문제 등을 적기 위한 간단한 노트이며

이는 공부용 소스이기 때문에 소스에 주석을 달아가며 설명을 적어뒀습니다.
따라서 해당 날짜의 소스를 보며 노트를 참고하는 식으로 봐야합니다.

소스는 깃헙에 올려놓았습니다.

1. AccountEmailAthenticate

- About Repository

repository를 도메인 계층으로 보느냐 ayer계층(controller,service,dao..)으로 보느냐에 따라 repository를 controller에서 쓰는 것에 대한 논의가 많은데 여기선 도메인게층으로 보고 controller에서 사용하겠습니다.

- Token is NPE ( Transaction Issue )

Email을 통해 Token을 인증하는 과정에서 DB에 저장된 내 정보에 Token이 없어서 문제가 생겼습니다.

문제가 생긴 로직입니다.

1. processNewAccount

public void processNewAccount(SignUpForm signUpForm) {
  Account newAccount = saveNewAccount(signUpForm);
  newAccount.generateEmailCheckToken();
  sendSignUpConfirmEmail(newAccount);
  }

2. saveNewAccount

private Account saveNewAccount(SignUpForm signUpForm) {
  Account account = Account.builder()
  .email(signUpForm.getEmail())
  .nickname(signUpForm.getNickname())
  .password(passwordEncoder.encode(signUpForm.getPassword()))
  .studyCreatedByWeb(true)
  .studyEnrollmentResultByWeb(true)
  .studyUpdatedByWeb(true)
  .build();

  return accountRepository.save(account);
  }

3. sendSignUpConfirmEmail

private void sendSignUpConfirmEmail(Account newAccount) {
  SimpleMailMessage mailMessage = new SimpleMailMessage();
  mailMessage.setTo(newAccount.getEmail());
  mailMessage.setSubject("스터디올래, 회원 가입 인증");
  mailMessage.setText("/check-email-token?token="+ newAccount.getEmailCheckToken()+"&email="+ newAccount.getEmail());
  javaMailSender.send(mailMessage);
  }

saveNewAccount(signUpForm)으로 저장 후
newAccount.generateEmailCheckToken(); 으로 토큰을 생성
sendSignUpConfirmEmail(newAccount); 회원가입 인증 메일 전송

결론 : 계정 저장하고 나서 토큰을 생성해서 저장된 계정엔 토큰이 없음

이에 대해 강의에서는 ” Detached Object라 DB에 Synch되지 않았다. “ 라고 말했습니다.

( 멋지고.. 하나도 못 알아들었다.. 🤗 Jpa 건너뛰고 들었더니.. Jpa에 대해 기본적으로 공부할 것이 있는것 같네요.. 😅 )

1번 processNewAccount에서 토큰을 생성했음에도 토큰이 저장이 되지 않는 이유는 1번에서의 newAccount 객체는 Detached Object라서 DB에 synch가 되지 않은 겁니다. 생성했음에도 null인 이유는 2번 saveNewAccount에서 이미 트랜잭션 처리를 했기 때문에 이 안에서 해당하는 entity가 JPA entity life cycle에 해당하는 Persistent 상태입니다. 2번을 나온 1번에선 Detached 상태입니다.

1번에서도 Persistent 상태를 유지하려면 @Transactional 을 붙이면 됩니다.

@Transactional
public void processNewAccount(SignUpForm signUpForm) {
    Account newAccount = saveNewAccount(signUpForm);
    newAccount.generateEmailCheckToken();
    sendSignUpConfirmEmail(newAccount);
}

JPA의 영속성에 대해 참고한 블로그

이런 문제를 조기에 발견해서 다행이지만.. TestCode에서 미처 확인하지 못했습니다.

TestCode 보완해서 다시 테스트했습니다.

@DisplayName("회원 가입 처리 - 입력값 정상")
@Test
void signUpSubmit_with_correct_input() throws Exception {

  mockMvc.perform(post("/sign-up")
          .param("nickname","oomi")
          .param("email","whdudal1217@naver.com")
          .param("password","1234578910")
          .with(csrf()))
      .andExpect(status().is3xxRedirection())
      .andExpect(view().name("redirect:/"));

  Account account = accountRepository.findByEmail("whdudal1217@naver.com");

  assertThat(account).isNotNull();
  assertThat(account.getPassword()).isNotEqualTo("1234578910");
  assertThat(account.getEmailCheckToken()).isNotNull();
  then(javaMailSender).should().send(any(SimpleMailMessage.class));
}

Git Revision Number : 09933326161c696067b1040bdba985e11f472246

2. Account Email Athenticate Test and Refactoring

Git Revision Number : 7cfd0e4fc4293e7836a3aa29cc32e0b312c230ca

Twitter Facebook LinkedIn