Skip to content

[RELEASE] 3차 - 리팩토링, 성능 개선, 테스트코드 작성 #239

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.

Sign up for GitHub

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

Jump to bottom
Merged
Ogu1208 merged 304 commits into from
Aug 7, 2025
Merged

[RELEASE] 3차 - 리팩토링, 성능 개선, 테스트코드 작성 #239

Ogu1208 merged 304 commits into from
Aug 7, 2025

Conversation

@Ogu1208
Copy link
Contributor

@Ogu1208 Ogu1208 commented Aug 7, 2025 •
edited
Loading

관련 Issue (필수)

  • close #0

주요 변경 사항 (필수)

  • 리팩토링
  • 성능 개선
  • 테스트코드 작성

리뷰어 참고 사항

없음

추가 정보

없음

PR 작성 체크리스트 (필수)

  • 제목이 Issue와 동일함을 확인했습니다.
  • 리뷰어를 지정했습니다.
  • 프로젝트를 연결했습니다.

Jimin730 and others added 30 commits July 23, 2025 10:37
@Jimin730
refactor: 불필요한 파일 삭제 (#166)
7ee23e1
@Jimin730
refactor: Service 네임 변경 (#166)
520edb9
@Jimin730
refactor: swagger docs 파일 분리 (#166)
1871d3a
@Jimin730
refactor: 회원 Service, controller 메서드 분리 (#166)
ab33318
@Jimin730
refactor: 슬래시 누락 수정 (#166)
2ecd4e0
@Jimin730
[REFACTOR] 회원 도메인 클래스 파일 책임 분리
5f37220
@Jimin730
refactor: auth 패키지 추가 및 관련 파일 이동 (#175)
c8d0917
@Jimin730
refactor: response 파일 패키지 이동 (#175)
d3b1b4c
@Jimin730
[REFACTOR] auth 패키지 추가 및 패키지 구조 변경
bc2da4a
@Jimin730
bug: objectMapper 설정 추가 (#178)
e34a068
@Jimin730
refactor: RedisConfig 수정 (#179)
518c8ba
@Jimin730
[BUG] 로그아웃 시 Redis Duration 타입 직렬화/역직렬화 오류
7a483ee
@Jimin730
refactor: 로그인 로직에 http only 및 secure cookie 적용 (#168)
ef6f0ba
@Jimin730
refactor: TokenError 경로 수정 (#168)
c3567ac
@Jimin730
refactor: 토큰 재발급 로직 리팩토링 (#168)
f51a9b1
@Jimin730
refactor: 토큰 재발급 로직 수정 (#168)
991dfc6
@Jimin730
refactor: @CookieValue 적용 (#168)
838c6a0
@Jimin730
refactor: setCookie 메서드 분리 (#168)
f8cb95e
@Jimin730
refactor: Success 이름 변경 (#168)
347a28b
@Jimin730
refactor: 상수 필드 적용 (#168)
b4dccbf
@Jimin730
[REFACTOR] Http only 적용
697e2ae
@Friox
fix: 엔티티 관계 및 제약조건 수정 (#181)
550cfbc
@Friox
fix: leftjoin 조건 수정 및 null check 로직 추가 (#181)
1f9e437
@Friox
[FIX] Booking 도메인 리팩토링 후 노선 중간 구간 예매 불가
d304e0e
@Jimin730
feat: 로그아웃 시 cookie 값 만료 로직 추가 (#183)
632178a
@Jimin730
feat: @CookieValue required 속성 추가 (#183)
842f9e7
@Jimin730
refactor: 중복된 코드 메서드 분리 (#183)
b819d82
@Jimin730
refactor: CookieManager 유틸 클래스로 cookie 관련 메서드 관리 (#183)
5022115
@Jimin730
[FEATURE] 로그아웃 시 Cookie에 담긴 refreshToken 만료 로직 추가
fe78fb9
@Jimin730
refactor: TokenProvider 책임 분리 (#185)
3d1b7bb
chanwonlee and others added 24 commits August 7, 2025 11:16
@chanwonlee
refactor: given, when, then 주석 추가 (#220)
b04061d
@chanwonlee
[TEST] TrainSeatQueryService 테스트
81f3d5b
@Ogu1208
refactor: 열차 조회 Controller가 ApplicationService를 통해 접근하도록 수정 (#201)
114294f
@Ogu1208
refactor: 추천 객차 선택 로직의 가독성 개선 (동작 동일) (#201)
dffe89b
@Ogu1208
test: TrainSearchApplicationService 테스트코드 작성 (각 메서드, 전체 플로우 시나리오) (#201)
eacd679
@Ogu1208
test: trainTestHelper, ReservationTestHelper에 커스텀 데이터 생성 메서드 추가 (#201)
724f3c3
@Ogu1208
test: 출발 시간 조건에 따라 해당 시간 이후 출발하는 열차들이 시간순으로 필터링되는지 확인 테스트 (#201)
c3b3f45
@Ogu1208
feat: 좌석 상태에서 AVAILABLE, LIMITED 기준을 전체 좌석의 25% 비율 기준으로 판단하도록 수정 (#201)
c31ba19
@Ogu1208
feat: 입석 가능 여부 판단 로직 수정 : 일반실 예약 불가능 && 입석 요청 인원 수용 가능 (#201)
8ead284
@Ogu1208
test: 다양한 기존 예약 상태에 따라 좌석 상태와 입석 정보의 표시 여부 검증 (#201)
41a87f0
@Ogu1208
test: 열차 조회 - 입석 관련 좌석 및 상태 계산 , standingInfo 검증 테스트 (#201)
2da5e47 
test: 입석 관련 테스트 메서드명 수정 (#201)
@Ogu1208
test: 열차 조회 - 비즈니스적으로 잘못된 검색 조건일 경우 예외 발생 테스트 (#201)
25d9f21
@Ogu1208
test: 열차 조회 - TrainSearchRequestTest DTO Validation 테스트 (#201)
02eaf17
@Ogu1208
test: 다양한 검색 시나리오에서 검색 결과가 없을 경우 빈 리스트를 반환을 검증하는 테스트코드 작성 (#201)
65307da
@Ogu1208
test: 열차 조회 - 다구간으로 구간 예약 겹침(overlap)만 반영해 잔여 좌석을 정확히 계산하는지 테스트 (#201)
290c49a
@Ogu1208
test: 열차 조회 - 존재하지 않는 스케줄 ID 조회 오류, 페이징이 올바르게 동작(중복, 정렬)하는지 테스트(#201)
ba0e6c7
@Ogu1208
test: 열차 조회 - 조회하는 구간의 요금 정보가 없는 경우 예외 발생 테스트 작성 (#201)
baef7c9
@Ogu1208
test: TrainTestHelper 코드 충돌 해결 (#201)
6b1e5b5
@Ogu1208
test: 열차 조회 - 사용하지 않는 메서드 및 의존성 주입 삭제 (#201)
5f92551
@Ogu1208
[TEST] 열차 조회 서비스 테스트(TrainSearchApplicationService, TrainSearchService)
d64f4a2 
[TEST] 열차 조회 서비스 테스트(TrainSearchApplicationService, TrainSearchService)
@EndlessMilkyway
feat: PaymentFixture 구현 (#224)
f404344
@EndlessMilkyway
feat: PaymentService 테스트 작성 (#224)
5461c86
@EndlessMilkyway
style: 코드 스타일 수정 (#224)
e70d7a3
@EndlessMilkyway
[TEST] 결제 관련 기능 테스트 코드 작성
e002f64
@Ogu1208 Ogu1208 self-assigned this Aug 7, 2025
@Ogu1208 Ogu1208 merged commit f2c942c into main Aug 7, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@Ogu1208 Ogu1208

Labels

🚀 release

Projects

🚅 코레일
Status: Done

Milestone

테스트 코드 (07.28 - 08.08)

Development

Successfully merging this pull request may close these issues.

7 participants

@Ogu1208 @Jimin730 @Friox @Yunsung-Jo @chanwonlee @EndlessMilkyway