🛠️ 리팩토링 ⚒️ 1차 PR - 프로젝트 구조 수정, test 코드 다시 작성 #90

zhyunk · 2024-05-19T12:44:51Z

프로젝트 구조를 도메인 관점으로 다시 작성해보았다.
테스트 코드 또한 변경된 구조에 맞춰서 수정하는 김에 조금 더 공부해서 새롭게 작성!

1차 리팩토링은 여기서 마침

issue : #67, #68, #69, #73, #75, #78, #80, #81, #82

프로젝트 구조 1차 리팩토링 - jwt 모듈 issue key : #78

프로젝트 구조 1차 리팩토링 테스트 코드는 함께 수정하지 못했다 http 테스트는 진행 issue key : #78

프로젝트 구조 1차 리팩토링 - 프로젝트 구조를 리팩토링하면서 api 설계시 중복되었던 userId(request body, api path 존재)를 path에서 삭제하는 것으로 추가 리팩토링 테스트 코드 수정 해야 됨 http 테스트는 진행 issue key : #78

@getter

- ObjectMapper Config 설정을 common-jwt 모듈에서 정의 - Message 객체에서 불필요한 @getter 어노테이션 삭제 - 모듈 구조 2차 수정 issue key : #78

api 설계 과정에 오류가 있었다 api path에도 userId를 넣고 request body 에도 userId를 넣었던 부분인데 고민하다가 path의 userId를 삭제했다. (gateway에서 path parameter때문에 심란했었기 때문) 그런데 이게 맞는가에 대한 생각이 뒤늦게나마 들고있다 일단 수정했지만, 이 부분(userId와 같은 정보를 path에 넣을지, body에 넣을지)조금 더 시간을 가지고 조사해보는게 좋을 것 같다 🤔

`client`랑 `scheduler`의 위치를 domain/member 하위로 옮김 domain/member 에서만 사용하기 때문에 이 위치로 옮기는게 맞는 것 같아서 이동 하지만 계속 고민하게 됨 issue key : #78

- exception handler 위치를 user 모듈과 article 모듈에서 common-jwt 모듈로 이동 issue key : #80

몇 가지 키워드에 대해 전체 바꾸기를 진행하면서 http 파일에 잘못 반영 된 부분이 있었다. gateway 구현 후 gateway의 http 실행 파일만 사용하느라 미처 확인 하지 못했던 부분으로 뒤늦게 발견하여 고침 그냥 삭제할까도 싶었지만 나중에 gateway 설정을 변경하게 되었을 경우에 http 실행 결과 대조군으로 사용하기 위해 남겨둠

issue key : #81

Role -> RoleEntity issue key : #78

issue key : #81

issue key : #78

crud 테스트 issue key : #81

issue key : #78, #81

issue key : #81

메일 발송 테스트 issue key : #81

- 테스트 실패 케이스를 다루기 위해 사용했던 `@Nested` 를 삭제하고 `@ParameterizedTest` 를 사용 - email service test - SendEmail Nested 클래스의 @import(MailConfig.class) 삭제 - 이메일 인증코드 유효시간 10초로 줄임 - 이메일 인증코드 출력 태그 h3 -> h1로 변경 issue key : #81

테스트 코드에서 인증코드 출력 태그를 h1로 변경해서 실행 코드에도 반영 issue key : #81

잘못 작성한 부분이 있었는데 의도한 대로 동작해서 인지하지 못한 채로 커밋을 작성하고 말았다;; 고침 issue key : #81

issue key : #81

user repository test - static 삭제 email service test - 이메일 인증코드 유효시간 10초 -> 3초 조절 issue key : #81

앞선 검증 중 실패 케이스가 나타나도 나중에 위치한 검증이 수행되는 assertAll 적용 issue key : #81

1. 알아보기 힘들었기 때문에 주석을 붙여주었다..🥲 2. saveEmailAuthCode 메서드를 EmailService내부에서만 사용해서 private로 접근제한자를 변경 issue key : #81

- 실제 email 발송 테스트를 EmailSendTest.class 로 별도로 분리 - 이메일 발송 과정에 대한 mock 테스트와 이메일 인증 후 redis 유저 정보에 email 정보를 저장하는 로직에 대한 mock 테스트 추가 issue key : #81

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

- update() 메서드 사용 방법 변경 - getNewArticleId() 메서드에 작성된 중복 내용 삭제 issue key : #82

issue key : #82

issue key : #81

issue key : #82

게시글 제목 , 내용 입력 데이터 검증 issue key : #82

issue key : #81, #82

issue key : #81

issue key : #82, #88

embedded h2 db에 적합하기로는 update가 제일이라 생각되어 변경 spring.jpa.hibernate.ddl-auto=update issue key : #81, #82

article entity -> articles issue key : #82

- content validator를 통과하는 조건을 '공백이 아닌 문자가 1개 이상 있음'으로 할 경우 java의 여러줄 문자열(""")이 통과하지 못하는 문제가 생겼다; 그래서 우선 조건을 `null이 아님`으로 변경 - article entity 에서 content 컬럼의 데이터 타입을 text로 했었는데 h2에선 CLOB로 설정해주어야 했다. jpa에서 @lob 어노테이션을 지원하여 적용 issue key : #82

정규식을 사용하지 않고 java에서 제공하는 `!("").trim().isblink()` 조건을 사용하니 여러 줄 문자열도 필터링 되었다..!! issue key : #82

이번에 리팩토링하면서 api가 변경된 부분인데 미처 redirect url을 수정하지 못했다. 고침 issue key : #82

issue key : #82

@disabled

전체 테스트 실행을 위해 ArticleApiTest.java는 @disabled 부착 issue key : #82

umemberBusiness.updateUserInfo() 메서드의 반환값을 UserResponse에서 String 으로 변경 후 test를 수정해놓지 않았다; 고침 issue key : #81

@disabled

해당 테스트는 전체 테스트 시 실행되지 않도록 @disabled 부착 issue key : #81

issue key : #82

- content validator: 정규식 주석으로 추가 - controller test: 테스트 케이스 추가 issue key : #82

issue key : #81

1. member api test- login 실행 후 status 확인 추가 2. authentication filter test 추가 issue key : #81

issue key : #81

zhyunk added 30 commits April 28, 2024 23:34

refactor: [jwt] 1차 리팩토링

223117b

프로젝트 구조 1차 리팩토링 - jwt 모듈 issue key : #78

refactor: [user] 1차 리팩토링

7501455

프로젝트 구조 1차 리팩토링 테스트 코드는 함께 수정하지 못했다 http 테스트는 진행 issue key : #78

refactor: [jwt, user, article] 2차 리팩토링

00479e2

- ObjectMapper Config 설정을 common-jwt 모듈에서 정의 - Message 객체에서 불필요한 @getter 어노테이션 삭제 - 모듈 구조 2차 수정 issue key : #78

refactor: [user] 3차 리팩토링

d113aab

`client`랑 `scheduler`의 위치를 domain/member 하위로 옮김 domain/member 에서만 사용하기 때문에 이 위치로 옮기는게 맞는 것 같아서 이동 하지만 계속 고민하게 됨 issue key : #78

refactor: [common-jwt, user, article] exception handler 위치 이동

0c86eb8

- exception handler 위치를 user 모듈과 article 모듈에서 common-jwt 모듈로 이동 issue key : #80

test: [user] 먼저 작성했던 test 삭제

9b68624

issue key : #81

refactor: [user] role entity 이름 변경

462811f

Role -> RoleEntity issue key : #78

refactor: [user] user entity field nullable 설정 추가

6a92660

issue key : #81

fix: [user] UserConverter.java - role entity 이름 변경

defd9c0

issue key : #78

test: [user] user repository

8a74fe3

crud 테스트 issue key : #81

fix: [user] session user - session id 필드에 not null 설정 추가

89e8a2c

issue key : #78, #81

test: [user] redis test container 추가

df57255

issue key : #81

test: [user] session user repository

ffb0fec

issue key : #81

test: [user] email service

35d24a9

메일 발송 테스트 issue key : #81

refactor: [user] email service mail body tag 수정

fb47a93

테스트 코드에서 인증코드 출력 태그를 h1로 변경해서 실행 코드에도 반영 issue key : #81

test: [user] user repository test 고침

9ff1be6

잘못 작성한 부분이 있었는데 의도한 대로 동작해서 인지하지 못한 채로 커밋을 작성하고 말았다;; 고침 issue key : #81

test: [user] user repository test refactoring

5ff74a2

issue key : #81

test: [user] user repository test 고침, email service test 수정

333adea

user repository test - static 삭제 email service test - 이메일 인증코드 유효시간 10초 -> 3초 조절 issue key : #81

test: [user] repository test 수정

944470e

앞선 검증 중 실패 케이스가 나타나도 나중에 위치한 검증이 수행되는 assertAll 적용 issue key : #81

refactor: [user] email service - 주석 작성 , saveEmailAuthCode private 변경

51eceee

1. 알아보기 힘들었기 때문에 주석을 붙여주었다..🥲 2. saveEmailAuthCode 메서드를 EmailService내부에서만 사용해서 private로 접근제한자를 변경 issue key : #81

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

16144ff

issue key : #81, #84

test: [user] email util 테스트

b339ed1

issue key : #81, #84

test: [user] email util test 리팩토링

821f51b

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

zhyunk added 28 commits May 15, 2024 17:59

refactor: [article] article service에서 @value 필드 분리

439460a

- update() 메서드 사용 방법 변경 - getNewArticleId() 메서드에 작성된 중복 내용 삭제 issue key : #82

test: [article] article service test

9307c0f

issue key : #82

fix: [article] article business - update 예외조건 고침

e87d3ad

issue key : #81

test: [article] article business test

1b53417

issue key : #82

test: [article] test security config.java 추가

ee9a240

issue key : #82

test: [article] article controller test

379e224

issue key : #82

fix: [article] 게시글 제목, 내용 validator 고침

afedc39

issue key : #82

test: [article] article controller test - validation 테스트

9870207

게시글 제목 , 내용 입력 데이터 검증 issue key : #82

test: [user, article] application.yml - 통합 테스트를 위한 h2 console 설정 추가

ea81441

issue key : #81, #82

test: [user] MemberApiTest.java, SignUpApiTest.java 위치 이동

86b296a

issue key : #81

test: [user] api test - article test를 위한 계정 생성 정보 입력

ee14b0d

issue key : #82, #88

chore: [user, article] application.yml - ddl-auto 변경 (test , main)

b765409

embedded h2 db에 적합하기로는 update가 제일이라 생각되어 변경 spring.jpa.hibernate.ddl-auto=update issue key : #81, #82

fix: [article] article entity mapping name 고침

f5629b2

article entity -> articles issue key : #82

fix: [article] content validator

f28db28

정규식을 사용하지 않고 java에서 제공하는 `!("").trim().isblink()` 조건을 사용하니 여러 줄 문자열도 필터링 되었다..!! issue key : #82

fix: [article] article controller - 게시글 수정 후 redirectUrl 고침

a4b611a

이번에 리팩토링하면서 api가 변경된 부분인데 미처 redirect url을 수정하지 못했다. 고침 issue key : #82

test: [user] member api test - article test를 위한 탈퇴 처리 정보 저장용

2c3339c

issue key : #82

test: [article] api test

dda89fd

issue key : #82

test: [article] article test에서 안쓰는 코드 정리, article api test disabled

b950faa

전체 테스트 실행을 위해 ArticleApiTest.java는 @disabled 부착 issue key : #82

test: [user] member business test - updateUserInfo_success()

c83a47a

umemberBusiness.updateUserInfo() 메서드의 반환값을 UserResponse에서 String 으로 변경 후 test를 수정해놓지 않았다; 고침 issue key : #81

test: [user] 이메일 실제 전송 테스트, member api 테스트, sign up api 테스트 - disabled

95ba5b0

해당 테스트는 전체 테스트 시 실행되지 않도록 @disabled 부착 issue key : #81

test: [article] api test 위치 이동

a86ae50

issue key : #82

test: [article] content validator - controller test 수정

1874c2f

- content validator: 정규식 주석으로 추가 - controller test: 테스트 케이스 추가 issue key : #82

fix: [user] authentication filter 불필요한 코드 삭제

d658f73

issue key : #81

test: [user] authentication filter test, member api test

8ab8afd

1. member api test- login 실행 후 status 확인 추가 2. authentication filter test 추가 issue key : #81

test: [user] session check filter test

5ad7ef8

issue key : #81

test: [user] member api test - @disabled

aa530ae

issue key : #81

test: [user] session check filter test - 메서드 이름 변경

0433eeb

issue key : #81

zhyunk self-assigned this May 19, 2024

zhyunk merged commit 0967312 into main May 19, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🛠️ 리팩토링 ⚒️ 1차 PR - 프로젝트 구조 수정, test 코드 다시 작성 #90

🛠️ 리팩토링 ⚒️ 1차 PR - 프로젝트 구조 수정, test 코드 다시 작성 #90

zhyunk commented May 19, 2024

🛠️ 리팩토링 ⚒️ 1차 PR - 프로젝트 구조 수정, test 코드 다시 작성 #90

🛠️ 리팩토링 ⚒️ 1차 PR - 프로젝트 구조 수정, test 코드 다시 작성 #90

Conversation

zhyunk commented May 19, 2024