Skip to content

Conversation

@Gyuhyeok99
Copy link
Contributor

관련 이슈

작업 내용

댓글 관련 통합 테스트 코드 추가하였습니다.

댓글관련통합테스트
  • 댓글 조회
  • 댓글 생성
  • 댓글 수정
  • 댓글 삭제

리뷰 요구사항 (선택)

조회와 생성/수정/삭제로 구분이 필요하면 새 refactor pr에서 진행하겠습니다.

이번에도 삭제 관련 테스트 코드에서 @transactional를 사용하게 되었는데 이대로 계속 가야할지 고민을 조금 해보겠습니다 😅

@Gyuhyeok99 Gyuhyeok99 added the 테스트 Added tests label Jan 28, 2025
@Gyuhyeok99 Gyuhyeok99 self-assigned this Jan 28, 2025
@Gyuhyeok99 Gyuhyeok99 linked an issue Jan 28, 2025 that may be closed by this pull request
1 task
Copy link
Collaborator

@nayonsoso nayonsoso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

이런 가독성의 테스트 코드라니... 🥹🥹🥹
사용자랑 게시판을 fixture로 적절하게 분리해서 더 좋아진 것 같습니다~
제 생각에는 이제 test.unit.service 코드는 지워도 될 것 같은데, 남겨두신 이유가 있으신가요?

@Gyuhyeok99
Copy link
Contributor Author

감사합니다! 아뇨 지워도 될 거 같습니다! 그냥 이 pr에서 지워버리겠습니다!

@Gyuhyeok99 Gyuhyeok99 merged commit b66e610 into main Jan 29, 2025
@Gyuhyeok99 Gyuhyeok99 deleted the test/172-add-comment-integration-test branch January 30, 2025 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

테스트 Added tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test: comment 관련 통합테스트 코드 추가

4 participants