Skip to content

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

Open
Roy-wonji wants to merge 1 commit into
devfrom
fix/#440
Open

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

Conversation

@Roy-wonji

Copy link
Copy Markdown

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

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

πŸ› 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 κΈ°μ€€μœΌλ‘œ λ§žμΆ˜λ‹€.
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