refactor/#230 게시글 고정 여부 추가 (#231)

minjo-on · web-flow · commit ee2cf837d7d7 · 2025-04-01T16:35:54.000+09:00
* refactor: 게시글 생성 시 고정 여부 추가

* refactor: 게시글 수정 시 고정 여부 추가

* refactor: testcode 수정

* Update PostAdminFacadeTest.java

* Update CommentFacadeTest.java

* Update PostFacadeTest.java

* Update PostDomainTest.java
diff --git a/aics-admin/src/main/java/kgu/developers/admin/post/application/PostAdminFacade.java b/aics-admin/src/main/java/kgu/developers/admin/post/application/PostAdminFacade.java
@@ -20,14 +20,14 @@ public class PostAdminFacade {
 	private final PostSchedulingService postSchedulingService;
 
 	public PostPersistResponse createPost(Long fileId, PostCreateRequest request) {
-		Long id = postCommandService.createPost(request.title(), request.content(), request.category(), fileId);
+		Long id = postCommandService.createPost(request.title(), request.content(), request.category(), fileId, request.isPinned());
 		return PostPersistResponse.from(id);
 	}
 
 	@Transactional
 	public void updatePost(Long postId, PostUpdateRequest request) {
 		Post post = postQueryService.getById(postId);
-		postCommandService.updatePost(post, request.title(), request.content(), request.category(), request.fileId());
+		postCommandService.updatePost(post, request.title(), request.content(), request.category(), request.fileId(), request.isPinned());
 	}
 
 	@Transactional
diff --git a/aics-admin/src/main/java/kgu/developers/admin/post/presentation/request/PostCreateRequest.java b/aics-admin/src/main/java/kgu/developers/admin/post/presentation/request/PostCreateRequest.java
@@ -28,6 +28,9 @@ public record PostCreateRequest(
 	String content,
 
 	@Schema(description = "게시물 카테고리", example = "NOTIFICATION", requiredMode = NOT_REQUIRED)
-	Category category
+	Category category,
+
+	@Schema(description = "게시글 고정 여부", example = "FALSE", requiredMode = REQUIRED)
+	boolean isPinned
 ) {
 }
diff --git a/aics-admin/src/main/java/kgu/developers/admin/post/presentation/request/PostUpdateRequest.java b/aics-admin/src/main/java/kgu/developers/admin/post/presentation/request/PostUpdateRequest.java
@@ -31,6 +31,9 @@ public record PostUpdateRequest(
 	Category category,
 
 	@Schema(description = "파일 ID", example = "1")
-	Long fileId
+	Long fileId,
+
+	@Schema(description = "게시글 고정 여부", example = "FALSE", requiredMode = REQUIRED)
+	boolean isPinned
 ) {
 }
diff --git a/aics-admin/src/testFixtures/java/post/application/PostAdminFacadeTest.java b/aics-admin/src/testFixtures/java/post/application/PostAdminFacadeTest.java
@@ -65,7 +65,7 @@ public void init() {
 
 		fakePostRepository.save(
 			Post.create(
-				"post title", "post content", NOTIFICATION, author, null
+				"post title", "post content", NOTIFICATION, author, null, false
 			)
 		);
 	}
diff --git a/aics-api/src/testFixtures/java/comment/application/CommentFacadeTest.java b/aics-api/src/testFixtures/java/comment/application/CommentFacadeTest.java
@@ -62,7 +62,7 @@ public void init() {
 			.build());
 
 		Post post = fakePostRepository.save(Post.create(
-			"테스트용 제목1", "테스트용 내용1", NEWS, author, null
+			"테스트용 제목1", "테스트용 내용1", NEWS, author, null, false
 		));
 
 		fakeCommentRepository.save(Comment.create(
diff --git a/aics-api/src/testFixtures/java/post/application/PostFacadeTest.java b/aics-api/src/testFixtures/java/post/application/PostFacadeTest.java
@@ -67,16 +67,16 @@ public void init() {
 		);
 
 		fakePostRepository.save(Post.create(
-			"first title", "first content", NEWS, author, null
+			"first title", "first content", NEWS, author, null, false
 		));
 
 		Post delete = fakePostRepository.save(Post.create(
-			"second title", "second content", NEWS, author, null
+			"second title", "second content", NEWS, author, null, false
 		));
 		delete.delete();
 
 		fakePostRepository.save(Post.create(
-			"third title", "third content", NEWS, author, null
+			"third title", "third content", NEWS, author, null, false
 		));
 	}
 
diff --git a/aics-domain/src/main/java/kgu/developers/domain/post/application/command/PostCommandService.java b/aics-domain/src/main/java/kgu/developers/domain/post/application/command/PostCommandService.java
@@ -18,21 +18,22 @@ public class PostCommandService {
 	private final PostRepository postRepository;
 	private final FileQueryService fileQueryService;
 
-	public Long createPost(String title, String content, Category category, Long fileId) {
+	public Long createPost(String title, String content, Category category, Long fileId, boolean isPinned) {
 		User author = userQueryService.me();
 
 		FileEntity file = null;
 		if (fileId != null)
 			file = fileQueryService.getFileById(fileId);
 
-		Post post = Post.create(title, content, category, author, file);
+		Post post = Post.create(title, content, category, author, file, isPinned);
 		return postRepository.save(post).getId();
 	}
 
-	public void updatePost(Post post, String title, String content, Category category, Long fileId) {
+	public void updatePost(Post post, String title, String content, Category category, Long fileId, boolean isPinned) {
 		post.updateTitle(title);
 		post.updateContent(content);
 		post.updateCategory(category);
+		post.updatePinned(isPinned);
 
 		FileEntity file = null;
 		if (fileId != null)
diff --git a/aics-domain/src/main/java/kgu/developers/domain/post/domain/Post.java b/aics-domain/src/main/java/kgu/developers/domain/post/domain/Post.java
@@ -65,14 +65,14 @@ public class Post extends BaseTimeEntity {
 	@OneToMany(mappedBy = "post", fetch = LAZY, cascade = ALL, orphanRemoval = true)
 	private List<Comment> comments = new ArrayList<>();
 
-	public static Post create(String title, String content, Category category, User author, FileEntity file) {
+	public static Post create(String title, String content, Category category, User author, FileEntity file, boolean isPinned) {
 		return Post.builder()
 			.title(title)
 			.content(content)
 			.views(0)
-			.isPinned(false)
+			.isPinned(isPinned)
 			.category(category)
-			.author(author) // NOTE: User Setter 주입 방지 위해 생성자 주입
+			.author(author)
 			.file(file)
 			.build();
 	}
@@ -89,8 +89,8 @@ public void updateCategory(Category category) {
 		this.category = category;
 	}
 
-	public void togglePinned() {
-		this.isPinned = !this.isPinned;
+	public void updatePinned(boolean isPinned) {
+		this.isPinned = isPinned;
 	}
 
 	public void increaseViews() {
@@ -100,4 +100,8 @@ public void increaseViews() {
 	public void updateFile(FileEntity file) {
 		this.file = file;
 	}
+
+	public void togglePinned() {
+		this.isPinned = !this.isPinned;
+	}
 }
diff --git a/aics-domain/src/testFixtures/java/comment/application/CommentCommandServiceTest.java b/aics-domain/src/testFixtures/java/comment/application/CommentCommandServiceTest.java
@@ -13,6 +13,8 @@
 import org.springframework.security.core.context.SecurityContext;
 import org.springframework.security.core.context.SecurityContextHolder;
 import org.springframework.security.core.userdetails.UserDetails;
+import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
+import org.springframework.security.crypto.password.PasswordEncoder;
 
 import kgu.developers.domain.comment.application.command.CommentCommandService;
 import kgu.developers.domain.comment.domain.Comment;
@@ -23,8 +25,6 @@
 import mock.repository.FakeCommentRepository;
 import mock.repository.FakePostRepository;
 import mock.repository.FakeUserRepository;
-import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
-import org.springframework.security.crypto.password.PasswordEncoder;
 
 public class CommentCommandServiceTest {
 	private CommentCommandService commentCommandService;
@@ -53,7 +53,7 @@ private static void saveTestUserAndPost(FakeUserRepository fakeUserRepository,
 		);
 		fakePostRepository.save(Post.create(
 			"SW 부트캠프 4기 교육생 모집", "SW전문인재양성사업단에서는 SW부트캠프 4기 교육생을 모집합니다.", NEWS,
-			User.builder().build(), null
+			User.builder().build(), null, false
 		));
 	}
 
diff --git a/aics-domain/src/testFixtures/java/comment/application/CommentQueryServiceTest.java b/aics-domain/src/testFixtures/java/comment/application/CommentQueryServiceTest.java
@@ -55,7 +55,7 @@ private static void saveTestComment(FakeCommentRepository fakeCommentRepository,
 	private static Post saveTestPost() {
 		FakePostRepository fakePostRepository = new FakePostRepository();
 		Post commentedPost = Post.create("SW 부트캠프 4기 교육생 모집",
-			"SW전문인재양성사업단에서는 SW부트캠프 4기 교육생을 모집합니다.", NEWS, User.builder().build(), null);
+			"SW전문인재양성사업단에서는 SW부트캠프 4기 교육생을 모집합니다.", NEWS, User.builder().build(), null, false);
 		return fakePostRepository.save(commentedPost);
 	}
 
diff --git a/aics-domain/src/testFixtures/java/comment/domain/CommentDomainTest.java b/aics-domain/src/testFixtures/java/comment/domain/CommentDomainTest.java
@@ -8,12 +8,12 @@
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
+import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
+import org.springframework.security.crypto.password.PasswordEncoder;
 
 import kgu.developers.domain.comment.domain.Comment;
 import kgu.developers.domain.post.domain.Post;
 import kgu.developers.domain.user.domain.User;
-import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
-import org.springframework.security.crypto.password.PasswordEncoder;
 
 public class CommentDomainTest {
 	private Comment comment;
@@ -45,7 +45,8 @@ private Post getPost(User author) {
 			"content.",
 			NEWS,
 			author,
-			null
+			null,
+			false
 		);
 	}
 
diff --git a/aics-domain/src/testFixtures/java/post/application/PostCommandServiceTest.java b/aics-domain/src/testFixtures/java/post/application/PostCommandServiceTest.java
@@ -65,7 +65,7 @@ public void createPost_Success() {
 		Long fileId = 1L;
 
 		// when
-		Long result = postCommandService.createPost(title, content, category, fileId);
+		Long result = postCommandService.createPost(title, content, category, fileId, false);
 
 		// then
 		assertEquals(1L, result);
@@ -82,7 +82,7 @@ public void updatePost_Success() {
 		Category newCategory = NOTIFICATION;
 
 		// when
-		postCommandService.updatePost(post, newTitle, newContent, newCategory, 1L);
+		postCommandService.updatePost(post, newTitle, newContent, newCategory, 1L, false);
 
 		// then
 		assertEquals(newTitle, post.getTitle());
diff --git a/aics-domain/src/testFixtures/java/post/application/PostQueryServiceTest.java b/aics-domain/src/testFixtures/java/post/application/PostQueryServiceTest.java
@@ -36,28 +36,28 @@ public void init() {
 
 		fakePostRepository.save(Post.create(
 			"테스트용 제목1", "테스트용 내용1",
-			NEWS, author, null
+			NEWS, author, null, true
 		));
 
 		fakePostRepository.save(Post.create(
 			"테스트용 제목2", "테스트용 내용2",
-			NEWS, author, null
+			NEWS, author, null, false
 		));
 
 		Post delete = fakePostRepository.save(Post.create(
 			"테스트용 제목3", "테스트용 내용3",
-			NEWS, author, null
+			NEWS, author, null, false
 		));
 		delete.delete();
 
 		fakePostRepository.save(Post.create(
 			"테스트용 제목4", "테스트용 내용4",
-			NOTIFICATION, author, null
+			NOTIFICATION, author, null, false
 		));
 
 		fakePostRepository.save(Post.create(
 			"테스트용 제목5", "테스트용 내용5",
-			NEWS, author, null
+			NEWS, author, null, false
 		));
 	}
 
diff --git a/aics-domain/src/testFixtures/java/post/domain/PostDomainTest.java b/aics-domain/src/testFixtures/java/post/domain/PostDomainTest.java
@@ -13,12 +13,12 @@
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
+import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
+import org.springframework.security.crypto.password.PasswordEncoder;
 
 import kgu.developers.domain.post.domain.Category;
 import kgu.developers.domain.post.domain.Post;
 import kgu.developers.domain.user.domain.User;
-import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
-import org.springframework.security.crypto.password.PasswordEncoder;
 
 public class PostDomainTest {
 	private Post post;
@@ -29,7 +29,7 @@ public class PostDomainTest {
 	@BeforeEach
 	public void init() {
 		User author = user();
-		post = Post.create(TITLE, CONTENT, NEWS, author, null);
+		post = Post.create(TITLE, CONTENT, NEWS, author, null, true);
 	}
 
 	private User user() {
@@ -52,7 +52,7 @@ public void createPost_Success() {
 		assertEquals(TITLE, post.getTitle());
 		assertEquals(CONTENT, post.getContent());
 		assertEquals(0, post.getViews());
-		assertFalse(post.isPinned());
+		assertTrue(post.isPinned());
 		assertEquals(NEWS, post.getCategory());
 	}
 
@@ -116,11 +116,11 @@ public void togglePostPinned_Success() {
 		// when
 		post.togglePinned();
 		// then
-		assertTrue(post.isPinned());
+		assertFalse(post.isPinned());
 
 		// when
 		post.togglePinned();
 		// then
-		assertFalse(post.isPinned());
+		assertTrue(post.isPinned());
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -20,14 +20,14 @@ public class PostAdminFacade {`
`20`	`20`	`private final PostSchedulingService postSchedulingService;`
`21`	`21`
`22`	`22`	`public PostPersistResponse createPost(Long fileId, PostCreateRequest request) {`
`23`		`- Long id = postCommandService.createPost(request.title(), request.content(), request.category(), fileId);`
	`23`	`+ Long id = postCommandService.createPost(request.title(), request.content(), request.category(), fileId, request.isPinned());`
`24`	`24`	`return PostPersistResponse.from(id);`
`25`	`25`	`}`
`26`	`26`
`27`	`27`	`@Transactional`
`28`	`28`	`public void updatePost(Long postId, PostUpdateRequest request) {`
`29`	`29`	`Post post = postQueryService.getById(postId);`
`30`		`- postCommandService.updatePost(post, request.title(), request.content(), request.category(), request.fileId());`
	`30`	`+ postCommandService.updatePost(post, request.title(), request.content(), request.category(), request.fileId(), request.isPinned());`
`31`	`31`	`}`
`32`	`32`
`33`	`33`	`@Transactional`
Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ public void init() {`
`65`	`65`
`66`	`66`	`fakePostRepository.save(`
`67`	`67`	`Post.create(`
`68`		`- "post title", "post content", NOTIFICATION, author, null`
	`68`	`+ "post title", "post content", NOTIFICATION, author, null, false`
`69`	`69`	`)`
`70`	`70`	`);`
`71`	`71`	`}`
Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ private static void saveTestComment(FakeCommentRepository fakeCommentRepository,`
`55`	`55`	`private static Post saveTestPost() {`
`56`	`56`	`FakePostRepository fakePostRepository = new FakePostRepository();`
`57`	`57`	`Post commentedPost = Post.create("SW 부트캠프 4기 교육생 모집",`
`58`		`- "SW전문인재양성사업단에서는 SW부트캠프 4기 교육생을 모집합니다.", NEWS, User.builder().build(), null);`
	`58`	`+ "SW전문인재양성사업단에서는 SW부트캠프 4기 교육생을 모집합니다.", NEWS, User.builder().build(), null, false);`
`59`	`59`	`return fakePostRepository.save(commentedPost);`
`60`	`60`	`}`
`61`	`61`