fix: PolicySyncServiceTest 전량실패 테스트가 길이절단 가드와 충돌해 CI 실패 - #224
Merged
Conversation
#219와 #221이 독립적으로 작업되며 PolicySyncServiceTest에서 같은 "길이 초과로 DB 저장 실패" 트리거를 썼는데, #221이 그 실패 자체를 없애버려(초과분을 잘라 저장) #219가 추가한 전량실패 테스트가 dev에서 깨졌다. plcyNm 누락(title NOT NULL 위반) 방식으로 트리거를 통일한다. Closes #223
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
변경 내용
PolicySyncServiceTest.실패율_초과로_전량_실패해도_실패_건수가_이력에_그대로_기록된다()의 실패 유도 방식을 길이 초과 대신plcyNm누락(title NOT NULL 위반)으로 변경배경
#219와 #221이 각각 독립 브랜치에서 작업되며
PolicySyncServiceTest에서 동일하게 "필드 길이 초과 → DB 저장 실패"를 실패 트리거로 사용했다. #221이 병합되며 길이 초과를 예외 대신 절단·저장으로 바꿔서, #219가 추가한 전량실패 테스트가 dev에서 실제로는 저장에 성공해버려FAILED를 기대하는 assertion이 깨졌다(dev→main PR #222의 CI에서 발견). 같은 파일의실패율이_10퍼센트를_넘으면_...테스트가 이미 쓰고 있는plcyNm누락 트리거로 통일해 해결한다.검증
./gradlew test --tests "com.bop.youthpick.sync.service.PolicySyncServiceTest"— 통과 (재현 전: 1개 실패, 수정 후: 전부 통과)./gradlew test전체,./gradlew spotlessCheck— 모두 통과연결 이슈
Closes #223