Feature/65 커뮤니티 댓글 생성 기능 #67

ayoung-dev · 2024-11-28T02:37:19Z

resolved :

[FEATURE] 커뮤니티 댓글 생성 기능 #65

📌 과제 설명

커뮤니티 댓글 생성 기능 구현

👩‍💻 요구 사항과 구현 내용

커뮤니티 댓글 / 대댓글 생성 기능 구현

✅ PR 포인트 & 궁금한 점

댓글과 대댓글 생성의 테스트를 두개로 나눠뒀는데 하나로 합치는게 나을지 궁금합니다.
board와 comment 파일이 헷갈릴 수 있겠다고 판단하여 폴더로 한 번 더 나누었는데 괜찮은지 여쭤봅니다.

m-a-king

수고하셨습니다. 작업에 가속이 붙으셨네요... 멋져요!

댓글이 삭제된 경우에는 대댓글이 존재할 수 있지만, 댓글이 존재한 흔적이 없다면 그 댓글이 부모가 될 수 없다

라는 검증이 필요할 것 같습니다! 혹시 제가 확인하지 못한걸까요?

m-a-king · 2024-11-28T03:24:48Z

src/test/java/com/somemore/community/repository/CommunityCommentRepositoryTest.java

+        assertThat(savedComment.getParentCommentId()).isNull();
+    }
+
+    @DisplayName("커뮤니티 게시글에 대댓글을 생성할 수 있다. (Repository)")


Suggested change

@DisplayName("커뮤니티 게시글에 대댓글을 생성할 수 있다. (Repository)")

@DisplayName("댓글에 대댓글을 생성할 수 있다. (Repository)")

이런 맥락이 더 좋지 않을까요?
그런 의미에서 parent가 없는 경우에 대한 검증이 여기서나 서비스 테스트에서 필요할 것 같아요!

테스트 네임은 바꾸겠습니다!

부모 댓글 검증을 조회에서만 생각했네요
추가하겠습니다! 감사합니다

넵!

board와 comment 파일이 헷갈릴 수 있겠다고 판단하여 폴더로 한 번 더 나누었는데 괜찮은지 여쭤봅니다.

이 부분도 저는 좋은 것 같습니다~

재중님 생각해봤는데
댓글 생성 쪽에서는 삭제된 댓글에는 대댓글 생성을 못하도록 하고
댓글 조회 쪽에서 삭제된 경우에도 대댓글이 있으면 조회를 하도록 해야할 거 같은데
혹시 더 생각해야할 부분이 있을까요?

leebs0521

고생하셨습니다.!!
전체적으로 잘 작성하신 것 같아요.

댓글 작성과 대댓글 작성 API가 분리되어 있는지 궁금하네요

leebs0521 · 2024-11-28T03:40:38Z

src/main/java/com/somemore/community/domain/CommunityComment.java

+@NoArgsConstructor(access = PROTECTED)
+@Entity
+@Table(name = "Community_comment")
+public class CommunityComment extends BaseEntity {


Community_comment -> community_comment 하시면 좋을 것 같습니다.

앗 수정하겠습니다

leebs0521 · 2024-11-28T03:42:10Z

src/main/java/com/somemore/community/dto/request/CommunityCommentCreateRequestDto.java

+        String content
+) {
+    public CommunityComment toEntity(UUID writerId, Long parentCommentId) {
+        return CommunityComment.builder()


parentCommentId는 어디서 오는지 궁금합니다!
requestDto에 필드에 담겨서 오는게 아닐까요?
아니면 대댓글 작성 API가 따로 있어서 Pathvariable에 담겨오는 걸까요?

댓글 작성과 대댓글 작성 API가 분리되어 있는지 궁금하네요

아뇨! 하나예요
requestDto에 담겨서 오는 게 맞네요 (왜 갑자기 이렇게 생각했을까요...)
수정하겠습니다!

* feat: 이미지 업로드 기능 구현 - aws s3 연동을 위한 의존성 추가 - 환경변수 추가 - S3 연결을 위한 config 클래스 구현 - 서비스 레이어에 이미지 업로드 기능 구현 - 업로드 실패 예외와 예외 메세지 추 - test.yml에 테스트에 필요한 환경변수 추가 - 테스트 코드 작성및 검증 완료 * chore: 환경변수 추가 - cicd workflow 에 이미지 업로드 관련 환경변수 추가 - appication.yml에 이미지 업로드 관련 환경변수 이름 변경 * feat: 이미지 파일 검증 기능 구현 - 이미지 크기를 8MB로 제한 - 이미지의 크기 초과를 검증하는 기능 구현 - 이미지의 확장자명을 검증하는 기능 구현 - 이미지 업로드시 검증 기능들을 수행하도록 수정 - 전역 예외처리 핸들러에 이미지 업로드 예외 추가 - 예외 메세지 추가 - 테스트 코드 작성및 검증 완 * fix: 코드 리뷰 사항 반영 - BASE_URL을 S3_BASE_URL라는 더 명시적인 이름으로 변경 - ImageUploadUtils private 생성자의 예외처리 메세지를 ExceptionMessage에 등록하여 사용하도록 수정 - 이미지 업로드 메서드의 내부 기능을 메서드로 추출하여 가독성 향상 - ImageUploadUtils의 일부 기능의 접근 제어자를 private으로 수정 - 테스트 코드의 when, then을 더 명확하게 분

7zrv · 2024-11-28T05:42:34Z

src/test/java/com/somemore/community/service/comment/CreateCommunityCommentServiceTest.java

+
+import static org.assertj.core.api.Assertions.assertThat;
+
+class CreateCommunityCommentServiceTest extends IntegrationTestSupport {


억지스러운 예외일수도 있지만 대댓글 작성중에 댓글이 삭제될수도 있다는 생각이 들었습니다
관련 예외도 테스트해보면 좋지 않을까... 생각만 해봤습니다

삭제된 댓글에는 대댓글을 달지 못하는 게 맞는 거 같아요.
그 부분 추가했습니다!

- repository 테스트에 findById 테스트 추가 - service 테스트에 대댓글 예외 테스트 추가

- parentCommentId Dto 필드로 수정

m-a-king · 2024-11-28T07:23:38Z

src/test/java/com/somemore/community/service/comment/CreateCommunityCommentServiceTest.java

+
+    @DisplayName("삭제된 댓글에 대댓글을 등록할 때 예외를 던진다.")
+    @Test
+    void createCommunityCommentRelyWithDeletedParentId() {


Rely -> Reply 오타 있어요~

와우 수정했습니다

m-a-king · 2024-11-28T07:29:17Z

src/main/java/com/somemore/community/service/comment/CreateCommunityCommentService.java

+        if (communityComment.getParentCommentId() != null) {
+            communityCommentRepository.findById(communityComment.getParentCommentId())
+                    .orElseThrow(() -> new BadRequestException(NOT_EXISTS_COMMUNITY_COMMENT.getMessage()));
+        }


추상화된 private 메서드로 추출할 수 있을 것 같습니다.

Suggested change

if (communityComment.getParentCommentId() != null) {

communityCommentRepository.findById(communityComment.getParentCommentId())

.orElseThrow(() -> new BadRequestException(NOT_EXISTS_COMMUNITY_COMMENT.getMessage()));

}

�checkParentComment(communityComment.getParentCommentId());

수정했습니다!

7zrv · 2024-11-28T07:34:20Z

src/test/java/com/somemore/community/service/comment/CreateCommunityCommentServiceTest.java

+        assertThat(communityCommentReply.get().getParentCommentId()).isEqualTo(commentId);
+    }
+
+    @DisplayName("삭제된 댓글에 대댓글을 등록할 때 예외를 던진다.")


좋습니다

m-a-king

신경 써야 할 검증이 너무 많고 선택적이네요. 수고가 많으십니다.
다음에 댓글, 대댓글 �기능이 완성되고 다시 확인해 보고 싶습니다!

- ServiceTest 오타 수정 - validateParentCommentExists 메서드 추출

- repository 테스트에 findById 테스트 추가 - service 테스트에 대댓글 예외 테스트 추가

- parentCommentId Dto 필드로 수정

- ServiceTest 오타 수정 - validateParentCommentExists 메서드 추출

…-create' into feature/65-add-community-comment-create

sonarqubecloud · 2024-11-28T08:28:16Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

leebs0521 · 2024-11-28T09:19:29Z

꼼꼼히 체크해주신것 같습니다 고생하셨어요.

ayoung-dev added 6 commits November 28, 2024 11:25

test(community): communityComment 생성 기능 테스트 작성

6094c71

feat(community): communityComment Entity, Repository 추가

4b666aa

feat(community): communityComment 생성 RequestDto, Usecase 추가 및 Service 구현

e087c0c

refactor(community): CommunityBoardRepository 네이밍 변경

9a4c087

refactor(community): community board/comment 폴더 구조 변경

58038ee

chore(community): 불필요한 import 및 public 키워드 제거

b686152

ayoung-dev added Feature 아영 labels Nov 28, 2024

ayoung-dev requested review from 7zrv, leebs0521 and m-a-king November 28, 2024 02:37

ayoung-dev self-assigned this Nov 28, 2024

ayoung-dev linked an issue Nov 28, 2024 that may be closed by this pull request

[FEATURE] 커뮤니티 댓글 생성 기능 #65

Closed

4 tasks

m-a-king reviewed Nov 28, 2024

View reviewed changes

leebs0521 reviewed Nov 28, 2024

View reviewed changes

7zrv reviewed Nov 28, 2024

View reviewed changes

ayoung-dev added 4 commits November 28, 2024 16:06

test(community): 커뮤니티 댓글 생성 테스트 추가

f68bb02

- repository 테스트에 findById 테스트 추가 - service 테스트에 대댓글 예외 테스트 추가

chore(community): 소문자로 네이밍 변경

f265514

fix(community): 커뮤니티 댓글 생성 requestDto 수정

10a0dbb

- parentCommentId Dto 필드로 수정

feat(community): 커뮤니티 댓글 생성 예외 처리 및 메세지 추가

3151a8b

m-a-king reviewed Nov 28, 2024

View reviewed changes

7zrv reviewed Nov 28, 2024

View reviewed changes

m-a-king approved these changes Nov 28, 2024

View reviewed changes

7zrv approved these changes Nov 28, 2024

View reviewed changes

ayoung-dev added 4 commits November 28, 2024 16:37

refactor(community): 코드 리뷰 사항 반영

21e7a2a

- ServiceTest 오타 수정 - validateParentCommentExists 메서드 추출

test(community): communityComment 생성 기능 테스트 작성

c5d8e3c

feat(community): communityComment Entity, Repository 추가

dbaf67e

feat(community): communityComment 생성 RequestDto, Usecase 추가 및 Service 구현

3c306dc

ayoung-dev added 11 commits November 28, 2024 16:39

refactor(community): CommunityBoardRepository 네이밍 변경

f4a5f6b

refactor(community): community board/comment 폴더 구조 변경

0f23e4a

chore(community): 불필요한 import 및 public 키워드 제거

b43a5e6

test(community): 커뮤니티 댓글 생성 테스트 추가

7835bb2

- repository 테스트에 findById 테스트 추가 - service 테스트에 대댓글 예외 테스트 추가

chore(community): 소문자로 네이밍 변경

6c0def6

fix(community): 커뮤니티 댓글 생성 requestDto 수정

62aa1e5

- parentCommentId Dto 필드로 수정

feat(community): 커뮤니티 댓글 생성 예외 처리 및 메세지 추가

01656ce

refactor(community): 코드 리뷰 사항 반영

b12b771

- ServiceTest 오타 수정 - validateParentCommentExists 메서드 추출

Merge remote-tracking branch 'origin/feature/65-add-community-comment…

27a199a

…-create' into feature/65-add-community-comment-create

feat(community): 커뮤니티 댓글 존재 여부 메서드 및 테스트 추가

d1caba6

chore(community): 불필요한 import 제거

f7d3478

leebs0521 approved these changes Nov 28, 2024

View reviewed changes

ayoung-dev merged commit 9ccd6ac into main Nov 28, 2024
2 checks passed

ayoung-dev deleted the feature/65-add-community-comment-create branch November 28, 2024 10:37

	@DisplayName("커뮤니티 게시글에 대댓글을 생성할 수 있다. (Repository)")
	@DisplayName("댓글에 대댓글을 생성할 수 있다. (Repository)")


		import static org.assertj.core.api.Assertions.assertThat;

		class CreateCommunityCommentServiceTest extends IntegrationTestSupport {

Feature/65 커뮤니티 댓글 생성 기능 #67

Feature/65 커뮤니티 댓글 생성 기능 #67

Uh oh!

Conversation

ayoung-dev commented Nov 28, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📌 과제 설명

👩‍💻 요구 사항과 구현 내용

✅ PR 포인트 & 궁금한 점

Uh oh!

m-a-king left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

leebs0521 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

m-a-king left a comment

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Nov 28, 2024

Quality Gate passed

Uh oh!

leebs0521 commented Nov 28, 2024

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

ayoung-dev commented Nov 28, 2024 •

edited

Loading