Skip to content

test: [AI] 좌석 전송 테스트 리팩토링#343

Open
boyekim wants to merge 1 commit into
mainfrom
boyekim/TSK-80
Open

test: [AI] 좌석 전송 테스트 리팩토링#343
boyekim wants to merge 1 commit into
mainfrom
boyekim/TSK-80

Conversation

@boyekim

@boyekim boyekim commented May 25, 2026

Copy link
Copy Markdown
Member

작업 내용

  • GeneralSeatSenderTest와 PinSeatSenderTest의 Thread.sleep 기반 검증을 Awaitility 기반 비동기 검증으로 정리했습니다.
  • 전송 주기, 중복 스케줄 등록 방지, 취소 후 추가 전송 없음 검증을 테스트 의도에 맞게 보강했습니다.
  • 주석 처리된 대형 SeatIntegrationTest 시나리오를 제거해 테스트 코드 상태를 정리했습니다.

고민 지점과 리뷰 포인트

  • 취소 테스트는 스케줄 활성 상태가 false가 되는지와 다음 전송 주기 이후 propagate 호출 수가 늘지 않는지를 함께 검증합니다.
  • GeneralSeatSender의 현재 전송 주기는 3초, PinSeatSender의 현재 전송 주기는 1초 기준으로 timeout을 파생시켰습니다.
  • Google Sheets 빈은 이 테스트 범위와 무관하므로 기존 테스트 컨벤션대로 Sheets를 MockitoBean으로 대체했습니다.

검증

  • ./gradlew compileJava compileTestJava
  • ./gradlew :test --tests "kr.allcll.backend.domain.seat.GeneralSeatSenderTest" --tests "kr.allcll.backend.domain.seat.PinSeatSenderTest"

관련 이슈

@boyekim boyekim added the ai-generated Claude(AI)가 생성한 PR label May 25, 2026
@boyekim boyekim self-assigned this May 25, 2026
@auto-assign auto-assign Bot requested review from 2Jin1031, goldm0ng and haeyoon1 May 25, 2026 08:07
@github-actions

github-actions Bot commented May 25, 2026

Copy link
Copy Markdown

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit cd7eebb.

♻️ This comment has been updated with latest results.

@chemistryx chemistryx force-pushed the boyekim/TSK-80 branch 2 times, most recently from 3d99176 to cd7eebb Compare June 20, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-generated Claude(AI)가 생성한 PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

좌석 전송 테스트 리팩토링

1 participant