Skip to content

merge: (#933) 테스트 커버리지 향상을 위한 단위/통합 테스트 추가 및 CI 워크플로우 변경#956

Merged
ryujaemin08 merged 43 commits intodevelopfrom
refactor/933-test-coverage
Jan 19, 2026
Merged

merge: (#933) 테스트 커버리지 향상을 위한 단위/통합 테스트 추가 및 CI 워크플로우 변경#956
ryujaemin08 merged 43 commits intodevelopfrom
refactor/933-test-coverage

Conversation

@ryujaemin08
Copy link
Member

@ryujaemin08 ryujaemin08 commented Jan 13, 2026

작업 내용 설명

  • notification 도메인 모델 단위 테스트 추가 (NotificationOfUser, TopicSubscription 등)
  • notification core 계층 Service 단위 테스트 추가 및 개선
  • notification core 계층 UseCase 단위 테스트 추가
  • notification persistence 어댑터 통합 테스트 추가
  • notification 스케줄러 및 보안 서비스 테스트 추가
  • infrastructure/persistence 계층 MockK 단위테스트 제거 (테스트 컨테이너를 이용한 통합 테스트로 대체)
  • auth 및 volunteer 단위 테스트 검증 로직 개선
  • CI 워크플로우를 각각의 마이크로 서비스에 병렬로 ci 를 실행하고, JacocoReport 를 수집하는 방식으로 변환함.

주요 변경 사항

  • notification 관련 전체 도메인 모델, 서비스, 유스케이스 테스트 추가
  • 계층별 테스트 전략 적용: Core 계층은 MockK 단위 테스트, Persistence/Infrastructure 계층은 통합 테스트
  • 불필요한 slot 검증을 shouldNotThrowAny로 개선
  • QueryAndReadMyNotificationsUseCaseTest 삭제 (중복 테스트)
  • 기존 전체 서버 테스트 커버리지 1~2% -> 주요 로직 위주로 테스트 작성하여 24~25% 로 올림.
  • 테스트 ci 속도 7m -> 4m 13s 로 향상

결과물

체크리스트

  • 어플리케이션 구동(혹은 테스트)시 오류는 없나요?
  • 생성된 코드에 Javadoc 주석을 추가 하였나요?
  • 생성된 코드에 대한 테스트 코드가 작성 되었나요?

관련 이슈

@ryujaemin08 ryujaemin08 added the refactor 코드 리팩토링 할 경우 label Jan 13, 2026
@ryujaemin08 ryujaemin08 requested a review from ilyoil2 as a code owner January 13, 2026 02:33
@ryujaemin08 ryujaemin08 self-assigned this Jan 13, 2026
@ryujaemin08 ryujaemin08 requested a review from 4mjeo as a code owner January 13, 2026 02:33
@ryujaemin08 ryujaemin08 added the refactor 코드 리팩토링 할 경우 label Jan 13, 2026
@ryujaemin08 ryujaemin08 force-pushed the refactor/933-test-coverage branch 2 times, most recently from da455c8 to 2deafe1 Compare January 13, 2026 06:13
@ryujaemin08 ryujaemin08 force-pushed the refactor/933-test-coverage branch from fe3eeef to ba05587 Compare January 14, 2026 04:54
@ryujaemin08 ryujaemin08 force-pushed the refactor/933-test-coverage branch from 5731f03 to c62e6fc Compare January 14, 2026 06:00
@ryujaemin08 ryujaemin08 changed the title merge: (#933) 테스트 커버리지 향상을 위한 단위 테스트 추가 및 리팩토링 merge: (#933) 테스트 커버리지 향상을 위한 단위/통합 테스트 추가 및 리팩토링 Jan 14, 2026
@github-actions
Copy link

📊 Main Service Coverage Report

Overall Project 24.52% 🍏
Files changed 88.24% 🍏

File Coverage
VotingTopic.kt 59.18% 🍏
CreateVotingTopicUseCase.kt 24.03% -1.55%
GetVolunteerServiceImpl.kt 15.56% 🍏

@github-actions
Copy link

📊 Notification Service Coverage Report

Overall Project 25.27% -0.02% 🍏
Files changed 0%

File Coverage
NotificationOfUserPersistenceAdapter.kt 0% -0.68%
FCMAdapter.kt 0% 🍏

@github-actions
Copy link

📊 Gateway Service Coverage Report

Overall Project 0% 🍏

There is no coverage information present for the Files changed

@team-aliens team-aliens deleted a comment from github-actions bot Jan 14, 2026
@ryujaemin08 ryujaemin08 merged commit f1330e6 into develop Jan 19, 2026
5 checks passed
@ryujaemin08 ryujaemin08 deleted the refactor/933-test-coverage branch January 19, 2026 00:13
@ryujaemin08 ryujaemin08 changed the title merge: (#933) 테스트 커버리지 향상을 위한 단위/통합 테스트 추가 및 리팩토링 merge: (#933) 테스트 커버리지 향상을 위한 단위/통합 테스트 추가 및 CI 워크플로우 변경 Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor 코드 리팩토링 할 경우

Projects

None yet

Development

Successfully merging this pull request may close these issues.

테스트 커버리지(Test Coverage) 비율 향상을 위한 리펙토링

2 participants