Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

모듈 내부 구조 변경 #78

Closed
7 tasks done
zhyunk opened this issue Apr 27, 2024 · 3 comments
Closed
7 tasks done

모듈 내부 구조 변경 #78

zhyunk opened this issue Apr 27, 2024 · 3 comments
Assignees
@zhyunk

Comments

@zhyunk
Copy link
Contributor

zhyunk commented Apr 27, 2024 •
edited
Loading

  • 모듈 내부 구조 변경
    • 1차: jwt, user, article
      • 패키지 구조를 계층 관점에서 도메인 관점으로 변경
    • 2차: jwt, user, article
      • ObjectMapper 설정 정의를 common-jwt 모듈로 이동
    • 3차: user
      • client, scheduler 패키지를 domain/member 하위로 이동
    • 4차: jwt, user, article
      • exception handler 관련 코드 common-jwt 모듈로 이동
  • 테스트 설계 다시 공부 #79
    • 테스트 설계
    • 단위 테스트
      • repository
      • service
      • business
      • controller
    • 통합 테스트
@zhyunk zhyunk self-assigned this Apr 27, 2024
zhyunk added a commit that referenced this issue Apr 28, 2024
@zhyunk
refactor: [jwt] 1차 리팩토링
223117b 
프로젝트 구조 1차 리팩토링 - jwt 모듈

issue key : #78
zhyunk added a commit that referenced this issue Apr 28, 2024
@zhyunk
refactor: [user] 1차 리팩토링
ab26e96 
프로젝트 구조 1차 리팩토링

테스트 코드는 함께 수정하지 못했다

https 테스트는 진행

issue key : #78
zhyunk added a commit that referenced this issue Apr 28, 2024
@zhyunk
refactor: [user] 1차 리팩토링
8935e80 
프로젝트 구조 1차 리팩토링

테스트 코드는 함께 수정하지 못했다

http 테스트는 진행

issue key : #78
zhyunk added a commit that referenced this issue Apr 28, 2024
@zhyunk
refactor: [user] 1차 리팩토링
7501455 
프로젝트 구조 1차 리팩토링

테스트 코드는 함께 수정하지 못했다

http 테스트는 진행

issue key : #78
zhyunk added a commit that referenced this issue Apr 28, 2024
@zhyunk
refactor: [article, jwt, gateway] 1차 리팩토링
ebdb745 
프로젝트 구조 1차 리팩토링

테스트 코드 수정 해야 됨

http 테스트는 진행

issue key : #78
zhyunk added a commit that referenced this issue Apr 28, 2024
@zhyunk
refactor: [article, jwt, gateway] 1차 리팩토링
66306d7 
프로젝트 구조 1차 리팩토링
- 프로젝트 구조를 리팩토링하면서 api 설계시 중복되었던 userId(request body, api path 존재)를 path에서 삭제하는 것으로 추가 리팩토링

테스트 코드 수정 해야 됨

http 테스트는 진행

issue key : #78
zhyunk added a commit that referenced this issue Apr 28, 2024
@zhyunk
refactor: [jwt, user, article] 2차 리팩토링
00479e2 
- ObjectMapper Config 설정을 common-jwt 모듈에서 정의
- Message 객체에서 불필요한 @getter 어노테이션 삭제
- 모듈 구조 2차 수정

issue key : #78
@zhyunk
Copy link
Contributor Author

zhyunk commented Apr 28, 2024 •
edited
Loading

2차 리팩토링 후 user 모듈에 client와 scheduler를 domain/member 하위로 옮기는게 나을까에 대한 고민

d113aab

zhyunk added a commit that referenced this issue Apr 28, 2024
@zhyunk
refactor: [user] 3차 리팩토링
d113aab 
`client`랑 `scheduler`의 위치를 domain/member 하위로 옮김
domain/member 에서만 사용하기 때문에 이 위치로 옮기는게 맞는 것 같아서 이동

하지만 계속 고민하게 됨

issue key : #78
@zhyunk
Copy link
Contributor Author

zhyunk commented Apr 29, 2024 •
edited
Loading

  • exception filter를 jwt 모듈로 옮기기
    -> 이게 먹힐까? 예전에 advice는 공통 모듈에서만 구현했을때 공통 모듈에 의존성을 둔 모듈들은 먹히지 않았었던 기억이 있는데, 그 때 잘못 구현했었을 수도 있으니 재시도
    　 -> 문득 든 생각이 advice도 같은 패키지만 검열하나 싶다 패키지 설정을 해줬어야했던것일까?!
    　　저녁에 퇴근하고 다양한 시도를 계획
    　 　 -> 놓치고 있었던 중요한 포인트 같아서 구글링해보니 basepackage 설정 옵션이 있었다
    　 　 　 https://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/web/bind/annotation/ControllerAdvice.html

0c86eb8

@zhyunk zhyunk mentioned this issue Apr 29, 2024
Closed
zhyunk added a commit that referenced this issue May 1, 2024
@zhyunk
refactor: [user] role entity 이름 변경
462811f 
Role -> RoleEntity

issue key : #78
zhyunk added a commit that referenced this issue May 1, 2024
@zhyunk
fix: [user] UserConverter.java - role entity 이름 변경
defd9c0 
issue key : #78
zhyunk added a commit that referenced this issue May 1, 2024
@zhyunk
fix: [user] session user - session id 필드에 not null 설정 추가
89e8a2c 
issue key : #78, #81
@zhyunk
Copy link
Contributor Author

zhyunk commented May 3, 2024 •
edited
Loading

test 공부 이슈로 마감 기한을 5월 4일에서 12일로 늘림

@zhyunk zhyunk closed this as completed May 18, 2024
@zhyunk zhyunk mentioned this issue May 19, 2024
Merged
zhyunk added a commit that referenced this issue May 19, 2024
@zhyunk
refactor: 프로젝트 구조 수정, test 코드 다시 작성 (#90)
0967312 
- 프로젝트 구조를 레이어 관점에서 도메인 관점으로 변경 
- 테스트 코드 또한 변경된 구조에 맞춰서 수정하는 김에 조금 더 공부해서 새롭게 작성

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

issue : #67, #68, #69, #73, #75, #78, #80, #81, #82
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zhyunk zhyunk

Labels
None yet
Projects
SB2 - Refactoring 🛠️
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@zhyunk