feat: Modifying Annotation 트러블 슈팅 글 작성 #5
Merged
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
kpeel5839
approved these changes
Oct 3, 2023
Comment on lines
+20
to
+21
| 서비스 계층에서 데이터를 삭제하는 테스트를 수행하는 도중, 데이터가 삭제되지 않고 조회되는 문제가 발생하였습니다. | ||
| <br> 지금부터, 그 이야기를 시작해 보려 합니다. |
Comment on lines
+134
to
+142
| (1) `@Query` 어노테이션을 사용하여 직접 쿼리를 작성하는 방법과, (2) 테스트 코드에서 flush & clear를 명시적으로 수행하는 방법이 있었습니다. | ||
| <br> 단순히 테스트 통과에 목적을 둔 것이 아니기 때문에, 실제 프로덕트 코드에서도 예상치 못한 동작을 방지하기 위해 (1)번 방법을 선택하였습니다. | ||
| <br> 이에 따라, `Named Query -> JPQL`로 변경함으로써 문제를 해결할 수 있었습니다. | ||
| ```java | ||
| public interface BookmarkRepository extends JpaRepository<Bookmark, Long> { | ||
| @Modifying(clearAutomatically = true) | ||
| @Query("delete from Bookmark b where b.member.id = :memberId") | ||
| void deleteAllByMemberId(@Param(value = "memberId") Long memberId); | ||
| } |
There was a problem hiding this comment.
지금까지는 굉장히 운이 좋았던거군요!
이거 덕분에 저번에 RefreshToken 삭제가 왜 안되었었는지 알았어요!
아주 좋은 글입니다
yoondgu
approved these changes
Oct 4, 2023
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.
바로보기