[Feat/#10] 리뷰, 기록 부분 기능 구현 #13
Open
Conversation
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
- 사용자 ID로 리뷰 리스트 조회 레포지토리 기능 구현 및 테스트
… 기능 구현 및 테스트 - 서비스 계층까지 구현
…Controller): 리뷰 생성 기능 구현 - 이어서 Mock 기반 테스트 코드 작성
…): 리뷰 생성 기능 테스트 코드 작성 - 추가적으로 테스트 하는데 있어서 어려움이 있어 임시로 BookRepository, UserRepository를 생성해주었습니다. - 생성 해줌에 따라 기존에 있던 리뷰 조회 기능 테스트 코드 로직 또한 바뀌었습니다.
…삭제 기능 구현 및 테스트 - 약간의 변수명 변경을 곁들인..
…Response, RecordService, RecordServiceTest, UserGroupRepository): 기록 조회 기능 구현 및 테스트 구현 중 기존 엔티티 필드 추가가 필요해서 추가해주었고, 테스트를 위해 필요한 레포지토리를 생성했습니다.
…upRepository): 기록 생성 기능 구현 및 테스트 - 이미지 부분은 이 후 고민 해보기로 하였습니다.
- 테스트 코드가 너무 어지러워 리팩토링 했습니다
- 기록 생성 기능 구현에 따라 테스트 코드 로직 변경
…록 생성 & 조회 시, 이미지 부분 마저 구현
- 기록 리스트 반환 시, 코드 간결하게 수정하였습니다 - 기록 수정 시, 필요 없는 save 호출 제거를 해주었습니다. 이유) JPA 엔티티 상태 변경을 할 경우, 트랜잭션 종료시 dirty checking을 통해 자동으로 변경 사항이 저장되기 때문
82everywin
approved these changes
Dec 3, 2024
src/main/java/com/bookmile/backend/domain/record/dto/RecordListResponse.java
Outdated
Show resolved
Hide resolved
src/main/java/com/bookmile/backend/domain/record/entity/Record.java
Outdated
Show resolved
Hide resolved
src/main/java/com/bookmile/backend/domain/record/entity/Record.java
Outdated
Show resolved
Hide resolved
82everywin
reviewed
Dec 3, 2024
- 자연스럽게 대문자로 적고 넘어갔었던... ~
…ervice, RequestRecord, ImageController, ImageListResponse, ImageRepository, ImageSaveRequest, ImageService, RequestUpdateRecord, RecordServiceTest): Record-Image 간 로직 수정 - 연관 관계 다시 명시 해주었습니다. - 이에 따라 Image 자체에서 조회, 생성, 삭제 기능 구현 하였습니다. - 이 후 테스트 코드 작성과 일부 코드 수정이 있을 예정입니다.
… 조회, 삭제 기능 테스트 코드 작성 - ImageServiceTest 파일을 포함해 다른 테스트 코드 또한 트랜잭션 어노테이션 적용을 해주었습니다.
- 수정 후, 테스트 코드 작동으로 올바르게 merge 된 것을 확인 했습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
✅ PR 타입(하나 이상의 PR 타입을 선택해주세요)
연결된 issue
close #
💻작업사항
필요시 실행결과 스크린샷 첨부
💡작성한 이슈 외에 작업한 사항
🩷 Approve 하기 전 확인해주세요!
리뷰어가 확인해줬으면 하는 사항 적어주세요.