Skip to content

응모 가능 기간 검증 및 합성 테스트 데이터 보완 - #19

Merged
masondev1024 merged 1 commit into
devfrom
fix/raffle-open-window
Sep 27, 2026
Merged

masondev1024 merged 1 commit into
devfrom
fix/raffle-open-window

Conversation

@masondev1024

Copy link
Copy Markdown
Owner

변경 내용

  • 응모 transaction에서 상품의 마감 시간·추첨 완료 여부를 DB 시각으로 검사합니다.
  • 대상 상품을 잠근 상태에서 entry와 outbox를 함께 기록하고, 마감·추첨 완료·없는 상품은 기록 없이 거부합니다.
  • 명시적으로 활성화한 합성 seed는 실행 시점부터 하루 뒤의 마감 시간을 사용합니다. 기존 상품은 재실행해도 다시 열지 않습니다.

변경 이유

실제 서비스 검증 준비 중 샘플 상품의 날짜가 지났지만 API는 여전히 승인하는 문제를 확인했습니다. 정상 응모 검증과 마감 상품 거부 검증을 구분하기 위한 수정입니다.

데이터·실행 영향

  • 기존 상품의 마감 시각이나 데이터를 수정하지 않습니다. production seed는 여전히 비활성화가 기본입니다.
  • 같은 상품의 응모는 짧은 DB row lock을 공유합니다. 추첨과 응모의 일관성을 우선하며 고부하 write 성능은 별도 검증 대상입니다.
  • CI 통과 후 dev에 반영하고 기존 main 릴리스 PR #17에 포함합니다.

검증

  • 수정 전: 신규 경계 검사 3곳 실패(검증 없는 승인 및 과거 seed 날짜) 확인.
  • 수정 후: Python 3.12 전체 653 tests passed, skip 0. 원격 CI/보안 검사는 이 PR의 checks에서 확인합니다.
  • 실환경에서는 정상 승인과 마감·추첨 완료 상품 거부, entry/outbox 미생성을 별도로 확인합니다.

되돌리기

  • 코드 revert는 dev → main PR로 수행합니다. 이 변경에는 기존 데이터 수정이나 파괴적 DDL이 없습니다.

@masondev1024
masondev1024 merged commit 11e1659 into dev Sep 27, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant