[fix] 추첨 이벤트 참여 대상이 아닌 이벤트 유저가 참여할 수 있는 문제 수정(#107) #109
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
blaxsior
commented
Aug 20, 2024
| @@ -77,6 +77,10 @@ public boolean isEnded(LocalDateTime now) { | |||
| @JoinColumn(name="event_frame_id") | |||
| private EventFrame eventFrame; | |||
|
|
|||
| // eventFrame을 거치지 않고 유저를 확인하기 위해 사용 | |||
| @Column(name = "event_frame_id", updatable = false, insertable = false) | |||
There was a problem hiding this comment.
이벤트 프레임을 fetch하는 대신 외래키끼리 비교할 수 있도록 외래키를 노출했습니다. 단, 외래키와 JoinColumn이 동시에 존재할 때 둘 모두로 업데이트하려고하면 예외가 발생하므로 외래키로는 업데이트나 삽입 동작이 불가능하도록 false로 지정했습니다.
blaxsior
commented
Aug 20, 2024
| @@ -25,14 +26,15 @@ | |||
| import static org.mockito.Mockito.*; | |||
|
|
|||
| class EventParticipationServiceTest { | |||
| EventMetadataRepository emRepository = mock(EventMetadataRepository.class); | |||
There was a problem hiding this comment.
중복을 제거하고, 의도와는 다르게 동작하던 테스트 코드를 수정했습니다. instanceOf와 hasMessage을 추가하여 기존 테스트코드에 문제가 있음을 식별하고 수정했습니다.
1 task
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.
#️⃣ 연관 이슈
📝 작업 내용
이벤트 유저가 자신이 속하지 않은 이벤트 프레임의 추첨 이벤트에 접근하지 못하도록 제한하는 조건을 추가했습니다. 이벤트 유저와 이벤트는 모두 db 수준에서 이벤트 프레임에 대한 외래키를 가지고 있으므로, 이벤트 프레임을 fetch하지 않더라도 외래키 비교를 통해 두 객체가 동일한 이벤트 프레임에 속하는지 비교할 수 있습니다.
불필요한 db 접근을 줄이기 위해 이벤트 프레임 엔티티를 각각 fetch하는 대신 외래키를 노출하여 비교하는 방식으로 문제를 처리했습니다.