Skip to content

πŸ› Fix: BattleVoteServiceImplTest μ‹œκ°„λŒ€ 뢈일치둜 15μ‹œ(UTC) 이후 CI μ‹€νŒ¨ - #442

Merged
Roy-wonji merged 1 commit into
devfrom
fix/#440
Sep 7, 2026
Merged

πŸ› Fix: BattleVoteServiceImplTest μ‹œκ°„λŒ€ 뢈일치둜 15μ‹œ(UTC) 이후 CI μ‹€νŒ¨#442
Roy-wonji merged 1 commit into
devfrom
fix/#440

Conversation

@Roy-wonji

Copy link
Copy Markdown
Contributor

#️⃣ μ—°κ΄€λœ 이슈

πŸ“ μž‘μ—… λ‚΄μš©

πŸ› Fix

λ‚΄μš© 파일
"였늘" νŒλ‹¨ 기쀀을 ν”„λ‘œλ•μ…˜κ³Ό λ™μΌν•˜κ²Œ KST둜 맞좀 BattleVoteServiceImplTest.java

πŸ“Œ 곡유 사항

  1. κ΄‘κ³  μž‘μ—…(✨ Feat: ad.picke.store 제휴 κ΄‘κ³ (쿠팑 νŒŒνŠΈλ„ˆμŠ€Β·μ• λ“œν”½) 연동 #441) 쀑에 CIκ°€ κΉ¨μ Έ 원인을 μΆ”μ ν•˜λ‹€ λ°œκ²¬ν•œ κΈ°μ‘΄ κ²°ν•¨μž…λ‹ˆλ‹€. κ΄‘κ³  μ½”λ“œμ™€λŠ” λ¬΄κ΄€ν•©λ‹ˆλ‹€.
  2. 같은 λΈŒλžœμΉ˜μ—μ„œ 14:42 UTC 싀행은 톡과, 15:25 UTC 싀행은 μ‹€νŒ¨ν•œ 게 결정적 λ‹¨μ„œμ˜€μŠ΅λ‹ˆλ‹€. 15:00 UTC = KST μžμ •μž…λ‹ˆλ‹€.
  3. BattleVoteServiceImpl은 LocalDate.now(KST)둜 μ˜€λŠ˜μ„ νŒλ‹¨ν•˜λŠ”λ° ν…ŒμŠ€νŠΈλŠ” μ‹œμŠ€ν…œ κΈ°λ³Έ μ‹œκ°„λŒ€μ˜ LocalDate.now()λ₯Ό μΌμŠ΅λ‹ˆλ‹€. UTC λŸ¬λ„ˆμ—μ„œ 15μ‹œκ°€ λ„˜μœΌλ©΄ ν•˜λ£¨κ°€ μ–΄κΈ‹λ‚˜ 였늘 배틀이 μ–΄μ œ λ°°ν‹€λ‘œ μ·¨κΈ‰λ˜λ©΄μ„œ ν¬λ ˆλ”§ 차감 λΆ„κΈ°κ°€ λ°˜λŒ€λ‘œ νƒ”μŠ΅λ‹ˆλ‹€.
  4. 즉 이 ν…ŒμŠ€νŠΈλŠ” 맀일 KST 00:00~09:00 사이에 CIλ₯Ό 돌리면 κΉ¨μ§€κ³  μžˆμ—ˆμŠ΅λ‹ˆλ‹€. λ‘œμ»¬μ€ KST라 항상 ν†΅κ³Όν•΄μ„œ λ“œλŸ¬λ‚˜μ§€ μ•Šμ•˜μŠ΅λ‹ˆλ‹€.
  5. ✨ Feat: ad.picke.store 제휴 κ΄‘κ³ (쿠팑 νŒŒνŠΈλ„ˆμŠ€Β·μ• λ“œν”½) 연동 #441 이 이 μˆ˜μ •μ— μ˜μ‘΄ν•©λ‹ˆλ‹€. λ¨Όμ € λ¨Έμ§€ λΆ€νƒλ“œλ¦½λ‹ˆλ‹€.

βœ… 체크리슀트

  • μ»¨λ²€μ…˜μ— λ§žλŠ” Type을 μ„ νƒν–ˆλ‚˜μš”?
  • Development에 이슈λ₯Ό μ—°λ™ν–ˆλ‚˜μš”?
  • Merge ν•˜λ €λŠ” λΈŒλžœμΉ˜κ°€ μ˜¬λ°”λ₯΄κ²Œ μ„€μ •λ˜μ–΄ μžˆλ‚˜μš”?
  • λ‘œμ»¬μ—μ„œ μ‹€ν–‰ν–ˆμ„ λ•Œ μ—λŸ¬κ°€ λ°œμƒν•˜μ§€ μ•Šλ‚˜μš”?

πŸ“Έ μŠ€ν¬λ¦°μƒ·

μž¬ν˜„ 및 검증 (ν˜„μž¬ μ‹œκ°μ΄ 15:30 UTC라 μ‹€νŒ¨ ꡬ간):

$ git stash push <ν…ŒμŠ€νŠΈνŒŒμΌ>            # μˆ˜μ • μ „
$ TZ=UTC ./gradlew test --tests "*BattleVoteServiceImplTest*"
  BattleVoteServiceImplTest > 였늘 배틀이면 졜초 사전 νˆ¬ν‘œ μ‹œ ν¬λ ˆλ”§μ„ μ°¨κ°ν•˜μ§€ μ•ŠλŠ”λ‹€ FAILED
  BattleVoteServiceImplTest > 였늘의 배틀이라도 였늘 이미 무료 μ§„μž…μ„ μ‚¬μš©ν–ˆλ‹€λ©΄ ν¬λ ˆλ”§μ„ μ°¨κ°ν•œλ‹€ FAILED
  7 tests completed, 2 failed

$ git stash pop                          # μˆ˜μ • ν›„
$ TZ=UTC ./gradlew test --tests "*BattleVoteServiceImplTest*"
  BUILD SUCCESSFUL

πŸ’¬ 리뷰 μš”κ΅¬μ‚¬ν•­

  1. 같은 νŒ¨ν„΄(LocalDate.now() ν…ŒμŠ€νŠΈ vs LocalDate.now(KST) ν”„λ‘œλ•μ…˜)이 λ‹€λ₯Έ 도메인에도 μžˆμ„ 수 μžˆμŠ΅λ‹ˆλ‹€. μ΄λ²ˆμ—” CIλ₯Ό 막은 이 파일만 κ³ μ³€λŠ”λ°, μ „μˆ˜ 점검을 별도 이슈둜 λΊ„μ§€ 의견 μ£Όμ„Έμš”.

BattleVoteServiceImpl은 "였늘"을 LocalDate.now(KST)둜 νŒλ‹¨ν•˜λŠ”λ°
ν…ŒμŠ€νŠΈλŠ” μ‹œμŠ€ν…œ κΈ°λ³Έ μ‹œκ°„λŒ€μ˜ LocalDate.now()λ₯Ό 썼닀.

UTC λŸ¬λ„ˆμ—μ„œλŠ” 15:00 UTC(=KST μžμ •) μ΄ν›„λ‘œ 두 값이 ν•˜λ£¨μ”© μ–΄κΈ‹λ‚˜
였늘 배틀이 μ–΄μ œ λ°°ν‹€λ‘œ μ·¨κΈ‰λ˜λ©΄μ„œ ν¬λ ˆλ”§ 차감 λΆ„κΈ°κ°€ λ°˜λŒ€λ‘œ 탔닀.
같은 λΈŒλžœμΉ˜μ—μ„œ 14:42 UTC 싀행은 ν†΅κ³Όν•˜κ³  15:25 UTC 싀행은 μ‹€νŒ¨ν–ˆλ‹€.

ν…ŒμŠ€νŠΈλ„ KST κΈ°μ€€μœΌλ‘œ λ§žμΆ˜λ‹€.

@si-zero si-zero left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UTC λŸ¬λ„ˆμ—μ„œ 15μ‹œ μ΄ν›„λ‘œ ν•˜λ£¨ λ°€λ¦¬λ˜ κ±° 원인 μ •ν™•ν•˜κ²Œ μž‘μ•˜μŒ. ν”„λ‘œλ•μ…˜μ΄ LocalDate.now(KST)둜 "였늘"을 νŒλ‹¨ν•˜λ‹ˆκΉŒ ν…ŒμŠ€νŠΈλ„ today()(KST)둜 맞좘 게 λ§žλŠ” λ°©ν–₯이고, μŠ€ν… μΈμžκΉŒμ§€ 같이 λ°”κΏ”μ„œ 일관성 있음. μ΄λŒ€λ‘œ 머지해도 됨.

ν•œ κ°€μ§€λ§Œ: μ§€κΈˆμ€ 이 ν…ŒμŠ€νŠΈ 클래슀만 KST 헬퍼λ₯Ό λ“€κ³  μžˆλŠ”λ°, μ‹œκ°„λŒ€ μ˜μ‘΄ν•˜λŠ” ν…ŒμŠ€νŠΈκ°€ 더 있으면 맀번 같은 헬퍼λ₯Ό λ³΅λΆ™ν•˜κ²Œ 됨. λ‚˜μ€‘μ— μ—¬μœ λ  λ•Œ κ³ μ • Clock μ£Όμž…μ΄λ‚˜ 곡용 ν…ŒμŠ€νŠΈ μœ ν‹Έλ‘œ λΉΌλŠ” κ±Έ 고렀해보면 μ’‹κ² μŒ. 이 PRμ—μ„œ ν•  건 μ•„λ‹˜.

@Roy-wonji
Roy-wonji merged commit 7b0b68f into dev Sep 7, 2026
1 check passed
@Roy-wonji
Roy-wonji deleted the fix/#440 branch September 7, 2026 07:04
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.

2 participants