fix(store): storeReviews의 0 값 cursor 보존#160
Conversation
storeProducts(#158)와 동일하게 parseId("0")=0n이 args.cursor truthiness 체크에서 falsy로 떨어져 zero cursor가 페이지를 리셋하던 문제를 수정한다. listStoreReviews의 cursor 분기를 !== undefined로 바꾼다.
|
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 |
🧹 knip — dead-code 리포트전체 리포트
|
🩺 NestJS Doctor — 89/100 (Good)진단 258건 (error 0).
architecture / security 상위 항목
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Coverage report
Test suite run success1377 tests passing in 165 suites. Report generated by 🧪jest coverage report action from a8fd88b |
Summary
storeReviews의 cursor가"0"일 때 페이지가 리셋되던 버그를 수정한다. 머지된 PR #159(storeReviews) 이후, PR #158(storeProducts) 리뷰에서 발견된 동일 패턴(0nfalsy 누락)을 storeReviews에도 적용하는 후속이다.Scope
listStoreReviews의 cursor 분기를args.cursor ?→args.cursor !== undefined로 변경parseId("0") = 0n이 truthiness 체크에서 falsy로 떨어져 zero cursor가 필터를 드롭(=첫 페이지 재조회)하던 것을,0n을 유효 인자로 다뤄 빈 결과(올바른 동작)가 되도록 수정배경
0n보존 이슈(:740)를 지적해 반영했고, storeReviews에도 같은 패턴이 있어 일관성을 위해 후속 정리Impact
Test plan
cursor "0"은 페이지를 리셋하지 않고 빈 결과를 반환하는 테스트 추가yarn validate전체 통과(pre-push)