Skip to content

test: 팀원 모집 ROLE_BASED HTTP 인수 테스트 추가 - #2406

Merged
taejinn merged 1 commit into
developfrom
test/2402-team-recruitment-role-flow-http-test
Sep 2, 2026
Merged

test: 팀원 모집 ROLE_BASED HTTP 인수 테스트 추가#2406
taejinn merged 1 commit into
developfrom
test/2402-team-recruitment-role-flow-http-test

Conversation

@taejinn

@taejinn taejinn commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

🔍 개요

  • ROLE_BASED 모집 생성부터 지원·거절·재지원·승인까지 실제 클라이언트 HTTP 여정을 고정합니다.

🚀 주요 변경 내용

  • 생성 응답의 실제 recruitment idtype=ROLE_BASED를 검증합니다.
  • 배열 순서 대신 실제 role_id로 역할을 조회하고 지원합니다.
  • 첫 지원 거절 후 다른 사용자의 재지원과 최종 승인을 검증합니다.
  • 상세 응답의 역할별 정원과 is_closed를 검증합니다.
  • TEAM 방과 승인 멤버 연결을 검증합니다.

💬 참고 사항

  • 영향
    • 심각도는 P2 ROLE_BASED 회귀 방지이며 변경 파일은 TeamRecruitmentApplicationFlowApiTest입니다.
    • Spring MVC → 인증 → service → MySQL 경계에서 응답 id와 선택한 role_id로 이어지는 HTTP 계약을 고정합니다.
  • 검증
    • 기존 flow를 포함한 테스트 12개와 CI build, Unit Test Results가 통과했습니다.
  • 관련 PR
    • 지원·승인 동시성 테스트는 #2393에서 다룹니다.

✅ Checklist (완료 조건)

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

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

coderabbitai Bot commented Sep 2, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 30 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: 24857824-4c9a-45e8-b659-2af7ed4d46be

📥 Commits

Reviewing files that changed from the base of the PR and between 67422e6 and 01720d2.

📒 Files selected for processing (1)
  • src/test/java/in/koreatech/koin/acceptance/domain/TeamRecruitmentApplicationFlowApiTest.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 Sep 2, 2026

Copy link
Copy Markdown

Unit Test Results

1 141 tests   1 138 ✔️  6m 39s ⏱️
   259 suites         3 💤
   259 files           0

Results for commit 01720d2.

♻️ This comment has been updated with latest results.

@taejinn
taejinn force-pushed the test/2402-team-recruitment-role-flow-http-test branch from 3313144 to 01720d2 Compare September 2, 2026 16:52
@taejinn
taejinn merged commit 3175139 into develop Sep 2, 2026
6 checks passed
@taejinn
taejinn deleted the test/2402-team-recruitment-role-flow-http-test branch September 2, 2026 17:13
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.

[공통] 팀원 모집 ROLE_BASED HTTP 여정 인수 테스트 보강

2 participants