Skip to content

test: 팀원 모집 알림 목록과 일괄 처리 HTTP 인수 테스트 추가 - #2405

Merged
taejinn merged 1 commit into
developfrom
test/2401-team-recruitment-notification-http-test
Sep 2, 2026
Merged

test: 팀원 모집 알림 목록과 일괄 처리 HTTP 인수 테스트 추가#2405
taejinn merged 1 commit into
developfrom
test/2401-team-recruitment-notification-http-test

Conversation

@taejinn

@taejinn taejinn commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

🔍 개요

  • 팀원 모집 알림 목록과 일괄 명령의 사용자 격리·페이지네이션·멱등 동작을 실제 HTTP와 DB 경계에서 고정합니다.

🚀 주요 변경 내용

  • 사용자별 알림 격리와 page/limit pagination을 검증합니다.
  • 전체 읽음 반복 호출의 204와 최초 readAt 보존을 검증합니다.
  • 전체 삭제 반복 호출의 204와 soft delete를 검증합니다.
  • 다른 사용자의 unread·삭제 상태 보존을 검증합니다.

💬 참고 사항

  • 영향
    • 심각도는 P2 알림 API 회귀 방지이며 변경 파일은 TeamRecruitmentNotificationApiTest 1개입니다.
    • Spring MVC → 인증 → service → MySQL 경계에서 재시도 가능한 일괄 204와 recipient별 목록 계약을 고정합니다.
  • 검증
    • 인수 테스트 1개에서 목록·전체 읽음·전체 삭제 시나리오를 확인했으며 CI build와 Unit Test Results가 통과했습니다.
  • 관련 PR
    • TEAM 채팅 notification/outbox 테스트는 #2392에서 다룹니다.

✅ 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 40 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: 3a68343f-d984-4a77-af49-63f75dc2ba0c

📥 Commits

Reviewing files that changed from the base of the PR and between f19c254 and 86c1da4.

📒 Files selected for processing (1)
  • src/test/java/in/koreatech/koin/acceptance/domain/TeamRecruitmentNotificationApiTest.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 140 tests   1 137 ✔️  2m 45s ⏱️
   258 suites         3 💤
   258 files           0

Results for commit 86c1da4.

♻️ This comment has been updated with latest results.

@taejinn
taejinn force-pushed the test/2401-team-recruitment-notification-http-test branch from f53f40b to 86c1da4 Compare September 2, 2026 16:42
@taejinn
taejinn merged commit 67422e6 into develop Sep 2, 2026
6 checks passed
@taejinn
taejinn deleted the test/2401-team-recruitment-notification-http-test branch September 2, 2026 16:51
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.

[공통] 팀원 모집 알림 목록 및 일괄 처리 HTTP 인수 테스트 보강

2 participants