refactoring - email service #84

zhyunk · 2024-05-07T09:56:36Z

Email Service에 있는 이메일 전송 메서드는 business로 가는게 맞을 것 같다는 생각이 들었다

고쳐야지 🔨

- 실제 발송 테스트시 만료 시간을 test/application.yml에서 가져오는 것으로 변경 - 실제 발송 테스트 파일 위치를 옮기고 리팩토링 내용에 기반하여 코드 수정 - EmailServiceTest에서 메일 발송 검증 코드를 삭제 issue key : #81, #84

1. JavaMailSender 사용 코드를 EmailUtil로 분리 2. EmailService에 위치했던 메일 전송 코드를 SignUpBusiness로 이동 3. Email 전송 메서드 수행 중 발생하는 에러 메세지에 원본 에러 메세지를 더함 issue key : #81, #84

- 실제 발송 테스트시 만료 시간을 test/application.yml에서 가져오는 것으로 변경 - 실제 발송 테스트 파일 위치를 옮기고 리팩토링 내용에 기반하여 코드 수정 - EmailServiceTest에서 메일 발송 검증 코드를 삭제 issue key : #81, #84

issue key : #81, #84

assertTrue(StringUtils.isNotBlank(authCode)); 👆 이 코드로 한번에 검사하는 것 보다 두 가지 케이스로 검사한 후 실패한 케이스를 알 수 있도록 하는 게 더 좋은 것 같아서 고침 issue key : #81, #84

zhyunk changed the title ~~refactoring~~ refactoring - email service May 7, 2024

zhyunk self-assigned this May 7, 2024

zhyunk added a commit that referenced this issue May 7, 2024

test: [user] sign up business - 이메일 전송 코드 검증

ada8b6f

issue key : #81, #84

zhyunk added a commit that referenced this issue May 7, 2024

test: [user] sign up business test - 이메일 전송 코드 검증

16144ff

issue key : #81, #84

zhyunk added a commit that referenced this issue May 7, 2024

test: [user] email util 테스트

b339ed1

issue key : #81, #84

zhyunk closed this as completed May 7, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactoring - email service #84

refactoring - email service #84

zhyunk commented May 7, 2024

refactoring - email service #84

refactoring - email service #84

Comments

zhyunk commented May 7, 2024