Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Test : 게시판 관련 로직 테스트 코드 작성 #398

Merged
merged 27 commits into from
Mar 25, 2024
Merged

Test : 게시판 관련 로직 테스트 코드 작성 #398

merged 27 commits into from
Mar 25, 2024

Conversation

oxix97
Copy link
Collaborator

@oxix97 oxix97 commented Mar 20, 2024 •
edited by GBGreenBravo
Loading

개요

  • PostReader
  • PostStore
  • PostFacade
  • PostController

위의 테스트 코드를 작성하였습니다.

작성을 이렇게 하는게 맞는지 좀 의문이 들어서 피드백 야무지게 해주시면 감사하겠습니다.

closed #395

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제
  • 배포 및 PR 관련

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

oxix97 and others added 15 commits March 20, 2024 13:44
@oxix97
Merge branch 'test/Kernel360#393-base-test' into dev
14b6073
@oxix97
Merge branch 'develop' of https://github.com/Kernel360/f1-Orury-Backend…
c9912a0 
… into dev
@GBGreenBravo
chore : 리팩토링 전 테스트코드 삭제
08cc657
@GBGreenBravo
fix : 크루지원자의 검증으로 변경
5bf30d2
@GBGreenBravo
fix : Gym 생성자에 createdAt, updatedAt 추가
b3f808a
@GBGreenBravo
feat : Request, Response를 생성하는 ClientFixtureFactory
fc88782
@GBGreenBravo
feat : Entity, Dto를 생성하는 DomainFixtureFactory
e57f5ab
@GBGreenBravo
test : FixtureFactory의 생성 메서드를 호출하도록 리팩토링
d2d7670
@oxix97
Chore : �Post, PostDto, PostLike 추가
678943b
@oxix97
Test : 게시판 조회 테스트
03b3efb
@oxix97
�Refactor : 인기 게시글 조회 로직 수정
612aec4
@oxix97
Chore : 테스트 코드 팩토리 누락 사항 추가
d26aed2
@oxix97
Chore : 테스트 중 알림 관련 Bean 생성 오류로 인한 조치
e0896d9
@oxix97
Test : 게시판 Facade 테스트
404a9ac
@oxix97
Test : 게시판 컨트롤러 테스트 (없어도 될거같음)
928379e
@oxix97 oxix97 added 💻 BE 백엔드 ✅ Test 테스트 코드를 추가, 변경, 리팩토링한 경우 labels Mar 20, 2024
@oxix97 oxix97 self-assigned this Mar 20, 2024
GBGreenBravo
GBGreenBravo approved these changes Mar 21, 2024
View reviewed changes
Copy link
Collaborator

@GBGreenBravo GBGreenBravo left a comment

Choose a reason for hiding this comment

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

매우 상세하게 테스트코드 짜셨네요..!!!!

orury-domain/src/main/java/org/orury/domain/post/domain/PostReader.java Outdated Show resolved Hide resolved
orury-client/src/test/java/org/orury/client/post/application/PostFacadeTest.java Outdated Show resolved Hide resolved
orury-client/src/test/java/org/orury/client/post/interfaces/PostControllerTest.java Outdated Show resolved Hide resolved
orury-domain/src/test/java/org/orury/domain/post/infrastructure/PostReaderTest.java Outdated Show resolved Hide resolved
orury-domain/src/test/java/org/orury/domain/post/infrastructure/PostReaderTest.java Show resolved Hide resolved
@GBGreenBravo
Merge pull request Kernel360#397 from GBGreenBravo/test/#396_Test_Fix…
46c50ef 
…tures_Integration

Test : Test Fixture 생성 코드 통합
c0olcode
c0olcode approved these changes Mar 21, 2024
View reviewed changes
Copy link
Member

@c0olcode c0olcode left a comment

Choose a reason for hiding this comment

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

이제 테스트코드는 코.......의 힘으로 ... LGTM ^^

kkkapuq
kkkapuq approved these changes Mar 25, 2024
View reviewed changes
Copy link
Collaborator

@kkkapuq kkkapuq left a comment

Choose a reason for hiding this comment

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

고생하셨습니다..!

@oxix97 oxix97 merged commit 585a277 into Kernel360:develop Mar 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GBGreenBravo GBGreenBravo GBGreenBravo approved these changes

@c0olcode c0olcode c0olcode approved these changes

@kkkapuq kkkapuq kkkapuq approved these changes

Assignees

@oxix97 oxix97

Labels
💻 BE 백엔드 ✅ Test 테스트 코드를 추가, 변경, 리팩토링한 경우
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Post 관련 로직 테스트 코드 작성
4 participants
@oxix97 @kkkapuq @c0olcode @GBGreenBravo