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.

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

Refactor/#555 패키지 구조 변경 및 패키지간 양방향 제거 #556

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

hectick
Copy link
Collaborator

@hectick hectick commented Nov 23, 2023

🔥 연관 이슈

📝 작업 요약

content 패키지 내로 comment, post, cache, thumbs를 이동시켰으며
member 패키지 내로 profile(기존의 member), auth, setting을 이동시켰고
logging은 global 패키지 내로 이동되었습니다.
그리고 각 패키지에 대한 양방향 의존을 끊어내었습니다. (config가 성격에 맞는 패키지 내부로 이동됨)

추후 보완해야 할 것은 다음과 같습니다.

  • 각 패키지의 내부 구조 변경(Post와 PostImageInfo라던가, Member패키지와 Profile패키지안의 member 도메인 등.... 지금은 외부의 패키지를 내부로 그대로 집어넣은 터라 쓸데없이 depth가 깊거나 구조가 난잡할 수 있습니다.)
  • (후순위)각 패키지간의 객체 참조 끊어내기
  • 테스트도 모듈 따라 갈수 있도록 맞추기..(support, config 등등 지금은 한곳에 모아져있음..)

Copy link

Unit Test Results

  70 files  ±0    70 suites  ±0   25s ⏱️ -1s
369 tests ±0  367 ✔️ ±0  2 💤 ±0  0 ±0 
372 runs  ±0  370 ✔️ ±0  2 💤 ±0  0 ±0 

Results for commit 4d83aac. ± Comparison against base commit 446640f.

This pull request removes 275 and adds 275 tests. Note that renamed tests count towards both.
edonymyeon.backend.auth.AuthIntegrationTest ‑ 닉네임 중복을 체크한다 중복O
edonymyeon.backend.auth.AuthIntegrationTest ‑ 닉네임 중복을 체크한다 중복X
edonymyeon.backend.auth.AuthIntegrationTest ‑ 세션만료된 세션정보로 회원 정보 조회시 예외처리
edonymyeon.backend.auth.AuthIntegrationTest ‑ 실제 회원가입 요청시 닉네임 중복
edonymyeon.backend.auth.AuthIntegrationTest ‑ 실제 회원가입 요청시 이메일 중복
edonymyeon.backend.auth.AuthIntegrationTest ‑ 이메일 중복을 체크한다 중복O
edonymyeon.backend.auth.AuthIntegrationTest ‑ 이메일 중복을 체크한다 중복X
edonymyeon.backend.auth.AuthIntegrationTest ‑ 카카오 로그인
edonymyeon.backend.auth.AuthIntegrationTest ‑ 카카오 처음 로그인시 회원가입 (MemberRepository)
edonymyeon.backend.auth.AuthIntegrationTest ‑ 쿠키가 잘 등록되는지 확인 (AuthService)
…
edonymyeon.backend.content.comment.application.CommentServiceTest ‑ 댓글이 삭제되면 조회되지 않는다 (CommentRepository)
edonymyeon.backend.content.comment.docs.CommentControllerDocsTest ‑ 댓글을 삭제한다
edonymyeon.backend.content.comment.docs.CommentControllerDocsTest ‑ 댓글을 작성한다
edonymyeon.backend.content.comment.docs.CommentControllerDocsTest ‑ 댓글을 조회한다
edonymyeon.backend.content.comment.integration.CommentIntegrationTest ‑ path variable의 게시글 id가 삭제하려는 댓글이 속한 게시글의 id와 같지 않으면 댓글을 삭제할 수 없다
edonymyeon.backend.content.comment.integration.CommentIntegrationTest ‑ 게시글이 존재하지 않으면 예외가 발생한다
edonymyeon.backend.content.comment.integration.CommentIntegrationTest ‑ 댓글 작성자가 댓글을 삭제한다
edonymyeon.backend.content.comment.integration.CommentIntegrationTest ‑ 댓글 작성자가 댓글을 조회하면 isWriter 값은 true다
edonymyeon.backend.content.comment.integration.CommentIntegrationTest ‑ 댓글 작성자가 아닌 사람은 댓글을 삭제할 수 없다
edonymyeon.backend.content.comment.integration.CommentIntegrationTest ‑ 댓글이 없으면 빈 리스트가 조회된다
…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

패키지간의 양방향을 끊어낸다.
3 participants