Skip to content

[조백선] sprint7 #211

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: 조백선
Choose a base branch
from

Conversation

bs8841
Copy link
Collaborator

@bs8841 bs8841 commented Jul 2, 2025

요구사항

기본

프로파일 기반 설정 관리

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

로그 관리

  • Lombok의 @slf4j 어노테이션을 활용해 로깅을 쉽게 추가할 수 있도록 구성하세요.
  • application.yaml에 기본 로깅 레벨을 설정하세요. 기본적으로 info 레벨로 설정합니다.
  • 환경 별 적절한 로깅 레벨을 프로파일 별로 설정해보세요.
  • SQL 로그를 보기위해 설정했던 레벨은 유지합니다.
  • 우리가 작성한 프로젝트의 로그는 개발 환경에서 debug, 운영 환경에서는 info 레벨로 설정합니다.
  • Spring Boot의 기본 로깅 구현체인 Logback의 설정 파일을 구성하세요.
  • logback-spring.xml 파일을 생성하세요.
  • 다음 예시와 같은 로그 메시지를 출력하기 위한 로깅 패턴과 출력 방식을 커스터마이징하세요.
  • 콘솔과 파일에 동시에 로그를 기록하도록 설정하세요.
  • 파일은 {프로젝트 루트}/.logs 경로에 저장되도록 설정하세요.
  • 로그 파일은 일자별로 롤링되도록 구성하세요.
  • 로그 파일은 30일간 보관하도록 구성하세요.
  • 서비스 레이어와 컨트롤러 레이어의 주요 메소드에 로깅을 추가하세요.
  • 로깅 레벨을 적절히 사용하세요: ERROR, WARN, INFO, DEBUG
  • 다음과 같은 메소드에 로깅을 추가하세요:
  • 사용자 생성/수정/삭제
  • 채널 생성/수정/삭제
  • 메시지 생성/수정/삭제
  • 파일 업로드/다운로드

예외 처리 고도화

  • 커스텀 예외를 설계하고 구현하세요.
  • 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를 위한 애플리케이션 정보를 추가하세요.
  • 주요 설정 정보 (datasource, jpa, storage, multipart)를 info에 포함하세요.
  • Spring Boot 서버를 실행 후 각종 정보를 확인해보세요. (/actuator/info 등)

단위 테스트

  • 서비스 레이어의 주요 메소드에 대한 단위 테스트를 작성하세요.
  • 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 (생성, 수정, 삭제, 목록 조회)
  • 채널 관련 API (생성, 수정, 삭제)
  • 메시지 관련 API (생성, 수정, 삭제, 목록 조회)
  • 각 테스트는 @transactional을 활용해 독립적으로 실행하세요.

심화

MDC를 활용한 로깅 고도화

  • 요청 ID, 요청 URL, 요청 방식 등의 정보를 MDC에 추가하는 인터셉터를 구현하세요.
  • 클래스명: MDCLoggingInterceptor
  • 패키지명: com.**.discodeit.config
  • 요청 ID는 랜덤한 문자열로 생성합니다. (UUID)
  • 요청 ID는 응답 헤더에 포함시켜 더 많은 분석이 가능하도록 합니다. (Discodeit-Request-ID)
  • WebMvcConfigurer를 통해 MDCLoggingInterceptor를 등록하세요.
  • 클래스명: WebMvcConfig
  • 패키지명: com.**.discodeit.config
  • Logback 패턴에 MDC 값을 포함시키세요.
  • 로그 출력 예시 패턴 적용: [{요청ID} | {요청 메소드} | {요청 URL}]

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

  • Spring Boot Admin 서버를 구현할 모듈을 생성하세요. (admin 모듈 생성)
  • admin 모듈의 메인 클래스에 @EnableAdminServer 어노테이션을 추가하고, 서버는 9090번 포트로 설정합니다.
  • admin 서버 실행 후 localhost:9090/applications 에 접속해봅니다.
  • discodeit 프로젝트에 Spring Boot Admin Client를 적용합니다.
  • 의존성을 추가합니다. (spring-boot-admin-starter-client)
  • admin 서버에 등록될 수 있도록 설정 정보를 추가합니다.
    • application.yaml: instance name 설정
    • application-dev.yaml: admin server url (http://localhost:9090)
    • application-prod.yaml: admin server url (${SPRING_BOOT_ADMIN_CLIENT_URL})
  • discodeit 서버를 실행하고, admin 대시보드에 인스턴스가 추가되었는지 확인합니다.
  • admin 대시보드 화면에서 주요 메트릭 정보를 확인합니다. (요청 횟수, 응답시간, 서비스 상태 등)

테스트 커버리지 관리

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

주요 변경사항

스크린샷

image

멘토에게

  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

@bs8841 bs8841 added 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. 지각제출⏰ 제출일 이후에 늦게 제출한 PR입니다. labels Jul 2, 2025
@bs8841 bs8841 requested a review from codingjigi July 2, 2025 10:41
Copy link
Collaborator

@codingjigi codingjigi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MDC 적용하신 부분 진짜 잘하셨네요 ㅎㅎ 굳이 피드백 드릴 내용이 많이 없었어요.
궁금한 점만 리뷰 남겼습니다

* 요청마다 MDC에 컨텍스트 정보를 추가하는 인터셉터
*/
@Slf4j
public class MDCLoggingInterceptor implements HandlerInterceptor {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오..

}
private final AuthService authService;

@PostMapping(path = "login")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

앞에 / 붙여주세요~

@RestController
@RequestMapping("/api/binaryContents")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

REST API 설계 원칙에 맞춰서 소문자로 작성해주세요


import lombok.Builder;

@Builder
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

굳이 빌더 안써도 되지 않을까요 생성자로

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. 지각제출⏰ 제출일 이후에 늦게 제출한 PR입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants