Skip to content

[REFACTOR] 인증 영역의 잔존 코드와 주입 컨벤션 정리 - #188

Merged
sun007021 merged 3 commits into
developfrom
refactor/187
Aug 6, 2026
Merged

[REFACTOR] 인증 영역의 잔존 코드와 주입 컨벤션 정리#188
sun007021 merged 3 commits into
developfrom
refactor/187

Conversation

@poketopa

@poketopa poketopa commented Aug 4, 2026

Copy link
Copy Markdown
Member

관련 이슈

Closes #187

변경 배경

인증 영역에 실행되지 않는 주석 코드와 테스트 디버그 출력이 남아 있었고, AuthService만 프로젝트의 다른 서비스와 달리 생성자를 직접 선언하고 있었다.

기존 인증 동작과 API 계약을 유지하면서 현재 프로젝트에서 반복적으로 사용하는 방식으로 정리

변경 내용

  • 완전히 주석 처리되어 사용되지 않는 TokenManager 제거
  • AuthServiceTestSystem.out.println 두 곳 제거
  • AuthService 생성자 주입을 @RequiredArgsConstructor로 통일
  • 기존 공개 생성자 시그니처 (JwtTokenProvider, AdminRepository) 유지 확인

검증

  • Java 17에서 AuthServiceTest 통과
  • Java 17에서 전체 테스트 강제 재실행
  • 103 tests / 0 failures / 0 errors / 0 skipped
  • GitHub Actions backend-ci.yml 테스트 통과
  • 별도 코드 리뷰 APPROVED, 지적 0건
  • 변경 동작 확인

@poketopa
poketopa marked this pull request as ready for review August 4, 2026 22:46
@sun007021
sun007021 merged commit 2feb357 into develop Aug 6, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants