[fix] AiCommentServiceTest 시그니처 변경에 맞게 수정

[k3vin7]

Summary

• #130에서 scheduleReplyForAiComment() 시그니처를 ID 기반으로 변경한 것에 맞게 테스트 수정
• aiReplyCount 검증을 DB에서 재조회하는 방식으로 변경

[gemini-code-assist]

Code Review

This pull request refactors the AI reply scheduling logic to use an event-driven architecture, introducing a new event and listener to decouple the comment and AI comment services. The review feedback suggests simplifying the implementation by removing the redundant postId parameter from the service method and event record, as the post can be accessed directly through the comment entity. Additionally, it is recommended to use more explicit error handling when fetching entities and to utilize JPA relationships to improve efficiency.

[gemini-code-assist]

postId 파라미터는 불필요해 보입니다. aiComment 엔티티가 이미 Post에 대한 참조를 가지고 있으므로 이를 활용할 수 있습니다. 또한, 이벤트 리스너를 통해 전달된 ID로 엔티티를 조회할 때 데이터가 없는 것은 정상적인 상황이 아닐 가능성이 높으므로, 단순히 return 하기보다 orElseThrow를 통해 명시적인 예외를 던져 관측성을 높이는 것이 좋습니다.

    public void scheduleReplyForAiComment(Long aiCommentId) {
        Comment aiComment = commentRepository.findById(aiCommentId)
                .orElseThrow(() -> new NotFoundException("AI 댓글을 찾을 수 없습니다. id=" + aiCommentId));

[gemini-code-assist]

앞서 언급한 대로 aiComment.getPost()를 사용하면 별도의 DB 조회 없이 게시물 엔티티에 접근할 수 있습니다. JPA 프록시를 활용하면 불필요한 SELECT 쿼리 발생도 방지할 수 있어 더 효율적입니다.

Suggested change

	Post post = postRepository.findById(postId).orElse(null);
	if (post == null) {
	return;
	}
	Post post = aiComment.getPost();

[gemini-code-assist]

AiCommentService에서 aiCommentId만으로 필요한 정보를 모두 가져올 수 있게 되었으므로, 이벤트 객체에서도 불필요한 postId 필드를 제거하는 것이 좋습니다.

public record AiReplyScheduleEvent(
        Long aiCommentId
) {

[gemini-code-assist]

서비스 메서드 시그니처 변경에 맞춰 호출부를 수정해야 합니다.

Suggested change

	aiCommentService.scheduleReplyForAiComment(event.aiCommentId(), event.postId());
	aiCommentService.scheduleReplyForAiComment(event.aiCommentId());

[gemini-code-assist]

이벤트 생성 시 불필요한 post.getId() 전달을 제거합니다.

Suggested change

	eventPublisher.publishEvent(new AiReplyScheduleEvent(parentComment.getId(), post.getId()));
	eventPublisher.publishEvent(new AiReplyScheduleEvent(parentComment.getId()));

[gemini-code-assist]

서비스 메서드 시그니처 변경에 맞춰 테스트 코드를 수정합니다.

Suggested change

	aiCommentService.scheduleReplyForAiComment(aiComment.getId(), post.getId());
	aiCommentService.scheduleReplyForAiComment(aiComment.getId());

[gemini-code-assist]

서비스 메서드 시그니처 변경에 맞춰 테스트 코드를 수정합니다.

Suggested change

	aiCommentService.scheduleReplyForAiComment(aiComment.getId(), post.getId());
	aiCommentService.scheduleReplyForAiComment(aiComment.getId());