Skip to content

test: MeetingUploadService 핵심 시나리오 단위 테스트 추가 - #41

Open
c1oud-dev wants to merge 1 commit into
mainfrom
test/phase5/10-2
Open

test: MeetingUploadService 핵심 시나리오 단위 테스트 추가#41
c1oud-dev wants to merge 1 commit into
mainfrom
test/phase5/10-2

Conversation

@c1oud-dev

Copy link
Copy Markdown
Owner

작업 내용

변경 사항

MeetingUploadServiceTest 신규 작성

  • 허용되지 않는 파일 형식 업로드 시 예외 검증
  • 팀 소속이 아닌 사용자의 팀 회의 업로드 시 예외 검증
  • 파일 크기가 플랜 최대 허용치를 초과할 때 예외 검증 (실제 PlanPolicy 반환값 기준)
  • 월간 업로드 횟수가 플랜 한도에 도달했을 때 예외 검증 (실제 PlanPolicy 반환값 기준)
  • 정상 업로드 시 UsageRecord가 올바른 usageType/파일 크기로 저장되는지 검증

기타

  • MultipartFile은 Mockito.mock으로 생성, User는 실제 정적 팩토리(User.create) 사용
  • 사용량 기록 검증 시나리오는 TransactionSynchronizationManager를 초기화/정리하여 afterCommit 등록 과정에서 예외 없이 통과하도록 처리 (afterCommit 콜백 자체는 이번 범위에서 검증하지 않음)
  • afterCommit 비동기 처리 등록 자체의 검증은 별도 통합 테스트가 필요한 영역으로 판단, 이번 단위 테스트 범위에서 제외

참고

  • afterCommit 트리거 검증(비동기 회의 처리 요청)은 @SpringBootTest 기반 통합 테스트가 필요. 별도 pr로 분리.

@c1oud-dev c1oud-dev self-assigned this Aug 3, 2026
@c1oud-dev c1oud-dev added the test 테스트 코드 label Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test 테스트 코드

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant