[🎉deploy] CI/CD 테스트 진행 #11

junggyo1020 · 2025-03-25T12:11:11Z

⚙️ ISSUE

closed #

📄 Work Description

CI/CD 배포 자동화 테스트를 진행합니다.

진행상황

CI & CD 로직 정상적으로 실행 확인

컨테이너 정상적으로 실행 확인

✅ PR check list

Reviewers
Assignees
Labels

서버 내부 오류 및 게이트웨이 관련 예외를 처리하기 위한 전역 에러 코드 열거형 구현. 공통 접두사를 포함한 에러 메시지 형식을 제공함.

BaseException 및 주요 시스템 예외(501, 502, 503, 504 등)를 처리하는 전역 예외 핸들러 정의. 각 예외에 대해 GlobalErrorCode를 매핑하여 일관된 에러 응답을 반환하도록 구성함.

성공 응답에 사용될 HTTP 상태 코드와 메시지를 제공하는 공통 인터페이스 생성.

성공 결과를 클라이언트에 전달하기 위한 공통 응답 객체 정의. HttpStatus를 int로 변환하여 응답 형식을 표준화하고, 결과 데이터는 null일 경우 제외되도록 설정함.

- 이름 null 허용 불가, 최대 길이 초과, 형식 오류에 대한 에러 코드 추가 - 공통 메시지 접두사 PREFIX 적용 - 이름 최대 길이 상수 NAME_MAX_LENGTH 정의

- UserErrorCode를 인자로 받아 BaseException을 상속하는 예외 정의 - 사용자 관련 예외 처리를 도메인 단위로 구분 가능하도록 구성

- 기존 IllegalArgumentException을 UserException으로 대체하여 도메인 예외 통일 - UserErrorCode를 활용해 에러 메시지 관리 일원화 - 이름 검증 로직에서 null, 길이, 형식 예외를 도메인 기반으로 처리

- 기존 IllegalArgumentException 검증을 UserException으로 변경 - UserErrorCode의 메시지를 기반으로 예외 메시지 검증 수행 - 사용자 이름 검증 관련 테스트 일관성 유지

푸시 토큰이 null일 수 없다는 검증을 위한 에러 코드를 UserErrorCode에 추가했습니다.

@Embedded

기존 String 타입의 token 필드를 PushToken VO로 변경하고, @Embedded 및 @AttributeOverride를 적용하여 도메인 주도 설계 기반의 값 객체 구조를 반영했습니다.

푸시 토큰 값을 표현하는 PushToken VO를 구현했습니다. null 값 검증 로직을 포함하고 있으며, 추후 토큰 유효성에 대한 비즈니스 로직과 테스트를 추가할 예정입니다.

PushToken 값 객체에 대한 단위 테스트를 추가했습니다. - 유효한 토큰으로 객체 생성 성공 - null 토큰에 대한 예외 처리 검증

- 푸시 토큰 null 허용 불가 에러 코드 추가 - 푸시 상태가 ON, OFF 외의 값일 경우의 에러 코드 추가 - 푸시 알림이 이미 활성화/비활성화된 상태에 대한 예외 코드 추가

- PushNotificationStatus VO 필드 추가 및 ENUM 매핑 적용 - 푸시 알림 수신 여부 확인 메서드 canReceivePushNotification() 구현 - 푸시 알림 활성화 및 비활성화 메서드 추가

- ENABLED, DISABLED 상태를 나타내는 Enum 정의 - 문자열 기반 상태 파싱을 위한 of(), isValid() 메서드 구현 - 상태 전환 시 중복 전환 방지를 위한 예외 처리 로직 추가 - 알림 수신 가능 여부 확인 메서드 canReceiveNotification() 구현

- of(), isValid() 메서드의 유효/무효 입력에 대한 테스트 추가 - 상태 전환 시 중복 호출에 대한 예외 처리 테스트 추가 - 상태별 알림 수신 가능 여부 테스트 및 상태 전환 테스트 포함 - 성공/실패 케이스를 명확히 나누어 가독성 향상

JungYoonShin

수고하셨슴동!

JungYoonShin · 2025-03-26T05:17:54Z

.github/workflows/DEV-CI.yml

+      # # Gradle caching - 빌드 시간 향상
+      # - name: Gradle Caching
+      #   uses: actions/cache@v3
+      #   with:
+      #     path: |
+      #       ~/.gradle/caches
+      #       ~/.gradle/wrapper
+      #     key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle*', '**/gradle-wrapper.properties') }}
+      #     restore-keys: |
+      #       ${{ runner.os }}-gradle-
+
+      # Gradle build - 테스트 없이 gradle 빌드


이거 주석처리한 이유가 먼가용?? (단순 궁금ㅎㅎ)

해당 빌드 캐싱이 자주 빌드해야하는 상황에서 오히려 시간이 걸리더라구요! 이유를 파악하고 이후 반영을 고려하고 있습니다:)

JungYoonShin · 2025-03-26T05:18:28Z

HELP.md

이 파일은 모져?!?! 잘못 올라왓나

start.spring.io에서 최근에 gradle파일을 추가하는 경우 자동으로 생성되는 것 같아요! 이부분은 반영되지 않도록 gitignore에 추가해야겠네요!! 의견 감사합니다 ㅎㅎ

jsoonworld · 2025-03-26T05:34:39Z

고생 많으셨습니다~! 😊

추가로 deploy 파일에 도커 이미지 삭제 로직도 포함되었는지 한 번 확인 부탁드릴게요.
예전에 기존 서버에서는 이 부분이 빠져 있어서 간헐적으로 문제가 생겼던 적이 있었거든요.

그리고 서버가 중간에 죽고 다시 올라올 때처럼,
예외 상황이 생길 수 있는 흐름도 한 번 더 점검해보시면 더 안정적으로 동작할 것 같아요 🙏

- LOGGED_IN, LOGGED_OUT, WITHDRAWN 상태를 나타내는 enum 정의 - 문자열 파싱 메서드 of(), isValid() 구현 - 로그인, 로그아웃, 탈퇴 상태 확인을 위한 메서드 추가 - 유효하지 않은 상태에 대해 UserException 발생 처리

- 유효하지 않은 계정 상태 입력 시 발생하는 INVALID_ACCOUNT_STATUS 예외 코드 정의 - 허용된 상태 값: active, inactive, withdrawn

- 기존 State 필드 제거 및 AccountStatus enum 기반으로 변경 - 계정 활성 상태 판단 메서드 isActiveUser() 구현 - 탈퇴 여부를 AccountStatus가 직접 판단하도록 역할 위임

@jsonvalue

- 각 상태에 외부 노출용 문자열 값(value) 필드 추가 (active, inactive, withdrawn) - Jackson 직렬화/역직렬화를 위한 @JsonCreator 및 @jsonvalue 어노테이션 적용 - 문자열 입력 파싱을 value 기준으로 처리하도록 from(), isValid() 메서드 개선 - 의미 기반 메서드 isActive(), isInactive(), isWithdrawn() 등으로 명확성 향상

- from() 메서드의 유효/무효 입력에 대한 예외 처리 테스트 구현 - isValid() 메서드의 검증 테스트 추가 - isActive(), isInactive(), isWithdrawn() 메서드 테스트로 상태 판별 기능 확인 - value() 메서드를 통한 직렬화 값 반환 검증

- `ScrapUserIdsRequest` 레코드를 추가하여 스크랩 유저 ID 목록을 담는 요청 DTO 구현 - `of` 메서드를 사용하여 리스트를 받아 DTO 인스턴스를 생성할 수 있도록 추가 - 클라이언트와 API 통신에서 유저 ID 목록을 전송하는 데 사용

[✨ feat] 운영서버-알림서버 동기화 응답 반환

junggyo1020 added 🎉deploy CI/CD 관련 코드 수정 및 추가 🐶정교🐶 labels Mar 25, 2025

junggyo1020 requested review from JungYoonShin and jsoonworld March 25, 2025 12:11

junggyo1020 self-assigned this Mar 25, 2025

jsoonworld added 18 commits March 26, 2025 10:15

feat(GlobalErrorCode) : 500번대 글로벌 에러 코드 정의

d2ad1b6

서버 내부 오류 및 게이트웨이 관련 예외를 처리하기 위한 전역 에러 코드 열거형 구현. 공통 접두사를 포함한 에러 메시지 형식을 제공함.

feat(SuccessCode) : 성공 응답 코드 인터페이스 정의

8c47fbf

성공 응답에 사용될 HTTP 상태 코드와 메시지를 제공하는 공통 인터페이스 생성.

del(temp) : 사용하지 않는 파일 삭제

dd137ab

feat(UserErrorCode) : 사용자 이름 제약 및 에러 코드 정의

73d69e8

- 이름 null 허용 불가, 최대 길이 초과, 형식 오류에 대한 에러 코드 추가 - 공통 메시지 접두사 PREFIX 적용 - 이름 최대 길이 상수 NAME_MAX_LENGTH 정의

feat(UserException) : 사용자 도메인 전용 예외 클래스 구현

1f21ebb

- UserErrorCode를 인자로 받아 BaseException을 상속하는 예외 정의 - 사용자 관련 예외 처리를 도메인 단위로 구분 가능하도록 구성

feat(UserErrorCode) : 푸시 토큰 null 검사 에러 코드 구현

7ffabf3

푸시 토큰이 null일 수 없다는 검증을 위한 에러 코드를 UserErrorCode에 추가했습니다.

feat(User) : PushToken VO 적용을 위한 필드 타입 변경

20eaa70

기존 String 타입의 token 필드를 PushToken VO로 변경하고, @Embedded 및 @AttributeOverride를 적용하여 도메인 주도 설계 기반의 값 객체 구조를 반영했습니다.

feat(PushToken) : PushToken VO 구현

5b1b217

푸시 토큰 값을 표현하는 PushToken VO를 구현했습니다. null 값 검증 로직을 포함하고 있으며, 추후 토큰 유효성에 대한 비즈니스 로직과 테스트를 추가할 예정입니다.

test(PushToken) : PushToken VO 테스트 코드 작성

535d7b8

PushToken 값 객체에 대한 단위 테스트를 추가했습니다. - 유효한 토큰으로 객체 생성 성공 - null 토큰에 대한 예외 처리 검증

refactor(UserNameTest) : UserName 테스트 패키지 위치 이동

f3a4734

feat(UserErrorCode) : 푸시 알림 관련 에러 코드 구현

e5a5972

- 푸시 토큰 null 허용 불가 에러 코드 추가 - 푸시 상태가 ON, OFF 외의 값일 경우의 에러 코드 추가 - 푸시 알림이 이미 활성화/비활성화된 상태에 대한 예외 코드 추가

feat(User) : 푸시 알림 상태 관리 기능 및 필드 추가

3b5d4c4

- PushNotificationStatus VO 필드 추가 및 ENUM 매핑 적용 - 푸시 알림 수신 여부 확인 메서드 canReceivePushNotification() 구현 - 푸시 알림 활성화 및 비활성화 메서드 추가

JungYoonShin approved these changes Mar 26, 2025

View reviewed changes

jsoonworld added 5 commits March 26, 2025 14:52

feat(UserErrorCode) : 계정 상태 값 유효성 검사용 에러 코드 추가

197e74b

- 유효하지 않은 계정 상태 입력 시 발생하는 INVALID_ACCOUNT_STATUS 예외 코드 정의 - 허용된 상태 값: active, inactive, withdrawn

jsoonworld force-pushed the develop branch from 805a072 to a0871aa Compare April 2, 2025 12:29

jsoonworld and others added 27 commits April 3, 2025 22:55

merge(ScrapController) : 운영서버-알림서버 동기화 응답 반환

99694d7

[✨ feat] 운영서버-알림서버 동기화 응답 반환

Merge remote-tracking branch 'origin/develop' into develop

536f17e

[✅ chore] 브랜치 간 충돌 해결

c3ef3d8

Update Dockerfile

b83604e

Update DOCKER-CD.yml

42c4c6b

Update DOCKER-CD.yml

3cf6ea2

Update DOCKER-CD.yml

fb25344

Update DOCKER-CD.yml

870f350

Update DOCKER-CD.yml

023d90e

Update DOCKER-CD.yml

014f6fe

Update DOCKER-CD.yml

daa8777

Update DOCKER-CD.yml

0502f5c

Update DOCKER-CD.yml

86c124f

[✨ feat] 긴 JSON 환경변수를 JSON 파일로 전달후 mount하는 방식으로 변경

1fb82fe

Update DOCKER-CD.yml

9c273c8

Update DOCKER-CD.yml

ffc0539

Update DOCKER-CD.yml

46d9f56

Update DOCKER-CD.yml

848bd39

[✨ feat] 긴 JSON 환경변수 인코딩방식 변경

f27a929

[✨ feat] 긴 JSON 환경변수 인코딩방식 변경

55c6aef

Update DOCKER-CD.yml

2e96f4e

Update DOCKER-CD.yml

3691c88

Update DOCKER-CD.yml

3b9afe6

Update DOCKER-CD.yml

40c8fe6

Update DOCKER-CD.yml

d640e75

Update DOCKER-CD.yml

4f92feb

Update DOCKER-CD.yml

c8321d8

JungYoonShin closed this Apr 7, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[🎉deploy] CI/CD 테스트 진행 #11

[🎉deploy] CI/CD 테스트 진행 #11

Uh oh!

junggyo1020 commented Mar 25, 2025 •

edited

Loading

Uh oh!

JungYoonShin left a comment

Uh oh!

JungYoonShin Mar 26, 2025

Uh oh!

junggyo1020 Mar 26, 2025

Uh oh!

JungYoonShin Mar 26, 2025

Uh oh!

junggyo1020 Mar 26, 2025

Uh oh!

jsoonworld commented Mar 26, 2025

Uh oh!

Uh oh!

[🎉deploy] CI/CD 테스트 진행 #11

[🎉deploy] CI/CD 테스트 진행 #11

Uh oh!

Conversation

junggyo1020 commented Mar 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⚙️ ISSUE

📄 Work Description

진행상황

CI & CD 로직 정상적으로 실행 확인

컨테이너 정상적으로 실행 확인

✅ PR check list

Uh oh!

JungYoonShin left a comment

Choose a reason for hiding this comment

Uh oh!

JungYoonShin Mar 26, 2025

Choose a reason for hiding this comment

Uh oh!

junggyo1020 Mar 26, 2025

Choose a reason for hiding this comment

Uh oh!

JungYoonShin Mar 26, 2025

Choose a reason for hiding this comment

Uh oh!

junggyo1020 Mar 26, 2025

Choose a reason for hiding this comment

Uh oh!

jsoonworld commented Mar 26, 2025

Uh oh!

Uh oh!

junggyo1020 commented Mar 25, 2025 •

edited

Loading