Skip to content

Conversation

@Gyuhyeok99
Copy link
Contributor

@Gyuhyeok99 Gyuhyeok99 commented Jan 29, 2025

관련 이슈

작업 내용

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

리뷰 요구사항 (선택)

작업을 하면서 느낀 게 현재 폴더구조에 board, post, comment가 있는데 전부 community 관련된 것들이더라구요. 특히 board에는 전체 게시글 조회하는 것만 있고 post에 게시글 생성, 단일조회, 수정, 삭제 등이 있는데 한 폴더에서 관리하는 게 더 좋을 거 같다는 생각이 드네요

추가로 게시판 전체 조회는 Page나 Slice 없이 그냥 List로 다 받아오던데 특별한 이유가 있는건가요?

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

@wibaek wibaek left a comment

Choose a reason for hiding this comment

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

lgtm

폴더가 나누어져 있는건 합쳐서 응집도를 높이는게 좋아보입니다. 당시에 폴더 구조를 도메인별로 구분할지 구조에 따라 구분할지 명확하지가 않았어서 과도하게 폴더가 분리된 것 같네요.

게시글 목록도 pagination이 적용 되면 좋을 것 같습니다.

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.

테스트코드 잘 작성해주셨네요😊

저도 패키지 합치는 것 동의합니다~
따로 PR 만들어서 진행해주심 될 것 같아요

게시글 조회할 때 페이지네이션 적용하는 것도 찬성입니다!

그리고 이 PR 이름은 게시글 관련 통합 테스트가 아니라 "게시판" 관련 통합테스트로 바뀌어야 할 것 같네요

@Gyuhyeok99 Gyuhyeok99 changed the title test: 게시글 관련 통합테스트 코드 추가 test: 게시판 관련 통합테스트 코드 추가 Jan 30, 2025
@Gyuhyeok99 Gyuhyeok99 merged commit 9654770 into main Jan 30, 2025
@Gyuhyeok99 Gyuhyeok99 deleted the test/174-add-board-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: board 관련 통합테스트 코드 추가

4 participants