feat(seller): GraphQL Input 전수 class-validator DTO 도입#99
Open
chanwoo7 wants to merge 3 commits into
Open
Conversation
A-2 검증 전략 P0-3 단계 4 (Seller Input 전수). - SellerCursorInput · SellerDateCursorInput 베이스 도입 (limit/cursor + 날짜 범위). SDL 기본값 limit=20, 상한 100. - 리스트/필터 DTO 4종: ProductList / OrderList / AuditLogList - 콘텐츠 DTO 4종: FAQ / Banner Create+Update - 대화 + 주문 상태 변경 DTO 2종 - 매장 DTO 5종: 기본정보 / 영업시간(dayOfWeek 0-6) / 임시휴무 / 픽업정책(양수 정수) / 일일 캐파시티 - 상품 DTO 7종: CRUD + Active 토글 + 이미지 추가/순서 + 카테고리/태그 - 옵션 DTO 6종: 그룹 CRUD + 아이템 CRUD + 순서 변경 - 커스텀 템플릿 DTO 4종: 템플릿 upsert/active + 텍스트 토큰 upsert/순서 도메인 invariant (minSelect <= maxSelect, banner linkType 별 link* 조합, SALE_PRICE_EXCEEDS_REGULAR 등) 는 service 에서 검증 (cross-field 규칙). 일부 link*Id / openTime / note / reason 등 optional 필드는 FE 의 명시적 null 송신 호환을 위해 `?: T | null` 로 정의. class-validator 데코레이터로 형식·범위·enum 검증을 전수 적용. 각 DTO 에 대응 spec (필수/선택 분기, 형식 오류, 경계값 등 의미 있는 케이스만; coverage 채우기 식 저품질 테스트 금지).
- Seller resolver 9종: @Args 인자 타입을 신규 DTO class 로 교체 (ValidationPipe 가 검증). 구 interface 다중 import 라인을 개별 import 로 분해. - Seller service 7종: import 경로를 dto/inputs/* 로 변경 (type import 유지). - types/seller-input.type.ts (268 줄) 삭제 — 모든 정의가 DTO class 로 이전됨. - 일부 spec 파일의 `as never` / `as unknown as X` 캐스트가 isolatedModules:false 하에서 불필요해져 lint-staged 가 자동 정리. - spec 1건의 startsAt/endsAt 입력을 string 에서 Date 로 정정 (DTO 가 Date 만 허용; GraphQL DateTime scalar 가 변환 후 도달). FE 영향: 정상 입력은 응답 동일. 잘못된 형식은 ValidationPipe 가 일관된 BAD_REQUEST 로 응답.
원인: tsconfig 의 isolatedModules: true 하에서 ts-jest 는 파일별 독립
컴파일을 수행하며 type-vs-value 식별 정보를 갖지 못한다. 그 결과 모든
non-primitive type annotation 에 보수적 런타임 가드를 emit:
__metadata("design:type", typeof (_a = typeof T !== "undefined" && T)
=== "function" ? _a : Object)
이 식은 T 가 type alias 일 때 항상 한쪽 분기만 발화 → istanbul 이
"unreachable 분기" 로 측정 (~5%p global 하락 원인).
해결: tsconfig 는 isolatedModules: true 유지 (build/nest cli 권장),
jest transform 에서만 isolatedModules: false 로 override → ts-jest 가
full type info 로 컴파일, `__metadata("design:type", String)` 같은
단순 emit. nodenext hybrid module 경고(TS151002) 는 diagnostics 로 무시.
효과:
- 전체 branches 84.53% → 92.85% (임계 86% 회복)
- seller/dto/inputs 64.28% → 100%
- 정확한 root-cause 해소이므로 istanbul-ignore 같은 회피 불요
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Coverage report
Test suite run success1166 tests passing in 130 suites. Report generated by 🧪jest coverage report action from a93efac |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@Args타입으로 사용 (ValidationPipe 검증)types/seller-input.type.ts(268 줄) 삭제?: T | nullts-jest decorator metadata 분기 해결 (별도 커밋)
isolatedModules: true하에서 ts-jest 가 type-vs-value 식별 정보 없이 보수적 런타임 가드(typeof T !== "undefined" && T ...)를 emit → istanbul 이 unreachable 분기로 계측isolatedModules: falseoverride +TS151002무시. 결과: branches 92.85%, seller/dto/inputs 100%배경 / Plan 연결
FE 영향
정상 입력 시 응답 동일. 잘못된 형식은 ValidationPipe 가 일관 처리 (
extensions.code = BAD_USER_INPUT).룰별 메시지 본문은 기존 한국어 문구 유지 (해당하는 곳
message옵션 명시).Test plan
yarn lintyarn test:cov1166/1166 (이전 996 + 신규 170)yarn dto:checkerrors 0npx tsc --noEmityarn validate) 실 통과 확인