전승현[sprint7] by seunghyeonjeon57-dot · Pull Request #148 · codeit-bootcamp-spring/9-sprint-mission

seunghyeonjeon57-dot · 2026-03-30T07:52:15Z

요구사항

기본

프로파일 기반 설정 관리

로그 출력 예시

패턴

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

패키지명: com.sprint.mission.discodeit.exception[.{도메인}]

ErrorCode Enum 클래스를 통해 예외 코드명과 메시지를 정의하세요.
모든 예외의 기본이 되는 DiscodeitException 클래스를 정의하세요.
DiscodeitException을 상속하는 주요 도메인 별 메인 예외 클래스를 정의하세요.

UserException, ChannelException 등
실제로 활용되는 클래스라기보다는 예외 클래스의 계층 구조를 명확하게 하기 위한 클래스 입니다.

도메인 메인 예외 클래스를 상속하는 구체적인 예외 클래스를 정의하세요.
기존에 구현했던 예외를 커스텀 예외로 대체하세요.
ErrorResponse를 통해 일관된 예외 응답을 정의하세요.
앞서 정의한 ErrorResponse와 @RestControllerAdvice를 활용해 예외를 처리하는 예외 핸들러를 구현하세요.

모든 핸들러는 일관된 응답(ErrorResponse)을 가져야 합니다.
유효성 검사

예시

@import({ErrorCodeStatusMapper.class})

주요 컨트롤러(User, Channel, Message)에 대해 최소 2개 이상(성공, 실패)의 테스트 케이스를 작성하세요.
MockMvc를 활용해 컨트롤러를 테스트하세요.
서비스 레이어를 모의(mock)하여 컨트롤러 로직만 테스트하세요.
JSON 응답을 검증하는 테스트를 포함하세요.

통합 테스트

심화

주요 변경사항

멘토에게

매운맛으로 솔직하게 부탁드리겠습니다
jaCoCo를 통해 요구사항인 User, Channel, Message 도메인의 테스트를 챙겼습니다. 하지만 실무에서는 모든 클래스를 100% 채우는 것이 비효율적일 수도 있다고 들었습니다. 멘토님은 어떤 기준으로 반드시 테스트해야 할 도메인과 테스트 비중을 낮춰도 되는 도메인을 구분하시는지 궁금합니다
이번에 DiscodeitException을 최상위로 두고 도메인별 예외(UserException 등)를 계층화했습니다. 프로젝트가 커지면 예외 클래스가 너무 많아질 것 같은데, 실무에서는 예외 클래스를 세분화하는 편인가요, 아니면 ErrorCode Enum으로만 구분하는 편인가요
단위, 슬라이스, 통합 테스트를 모두 작성해보니 통합 테스트는 설정이 무겁고 실행 속도가 느리다는 걸 체감했습니다. 실무 프로젝트에서 테스트 피드백 속도를 높이기 위해 통합 테스트의 비중을 어느 정도로 조절하시는지 궁금합니다.
이번 미션포함 해서 미션을 하면 할수록 AI에 대한 의존도가 좀 높아지는 기분인데 어떤식으로 사용해야 효율적일지 궁금합니다. 현재는 계속 질문을 던지고 그 개념에 대해 파고들어 이해하는 형식으로 사용하고 있습니다

q

…/전승현 [전승현]Sprint2

…t/전승현 전승현/스프린트미션3

eedys1234 · 2026-04-07T14:01:51Z

@seunghyeonjeon57-dot 님 merge 하는 브랜치가 잘못된 것 같습니다.

eedys1234 · 2026-04-07T14:03:19Z

.gitignore 파일 안에 .ser 파일 관련하여 명시하여 원격 저장소에 저장되는 것을 제외해야 할 것 같습니다.

eedys1234 · 2026-04-07T14:05:58Z

+@RequiredArgsConstructor
+public class AuthController implements AuthApi {
+
+  private final UserMapper mapper;


mapper 사용처가 없어보이는데 맞을까요? 맞다면 제거해도 될 것 같습니다.

eedys1234 · 2026-04-07T14:10:13Z

+
+  <springProfile name="dev">
+    <root level="INFO">
+      <appender-ref ref="CONSOLE"/>


환경 별로 세분화하신 부분 👍👍
구현하신 것과 같이 현업에서도 환경 별로 출력하는 곳을 다르게 설정합니다.

eedys1234 · 2026-04-07T14:11:06Z

+      </pattern>
+    </encoder>
+    <rollingPolicy class="ch.qos.logback.core.rolling.SizeAndTimeBasedRollingPolicy">
+      <fileNamePattern>${LOG_PATH}/archived/${LOG_FILE_NAME}-%d{yyyy-MM-dd}.%i.log</fileNamePattern>


파일의 크기가 너무 크면 서버에서 보기가 어려워 질 수 있어 조그마한 크기로 조각내는데 디테일하게 잘 구분해주셨습니다.

eedys1234 · 2026-04-07T14:12:41Z

+  datasource:
+    url: jdbc:postgresql://localhost:5432/discodeit
+    username: discodeit_user
+    password: discodeit1234


동일 username 이지만 망 별로 패스워드를 다르게 하는 것을 추천드려요.

eedys1234 · 2026-04-07T14:13:49Z

@@ -0,0 +1,49 @@
+spring:


prod에는 server.port 가 존재하지 않는 것 같아요.
공통적으로 사용되는 부분들은 applicaiton.yaml 파일을 생성하여 선언해주세요.

eedys1234 · 2026-04-07T14:23:16Z

+@SpringBootTest(webEnvironment = WebEnvironment.RANDOM_PORT)
+@ActiveProfiles("test")
+@Transactional
+@AutoConfigureMockMvc


선언하신 부분을 커스텀 애노테이션으로 선언하여 사용하면 어떨까요?

@Target(ElementType.TYPE) @Retention(RetentionPolicy.RUNTIME) @SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT) @ActiveProfiles("test") @Transactional @AutoConfigureMockMvc public @interface IntegrationTest { }

애노테이션 명을 통해 의미적으로 부여할 수 있을 것 같습니다.

eedys1234 · 2026-04-07T14:24:47Z

+            .contentType(MediaType.APPLICATION_JSON)
+            .content(mapper.writeValueAsString(request)))
+        .andExpect(status().isCreated());
+    var createChannel = channelRepository.findByName("하이").orElseThrow();


저장 API 호출 이후 저장된 데이터가 정상적으로 저장되었는지 확인하려면 조회 API를 별도로 호출하는 것을 추천드려요.

즉, 저장 API -> 조회 API 순이 될 것 같아요.

eedys1234 · 2026-04-07T14:25:35Z

+  }
+
+  @Test
+  @DisplayName("채널 삭제 API를 호출하면 Db에서 채널 삭제가되어야한다")


위와 동일하게 삭제하기 위해 저장 -> 삭제 -> 조회 순으로 API를 호출하는 것을 권장드려요.

기능 별 API 호출 함수를 선언하여 사용하는 방식이면 재사용성이 좋을 것 같아요.

eedys1234 · 2026-04-07T14:27:01Z

+  @InjectMocks
+  private BasicChannelService channelService;
+
+  @Nested


테스트들을 그룹핑하는 방법 좋습니다. 👍👍

eedys1234 · 2026-04-07T14:27:23Z

+      assertThat(result).isNotNull();
+      assertThat(result.name()).isEqualTo("공용채널1");
+
+      verify(channelRepository).save(any(Channel.class));


값 뿐만 아니라 호출 행위를 검증하는 부분도 좋습니다.

eedys1234 · 2026-04-07T14:37:51Z

jaCoCo를 통해 요구사항인 User, Channel, Message 도메인의 테스트를 챙겼습니다. 하지만 실무에서는 모든 클래스를 100% 채우는 것이 비효율적일 수도 있다고 들었습니다. 멘토님은 어떤 기준으로 반드시 테스트해야 할 도메인과 테스트 비중을 낮춰도 되는 도메인을 구분하시는지 궁금합니다

저는 버그가 발생할 가능성이 높은 즉 로직 복잡도가 높은 코드들에 대해 집중적으로 테스트 코드를 작성하는 편입니다.
추가적으로, 이후 개발자가 테스트를 진행하면서 버그가 발생한 코드가 있다면 추가 케이스에 대한 테스트 코드를 작성합니다.

이번에 DiscodeitException을 최상위로 두고 도메인별 예외(UserException 등)를 계층화했습니다. 프로젝트가 커지면 예외 클래스가 너무 많아질 것 같은데, 실무에서는 예외 클래스를 세분화하는 편인가요, 아니면 ErrorCode Enum으로만 구분하는 편인가요
단위, 슬라이스, 통합 테스트를 모두 작성해보니 통합 테스트는 설정이 무겁고 실행 속도가 느리다는 걸 체감했습니다. 실무 프로젝트에서 테스트 피드백 속도를 높이기 위해 통합 테스트의 비중을 어느 정도로 조절하시는지 궁금합니다.

개인적으로 통합 테스트를 선호하지 않아, 정말 통합 테스트가 필요한 기능들만 작성하는 편입니다.
인프라 별 예외 클래스는 세분화 하지만, 도메인 별 예외 케이스는 특정 예외 클래스를 선언하여 다음과 같이 활용하는 편입니다.

인프라 별 예외(특정 도메인에 종속되는 게 아닌 전역적으로 사용되는)
FeatureFlagException
AuthenticationException

public class ChannelExceptionFactory {
   public static ApplicationException requiredChannelName() {
      return new ApplicationException("채널명이 존재하지 않습니다.");
   }
}

throw ChannelExceptionFactory.requiredChannelName();

이번 미션포함 해서 미션을 하면 할수록 AI에 대한 의존도가 좀 높아지는 기분인데 어떤식으로 사용해야 효율적일지 궁금합니다. 현재는 계속 질문을 던지고 그 개념에 대해 파고들어 이해하는 형식으로 사용하고 있습니다

잘하고 계시는 것 같아요.
어떤 기능을 구현할지 질의를 한 다음 AI가 응답하는 소스코드를 기반으로 모르는 부분들을 세부적으로 다시 질의합니다.
또한 상세한 개념 뿐만 아니라 개념을 활용한 기능 혹은 결과물이 무엇인지 익히면 좋을 것 같습니다.

seunghyeonjeon57-dot and others added 29 commits January 9, 2026 14:42

2026-01-09

3f9d90f

q

sprint mission2

f9dde0e

중간 점검

163ea4d

Sprint Misson2 finish

9de5aaa

sprint misson complete

de238f5

수정본

2513049

sprint mission 2 수정본

b885096

final

1b4c6e0

completed final implementation of the project

66df86b

sprint mission2

5ca8248

finally status

b67f815

sprint mission3 진행중

4f7f1c6

sprint3:plus UserStatus

06e7f86

Merge pull request codeit-bootcamp-spring#4 from seunghyeonjeon57-dot…

f57d6b3

…/전승현 [전승현]Sprint2

feat: 추가기능요구사항 추가

317aa12

feat:최종 제출본

efeb516

feat:sprintMisson4 최종본

ffac697

feat:sprint4 최종제출본

954f77b

feat: add postman collection for mission 4

ea1a353

Merge pull request codeit-bootcamp-spring#41 from seunghyeonjeon57-do…

cd5317e

…t/전승현 전승현/스프린트미션3

Final Structural Fix: Removing 404 Errors

3288b80

Merge remote-tracking branch 'haguon/전승현' into 전승현

58fe5c4

feat:예외처리 구현

4381d85

feat:유효성 검사 완료

df14922

feat:actuator 추가

d88c73f

feat:유저 단위 테스트 완성

08c5cbf

feat:단위 테스트 완료

6edc3f3

feat : repository 테스트 구현

fa1579f

feat:심화요구사항까지완료

e4b4fc8

seunghyeonjeon57-dot requested a review from eedys1234 March 30, 2026 12:34

seunghyeonjeon57-dot requested a review from rlarudgkswkd March 30, 2026 12:34

eedys1234 reviewed Apr 7, 2026

View reviewed changes

Conversation

seunghyeonjeon57-dot commented Mar 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

요구사항

기본

패턴

심화

주요 변경사항

멘토에게

Uh oh!

eedys1234 commented Apr 7, 2026

Uh oh!

eedys1234 commented Apr 7, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

eedys1234 commented Apr 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

seunghyeonjeon57-dot commented Mar 30, 2026 •

edited

Loading