이진용 sprint7

[alpha-lens]

요구사항

기본 요구사항

프로파일 기반 설정 관리

• 개발, 운영 환경에 대한 프로파일을 구성하세요.
• application-dev.yaml, application-prod.yaml 파일을 생성하세요.
• 다음과 같은 설정값을 프로파일별로 분리하세요.
• 데이터베이스 연결 정보
• 서버 포트

로그 관리

• Lombok의 @slf4j 어노테이션을 활용해 로깅을 쉽게 추가할 수 있도록 구성하세요.
• application.yaml에 기본 로깅 레벨을 설정하세요. 기본적으로 info 레벨로 설정합니다.
• 환경 별 적절한 로깅 레벨을 프로파일 별로 설정해보세요.
  • SQL 로그를 보기위해 설정했던 레벨은 유지합니다.
  • 우리가 작성한 프로젝트의 로그는 개발 환경에서 debug, 운영 환경에서는 info 레벨로 설정합니다.
• Spring Boot의 기본 로깅 구현체인 Logback의 설정 파일을 구성하세요.
• logback-spring.xml 파일을 생성하세요.
• 다음 예시와 같은 로그 메시지를 출력하기 위한 로깅 패턴과 출력 방식을 커스터마이징하세요.
  • 출력 예시

# 패턴
{년}-{월}-{일} {시}:{분}:{초}:{밀리초} [{스레드명}] {로그 레벨(5글자로 맞춤)} {로거 이름(최대 36글자)} - {로그 메시지}{줄바꿈}

# 예시
25-01-01 10:33:55.740 [main] DEBUG c.s.m.discodeit.DiscodeitApplication - Running with Spring Boot v3.4.0, Spring v6.2.0

• 콘솔과 파일에 동시에 로그를 기록하도록 설정하세요.
• 파일은 {프로젝트 루트}/.logs 경로에 저장되도록 설정하세요.
• 로그 파일은 일자별로 롤링되도록 구성하세요.
• 로그 파일은 30일간 보관하도록 구성하세요.
• 서비스 레이어와 컨트롤러 레이어의 주요 메소드에 로깅을 추가하세요.
  • 로깅 레벨을 적절히 사용하세요: ERROR, WARN, INFO, DEBUG
  • 다음과 같은 메소드에 로깅을 추가하세요:
    • 사용자 생성/수정/삭제
    • 채널 생성/수정/삭제
    • 메시지 생성/수정/삭제
    • 파일 업로드/다운로드

예외 처리 고도화

• 커스텀 예외를 설계하고 구현하세요.
  • 패키지명: com.sprint.mission.discodeit.exception[.{도메인}]
• ErrorCode Enum 클래스를 통해 예외 코드명과 메시지를 정의하세요.
• 모든 예외의 기본이 되는 DiscodeitException 클래스를 정의하세요.
• DiscodeitException을 상속하는 주요 도메인 별 메인 예외 클래스를 정의하세요.
• 도메인 메인 예외 클래스를 상속하는 구체적인 예외 클래스를 정의하세요.
• 기존에 구현했던 예외를 커스텀 예외로 대체하세요.
• ErrorResponse를 통해 일관된 예외 응답을 정의하세요.
• 앞서 정의한 ErrorResponse와 @RestControllerAdvice를 활용해 예외를 처리하는 예외 핸들러를 구현하세요.

유효성 검사

• Spring Validation 의존성을 추가하세요.
• 주요 Request DTO에 제약 조건 관련 어노테이션을 추구하세요.
  • @NotNull, @notblank, @SiZe, @Email 등
• 컨트롤러에 @Valid 를 사용해 요청 데이터를 검증하세요.
• 검증 실패 시 발생하는 MethodArgumentNotValidException을 전역 예외 핸들러에서 처리하세요.
• 유효성 검증 실패 시 상세한 오류 메시지를 포함한 응답을 반환하세요.

Actuator

• Spring Boot Actuator 의존성을 추가하세요.
• 기본 Actuator 엔트포인트를 설정하세요.
  • health, info, metrics, loggers
• Actuator info를 위한 애플리케이션 정보를 추가하세요.
  • 애플리케이션 이름: Discodeit
  • 애플리케이션 버전: 1.7.0
  • 자바 버전: 17
  • 스프링 부트 버전: 3.4.0
  • 주요 설정 정보
    • 데이터소스: url, 드라이버 클래스 이름
    • jpa: ddl-auto
    • storage 설정: type, path
    • multipart 설정: max-file-size, max-request-size
• Spring Boot 서버를 실행 후 각종 정보를 확인해보세요.
  • /actuator/info
  • /actuator/metrics
  • /actuator/health
  • /actuator/loggers

단위 테스트

• 서비스 레이어의 주요 메소드에 대한 단위 테스트를 작성하세요.
• 다음 서비스의 핵심 메소드에 대해 각각 최소 2개 이상(성공, 실패)의 테스트 케이스를 작성하세요.
• UserService: create, update, delete 메소드
• ChannelService: create(PUBLIC, PRIVATE), update, delete, findByUserId 메소드
• MessageService: create, update, delete, findByChannelId 메소드
• Mockito를 활용해 Repository 의존성을 모의(mock)하세요.
• BDDMockito를 활용해 테스트 가독성을 높이세요.

슬라이스 테스트

• 레포지토리 레이어의 슬라이스 테스트를 작성하세요.
  • @DataJpaTest를 활용해 테스트를 구현하세요.
  • 테스트 환경을 구성하는 프로파일을 구성하세요.
    • application-test.yaml을 생성하세요.
    • 데이터소스는 H2 인메모리 데이터 베이스를 사용하고, PostgreSQL 호환 모드로 설정하세요.
    • H2 데이터베이스를 위해 필요한 의존성을 추가하세요.
    • 테스트 시작 시 스키마를 새로 생성하도록 설정하세요.
    • 디버깅에 용이하도록 로그 레벨을 적절히 설정하세요.
  • 테스트 실행 간 test 프로파일을 활성화 하세요.
  • JPA Audit 기능을 활성화 하기 위해 테스트 클래스에 @EnableJpaAuditing을 추가하세요.
  • 주요 레포지토리(User, Channel, Message)의 주요 쿼리 메소드에 대해 각각 최소 2개 이상(성공, 실패)의 테스트 케이스를 작성하세요.
    • 커스텀 쿼리 메소드
    • 페이징 및 정렬 메소드
  • 컨트롤러 레이어의 슬라이스 테스트를 작성하세요.
    • @WebMvcTest를 활용해 테스트를 구현하세요.
    • WebMvcTest에서 자동으로 등록되지 않는 유형의 Bean이 필요하다면 @import를 활용해 추가하세요.
  • 주요 컨트롤러(User, Channel, Message)에 대해 최소 2개 이상(성공, 실패)의 테스트 케이스를 작성하세요.
  • MockMvc를 활용해 컨트롤러를 테스트하세요.
  • 서비스 레이어를 모의(mock)하여 컨트롤러 로직만 테스트하세요.
  • JSON 응답을 검증하는 테스트를 포함하세요.

통합 테스트

• 통합 테스트 환경을 구성하세요.
  • @SpringBootTest를 활용해 Spring 애플리케이션 컨텍스트를 로드하세요.
  • H2 인메모리 데이터베이스를 활용하세요.
  • 테스트용 프로파일을 구성하세요.
• 주요 API 엔드포인트에 대한 통합 테스트를 작성하세요.
  • 주요 API에 대해 최소 2개 이상의 테스트 케이스를 작성하세요.
    • 사용자 관련 API (생성, 수정, 삭제, 목록 조회)
    • 채널 관련 API (생성, 수정, 삭제)
    • 메시지 관련 API (생성, 수정, 삭제, 목록 조회)
  • 각 테스트는 @transactional을 활용해 독립적으로 실행하세요.

심화 요구사항

MDC를 활용한 로깅 고도화

-x[] 요청 ID, 요청 URL, 요청 방식 등의 정보를 MDC에 추가하는 인터셉터를 구현하세요.
- [x] 클래스명: MDCLoggingInterceptor
- [x] 패키지명: com.**.discodeit.config
- [x] 요청 ID는 랜덤한 문자열로 생성합니다. (UUID)
- [x] 요청 ID는 응답 헤더에 포함시켜 더 많은 분석이 가능하도록 합니다.
- 헤더 이름: Discodeit-Request-ID

• WebMvcConfigurer를 통해 MDCLoggingInterceptor를 등록하세요.
  • 클래스명: WebMvcConfig
  • 패키지명: com.**.discodeit.config
• Logback 패턴에 MDC 값을 포함시키세요.

Spring Boot Admin을 활용한 메트릭 가시화

• Spring Boot Admin 서버를 구현할 모듈을 생성하세요.
• admin 모듈의 메인 클래스에 @EnableAdminServer 어노테이션을 추가하고, 서버는 9090번 포트로 설정합니다.
• admin 서버 실행 후 [localhost:9090/applications](http://localhost:9090/applications) 에 접속해봅니다.
• discodeit 프로젝트에 Spring Boot Admin Client를 적용합니다.
  • 의존성을 추가합니다.
  • admin 서버에 등록될 수 있도록 설정 정보를 추가합니다.
  • discodeit 서버를 실행하고, admin 대시보드에 discodeit 인스턴스가 추가되었는지 확인합니다.
• admin 대시보드 화면을 조작해보면서 각종 메트릭 정보를 확인해보세요.

테스트 커버리지 관리

• JaCoCo 플러그인을 추가하세요.
• 테스트 실행 후 생성된 리포트를 분석해보세요.
• com.sprint.mission.discodeit.service.basic 패키지에 대해서 60% 이상의 코드 커버리지를 달성하세요.

주요 변경사항

멘토에게 @joonfluence

• 어제 테스트 커버리지에 대해서 이야기해주셨는데, 오늘 진행하다보니 분기문에 대한 커버리지 내용도 확인했습니다. 이러한 부분까지 100%를 충족하는 것이 TDD에 부합하는 것인지가 궁금합니다. (찾아본 바, 수치에 목매이지 말아야 한다는 내용도 있었는데, 이 부분은 아직 잘 모르겠습니다.)
  • 이에 더하여, 지금은 60%를 맞추는 것에 목표로 두고 진행했는데, 보통 어느정도를 충족하는지도 궁금합니다. (지난 멘토링 때는 명세서 역할을 한다고 해주신 것 같은데, 그럼 '주요 포인트'에서만 Test Code를 작성해도 되는건지도 궁금합니다.)
• AI를 주로 사용한다고 하지 않는다는 가정 하에 (물론, 앞으로 더욱 그럴 일은 없겠지만) 테스트 코드 작성에 들어가는 시간과 실제 코드 로직을 작성하는데에 걸리는 시간의 비율이 궁금합니다.
  • 위에 더하여, AI에게 테스트코드 작성을 어느정도까지 맡기는지도 궁금합니다.

[joonfluence]

전체 요약

프로파일 기반 설정 분리, 로깅(MDC 포함), 커스텀 예외 체계, 유효성 검사, Actuator, Spring Boot Admin, 단위/슬라이스/통합 테스트, JaCoCo 커버리지 등 Sprint 7의 요구사항을 전반적으로 잘 구현했습니다. 예외 계층 구조와 테스트 코드의 가독성이 좋습니다.

리뷰 코멘트 요약:

• P2: 3건 (UUID/Instant에 @notblank 사용, ErrorResponse 필드명 오타, NPE 위험)
• P3: 4건 (prod 비밀번호 하드코딩, update 중복검사 자기자신 포함, null 대신 빈 리스트 사용, 로그 파일 경로)
• P4: 1건 (admin 모듈 client 의존성)
• P5: 1건 (로그 메시지 오타)

P2가 있어 REQUEST_CHANGES로 제출합니다.

추가 참고: TestJpaConfig가 src/main/java에 위치해 있습니다. 프로덕션 빌드에 포함되므로, 프로덕션용 JPA Audit 설정 클래스와 분리하거나 @WebMvcTest에서 exclude하는 방식을 고려해주세요.

[alpha-lens]

@joonfluence
멘토님께서 말씀해주신 부분, 교정하였습니다.
특히 P2에서 UUID에 NotBlank를 쓰고는 오류가 난 것을 확인해놓고 고치지 못한 것을 발견할 수 있었던 점,
P3에서 AndIdNot 부분에 대해서는 전혀 모르고 있었던 터라 많은 공부가 되었습니다. 감사합니다.