Skip to content

#612 [Fix] 펀딩 다중 수량 발급 이후 깨진 펀딩 테스트와 주문 목록 500 수정 - #613

Merged
si-zero merged 1 commit into
devfrom
fix/#612
Nov 7, 2026
Merged

si-zero merged 1 commit into
devfrom
fix/#612

Conversation

@si-zero

@si-zero si-zero commented Nov 7, 2026

Copy link
Copy Markdown
Member

#️⃣ 연관된 이슈

📝 작업 내용

dev 전체 테스트에서 실패하던 펀딩 테스트 5개(AdminFundingApiIntegrationTest 2, TumblbugFundingApiIntegrationTest 3)를 정리합니다.
커밋별로 다시 돌려 보니 #586(관리자 펀딩 주문·이용권 대시보드 API 확장, PR #596)부터 실패합니다. #584~#588 구간은 JwtFilter 401(#603)과 테스트 설정(#606) 때문에 전부 실패해서 가려져 있었습니다.

실패 원인 처리
어드민 주문 목록 GET /api/v1/admin/funding/orders 500 (테스트 2개) searchOrders 쿼리에서 enum 파라미터를 상수와 비교(:deliveryStatus = DeliveryStatus.SUCCESS)해 Hibernate 가 타입을 정하지 못함. 필터 유무와 상관없이 항상 500 전달 상태를 boolean 파라미터로 넘김
주문 목록 deliveryStatus 가 FAILED 를 내려주지 않음 응답 변환에서 SENT/PENDING 만 매핑 마지막 전달 기록이 실패면 FAILED. 주문 단위는 필터와 같은 기준
발급·미리보기 개수 불일치 (테스트 3개) #586·#587 에서 "수량 1 주문만 발급"이 "구매 수량(1~7)만큼 발급"으로 바뀜. 테스트는 수량 2 주문을 보류로 기대 바뀐 규칙에 맞게 기대값 갱신, 결제 대기 주문으로 NOT_ISSUABLE 검증 유지

🐛 Fix

내용 파일
주문 검색 전달 상태 조건을 boolean 파라미터로 변경 FundingOrderRepository.java, AdminFundingService.java
주문·이용권 전달 상태에 FAILED 반영 AdminFundingService.java
다중 수량 발급 기준으로 테스트 갱신 AdminFundingApiIntegrationTest.java, TumblbugFundingApiIntegrationTest.java
주문 전달 상태 기준 문서화 FUNDING_API.md

📌 공유 사항

  1. 주문 목록 500 은 dev 에서 실제로 나는 버그입니다. 어드민 펀딩 주문 목록 화면이 dev 에서 열리지 않았을 수 있습니다.
  2. 발급 규칙(구매 수량만큼 발급)은 ✨ Feat: 관리자 펀딩 주문·이용권 대시보드 API 확장 #586·✨ Feat: 텀블벅 Start·Run 등록 및 전달 흐름 분리 #587 의 의도로 보고 테스트를 맞췄습니다. 의도가 다르면 알려 주세요.

✅ 체크리스트

  • Reviewer에 팀원들을 선택했나요?
  • Assignees에 본인을 선택했나요?
  • 컨벤션에 맞는 Type을 선택했나요?
  • Development에 이슈를 연동했나요?
  • Merge 하려는 브랜치가 올바르게 설정되어 있나요?
  • 컨벤션을 지키고 있나요?
  • 로컬에서 실행했을 때 에러가 발생하지 않나요?
  • 팀원들에게 PR 링크 공유를 했나요?

📸 스크린샷

  • 전체 테스트 638개 통과 (실패 0)

💬 리뷰 요구사항

  1. 주문 단위 전달 상태를 "모두 SENT 면 SENT, 하나라도 FAILED 면 FAILED, 나머지 PENDING" 으로 정했습니다. 다중 수량 주문에서 이 기준이 화면 의도와 맞는지 확인 부탁드립니다.

@si-zero si-zero added the 🐛 Fix (버그 수정) 버그를 수정할 때 사용합니다. label Nov 7, 2026
@si-zero si-zero self-assigned this Nov 7, 2026
@si-zero
si-zero merged commit 8145e85 into dev Nov 7, 2026
1 check passed
@si-zero
si-zero deleted the fix/#612 branch November 7, 2026 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Fix (버그 수정) 버그를 수정할 때 사용합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 Fix: 펀딩 다중 수량 발급 이후 깨진 펀딩 테스트와 주문 목록 500 수정

1 participant