-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
[BE] issue241: 테스트 코드 리팩토링 #242
Merged
Merged
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
20656f8
refactor: ReviewDaoTest 리팩토링
tco0427 2bad521
refactor: createdDate 수정 및 schema 변경
tco0427 778c200
refactor: ReviewControllerTest 리팩토링
tco0427 db06960
refactor: MemberData Fixture 사용
tco0427 2e32b80
refactor: MyStudyControllerTest 리팩토링
tco0427 7ef8eb8
Merge branch 'develop' of https://github.com/woowacourse-teams/2022-m…
tco0427 65f6a01
refactor: MyStudyControllerTest 리팩토링 완료
tco0427 adfb22b
refactor: MyStudyDaoTest
tco0427 3a875e4
refactor: MyStudyDaoTest 수정 중
tco0427 64fa5b2
refactor: MyStudyDaoTest 완료
tco0427 d027489
fix: 깨지는 테스트 수정
tco0427 926f950
fix: id 값은 알 수 없으므로 null만 검사하도록 수정
tco0427 5de9bd5
refactor: MemberFixtures에서 Member 상수 -> 메서드로 변경
sc0116 34ac7f4
refactor: ReviewFixtures에서 Review 상수 -> 메서드로 변경
sc0116 38d6d3e
refactor: StudyDetailsDaoTest 리팩토링
tco0427 48a7f8f
Merge branch 'refactor/241-test-refactoring' of https://github.com/wo…
tco0427 d5bccdc
refactor: MyStudyServiceTest 리팩토링
tco0427 7e39ade
refactor: 불필요한 StudyFixtures 제거
tco0427 0e976f6
refactor: TagDaoTest 수정
tco0427 d0d87d8
refactor: ReviewDaoTest 리팩토링
tco0427 6d28741
refactor: 사용하지 않는 Fixture 제거
tco0427 4f7da73
fix: ReviewDaoTest 수정
tco0427 19b374e
feat: 스터디 최대 인원이 한명인 경우 바로 모집 종료가 된다.
tco0427 59a2a51
fix: 테스트 코드 통과하도록 수정
tco0427 36e69e7
refactor: 린론 피드백 반영
tco0427 1236b0f
Merge branch 'refactor/241-test-refactoring' of https://github.com/wo…
tco0427 7feb089
refactor: DTO 검증문 추가
tco0427 8ef9f56
fix: 테스트 수정
tco0427 e1efa66
refactor: 픽스쳐 추가
tco0427 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
굿!!