refactor/KD-30 User, Comment, Post의 도메인과 JPA 엔티티 분리 (#265)

* refactor: Post 도메인 엔티티 분리

* refactor: Comment 엔티티 도메인 분리

* refactor: User 엔티티 도메인 분리

* refactor: PostRepository delete 메서드 제거

* test: delete후 data를 merge하도록 변경

* refactor: 불필요한 import 제거

* refactor: 불필요한 중복 FROM 제거

* test: FakePostRepository, FakeCommentRepository의 save 메서드 로직을 조회->삭제->저장 순으로 변경

* fix: SwaggerConfig 설정 관련 커밋 실수 undo

* refactor: fromDomain 메서드명 toEntity로 변경

* refactor: CommentJpaEntity의 toDomain 파라미터명 올바르게 변경

* test: 테스트 통과를 위해 더미 post에 title, content 주입

Author: JangYeongHu

aics-admin/src/testFixtures/java/post/application/PostAdminFacadeTest.java

@@ -43,7 +43,7 @@ public void init() {
 		UserQueryService userQueryService = new UserQueryService(fakeUserRepository);
 		FileQueryService fileQueryService = new FileQueryService(fakeFileRepository);
 		this.postAdminFacade = new PostAdminFacade(
-			new PostCommandService(userQueryService, fakePostRepository, fileQueryService),
+			new PostCommandService(userQueryService, fakePostRepository),
 			new PostQueryService(fakePostRepository, fakeFileRepository, fakeUserRepository),
 			new PostSchedulingService(fakePostRepository)
 		);

aics-api/src/testFixtures/java/comment/application/CommentFacadeTest.java

@@ -78,6 +78,7 @@ public void init() {
 			"test2", author.getId(), post.getId()
 		));
 		delete.delete();
+		fakeCommentRepository.save(delete);
 
 		UserDetails user = userQueryService.getUserById("202411345");
 		SecurityContext context = SecurityContextHolder.getContext();

aics-api/src/testFixtures/java/post/application/PostFacadeTest.java

@@ -44,10 +44,9 @@ public void init() {
 		FakeFileRepository fakeFileRepository = new FakeFileRepository();
 
 		UserQueryService userQueryService = new UserQueryService(fakeUserRepository);
-		FileQueryService fileQueryService = new FileQueryService(fakeFileRepository);
 
 		postFacade = new PostFacade(
-			new PostCommandService(userQueryService, fakePostRepository, fileQueryService),
+			new PostCommandService(userQueryService, fakePostRepository),
 			new PostQueryService(fakePostRepository, fakeFileRepository, fakeUserRepository),
 			new UserQueryService(fakeUserRepository)
 		);
@@ -75,6 +74,7 @@ public void init() {
 			"second title", "second content", NEWS, author.getId(), null, false
 		));
 		delete.delete();
+		fakePostRepository.save(delete);
 
 		fakePostRepository.save(Post.create(
 			"third title", "third content", NEWS, author.getId(), null, false

aics-common/src/main/java/kgu/developers/common/domain/BaseTimeEntity.java

@@ -4,6 +4,7 @@
 import jakarta.persistence.EntityListeners;
 import jakarta.persistence.MappedSuperclass;
 import lombok.Getter;
+import lombok.Setter;
 import org.hibernate.annotations.CreationTimestamp;
 import org.hibernate.annotations.UpdateTimestamp;
 import org.springframework.data.jpa.domain.support.AuditingEntityListener;
@@ -23,7 +24,8 @@ public abstract class BaseTimeEntity {
 	@Column(nullable = false)
 	protected LocalDateTime updatedAt;
 
-	@Column
+	@Setter
+    @Column
 	protected LocalDateTime deletedAt;
 
 	public void delete(){

aics-domain/src/main/java/kgu/developers/domain/comment/application/command/CommentCommandService.java

@@ -26,9 +26,11 @@ public Long createComment(String content, Long postId) {
 
 	public void updateComment(Comment comment, String content) {
 		comment.updateContent(content);
+		commentRepository.save(comment);
 	}
 
 	public void deleteComment(Comment comment) {
 		comment.delete();
+		commentRepository.save(comment);
 	}
 }

aics-domain/src/main/java/kgu/developers/domain/comment/domain/Comment.java

@@ -1,36 +1,30 @@
 package kgu.developers.domain.comment.domain;
 
-import static jakarta.persistence.GenerationType.IDENTITY;
-
-import jakarta.persistence.Column;
-import jakarta.persistence.Entity;
-import jakarta.persistence.GeneratedValue;
-import jakarta.persistence.Id;
-import kgu.developers.common.domain.BaseTimeEntity;
 import lombok.AllArgsConstructor;
 import lombok.Builder;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 
-@Entity
+import java.time.LocalDateTime;
+
 @Getter
 @Builder
 @NoArgsConstructor
 @AllArgsConstructor
-public class Comment extends BaseTimeEntity {
-	@Id
-	@GeneratedValue(strategy = IDENTITY)
+public class Comment {
+
 	private Long id;
 
-	@Column(nullable = false)
 	private String content;
 
-	@Column(nullable = false)
 	private Long postId;
 
-	@Column(nullable = false)
 	private String authorId;
 
+	protected LocalDateTime createdAt;
+	protected LocalDateTime updatedAt;
+	protected LocalDateTime deletedAt;
+
 	public static Comment create(String content, String authorId, Long postId) {
 		return Comment.builder()
 			.content(content)
@@ -43,4 +37,8 @@ public void updateContent(String content) {
 		this.content = content;
 	}
 
+	public void delete() {
+		this.deletedAt = LocalDateTime.now();
+	}
+
 }

aics-domain/src/main/java/kgu/developers/domain/comment/infrastructure/CommentJpaEntity.java

@@ -0,0 +1,72 @@
+package kgu.developers.domain.comment.infrastructure;
+
+import jakarta.persistence.Column;
+import jakarta.persistence.Entity;
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.Id;
+import jakarta.persistence.Table;
+import kgu.developers.common.domain.BaseTimeEntity;
+import kgu.developers.domain.comment.domain.Comment;
+import lombok.AllArgsConstructor;
+import lombok.Builder;
+import lombok.Getter;
+import lombok.NoArgsConstructor;
+
+import static jakarta.persistence.GenerationType.IDENTITY;
+
+@Entity
+@Getter
+@Builder
+@Table(name = "comment")
+@NoArgsConstructor
+@AllArgsConstructor
+public class CommentJpaEntity extends BaseTimeEntity {
+    @Id
+    @GeneratedValue(strategy = IDENTITY)
+    private Long id;
+
+    @Column(nullable = false)
+    private String content;
+
+    @Column(nullable = false)
+    private Long postId;
+
+    @Column(nullable = false)
+    private String authorId;
+
+    public static CommentJpaEntity toEntity(final Comment comment) {
+
+        if ( comment == null ) {
+            return null;
+        }
+
+        CommentJpaEntity entity = CommentJpaEntity.builder()
+            .id(comment.getId())
+            .content(comment.getContent())
+            .postId(comment.getPostId())
+            .authorId(comment.getAuthorId())
+            .build();
+
+        entity.setDeletedAt(comment.getDeletedAt());
+
+        return entity;
+    }
+
+    public static Comment toDomain(CommentJpaEntity entity) {
+
+        if ( entity == null ) {
+            return null;
+        }
+
+        return Comment.builder()
+            .id(entity.getId())
+            .content(entity.getContent())
+            .postId(entity.getPostId())
+            .authorId(entity.getAuthorId())
+            .createdAt(entity.getCreatedAt())
+            .updatedAt(entity.getUpdatedAt())
+            .deletedAt(entity.getDeletedAt())
+            .build();
+    }
+
+}

aics-domain/src/main/java/kgu/developers/domain/comment/infrastructure/CommentRepositoryImpl.java

@@ -17,12 +17,16 @@ public class CommentRepositoryImpl implements CommentRepository {
 
 	@Override
 	public Comment save(Comment comment) {
-		return jpaCommentRepository.save(comment);
+		CommentJpaEntity entity = CommentJpaEntity.toEntity(comment);
+		CommentJpaEntity savedEntity = jpaCommentRepository.save(entity);
+		return CommentJpaEntity.toDomain(savedEntity);
 	}
 
 	@Override
 	public Optional<Comment> findByIdAndDeletedAtIsNull(Long commentId) {
-		return jpaCommentRepository.findByIdAndDeletedAtIsNull(commentId);
+
+		return jpaCommentRepository.findByIdAndDeletedAtIsNull(commentId)
+			.map(CommentJpaEntity::toDomain);
 	}
 
 	@Override
@@ -32,7 +36,9 @@ public void deleteAllByDeletedAtBefore(int retentionDays) {
 
 	@Override
 	public List<Comment> findAllByPostIdAndDeletedAtIsNull(Long postId) {
-		return jpaCommentRepository.findAllByPostIdAndDeletedAtIsNull(postId);
+		return jpaCommentRepository.findAllByPostIdAndDeletedAtIsNull(postId).stream()
+			.map(CommentJpaEntity::toDomain)
+			.toList();
 	}
 
 }

aics-domain/src/main/java/kgu/developers/domain/comment/infrastructure/JpaCommentRepository.java

@@ -5,10 +5,8 @@
 
 import org.springframework.data.jpa.repository.JpaRepository;
 
-import kgu.developers.domain.comment.domain.Comment;
+public interface JpaCommentRepository extends JpaRepository<CommentJpaEntity, Long> {
+	List<CommentJpaEntity> findAllByPostIdAndDeletedAtIsNull(Long postId);
 
-public interface JpaCommentRepository extends JpaRepository<Comment, Long> {
-	List<Comment> findAllByPostIdAndDeletedAtIsNull(Long postId);
-
-	Optional<Comment> findByIdAndDeletedAtIsNull(Long commentId);
+	Optional<CommentJpaEntity> findByIdAndDeletedAtIsNull(Long commentId);
 }

aics-domain/src/main/java/kgu/developers/domain/comment/infrastructure/QueryCommentRepository.java

@@ -1,7 +1,5 @@
 package kgu.developers.domain.comment.infrastructure;
 
-import static kgu.developers.domain.comment.domain.QComment.*;
-
 import java.time.LocalDateTime;
 
 import org.springframework.stereotype.Repository;
@@ -17,6 +15,8 @@ public class QueryCommentRepository {
 	private final JPAQueryFactory queryFactory;
 
 	public void deleteAllByDeletedAtBefore(int retentionDays) {
+
+		QCommentJpaEntity comment = QCommentJpaEntity.commentJpaEntity;
 		LocalDateTime thresholdDate = LocalDateTime.now().minusDays(retentionDays);
 
 		queryFactory.delete(comment)