Skip to content

test: 팀원 모집 지원 동시성 MySQL 회귀 테스트 추가 - #2393

Merged
taejinn merged 1 commit into
developfrom
test/2380-team-recruitment-role-concurrency-test
Sep 2, 2026
Merged

test: 팀원 모집 지원 동시성 MySQL 회귀 테스트 추가#2393
taejinn merged 1 commit into
developfrom
test/2380-team-recruitment-role-concurrency-test

Conversation

@taejinn

@taejinn taejinn commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

🔍 개요

  • 팀원 모집 지원·승인의 핵심 동시성 불변식을 실제 MySQL과 service transaction 경계에서 고정합니다.

🚀 주요 변경 내용

  • 같은 사용자의 동시 지원 결과를 성공 1건과 TEAM_RECRUITMENT_APPLICATION_DUPLICATE 1건으로 검증합니다.
  • 동시 지원 후 지원서·알림·outbox·채팅 멤버가 각각 1건인지 검증합니다.
  • 마지막 한 자리 동시 승인 결과를 한 명 ACCEPTED, 경쟁 지원서 REJECTED로 검증합니다.
  • 경쟁 요청의 TEAM_RECRUITMENT_CLOSED 결과를 검증합니다.
  • 모집·역할 정원, TEAM 멤버, notification/outbox의 최종 연결과 개수를 검증합니다.

💬 참고 사항

  • 영향
    • 심각도는 P2 동시성 회귀 방지이며 불변식 회귀 시 P1 데이터 오류로 이어집니다.
    • 변경 파일은 TeamRecruitmentApplicationConcurrencyTest 1개이며 지원·승인 충돌 응답 계약을 고정합니다.
  • 검증
    • latch 동시 시작 → service transaction → MySQL lock → JDBC 최종 상태 경로의 동시성 테스트 2개가 반복 통과했습니다.
    • CI build와 Unit Test Results가 통과했습니다.
  • 관련 PR
    • ROLE_BASED 순차 HTTP 여정은 #2406에서 다룹니다.

✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

@taejinn taejinn added 공통 백엔드 공통으로 작업할 이슈입니다. 테스트 labels Aug 31, 2026
@coderabbitai

coderabbitai Bot commented Aug 31, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 50 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 25e4ea5a-0639-44c3-982d-747fbea2e6ea

📥 Commits

Reviewing files that changed from the base of the PR and between 814d457 and b953ad5.

📒 Files selected for processing (1)
  • src/test/java/in/koreatech/koin/acceptance/domain/TeamRecruitmentApplicationConcurrencyTest.java

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown

Unit Test Results

1 139 tests   1 136 ✔️  2m 35s ⏱️
   258 suites         3 💤
   258 files           0

Results for commit b953ad5.

♻️ This comment has been updated with latest results.

@taejinn
taejinn force-pushed the test/2380-team-recruitment-role-concurrency-test branch from f895d7b to 207c87a Compare September 2, 2026 11:27
@taejinn taejinn changed the title test: 팀원 모집 ROLE_BASED와 지원 동시성 회귀 테스트 추가 test: 팀원 모집 지원 동시성 MySQL 회귀 테스트 추가 Sep 2, 2026
@taejinn
taejinn marked this pull request as ready for review September 2, 2026 12:58
@taejinn
taejinn requested review from dnjswldnd-3513 and insik03 and removed request for Soundbar91 and dh2906 September 2, 2026 13:03
@taejinn
taejinn force-pushed the test/2380-team-recruitment-role-concurrency-test branch from 207c87a to b953ad5 Compare September 2, 2026 16:33
@taejinn
taejinn merged commit f19c254 into develop Sep 2, 2026
6 checks passed
@taejinn
taejinn deleted the test/2380-team-recruitment-role-concurrency-test branch September 2, 2026 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

공통 백엔드 공통으로 작업할 이슈입니다. 테스트

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[공통] 팀원 모집 지원·승인 MySQL 동시성 회귀 테스트 보강

2 participants