-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
feat: set up Vitest for Next.js #7
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
빠른 작업 너무 좋습니다 👍 💯 너무 고생하셨습니다~
The CJS build of Vite's Node API is deprecated. See https://vitejs.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated for more details.
실행하면 위 오류가 뜨는데 deprecate 된 부분 한 번 신경써주시면 감사하겠습니다~
Issue Number
close: #2
Description
위에서 close 키워드 제거해주시면 감사하겠습니다! 추후 편의를 위해 놔두었는데 이런 경우에는 없는게 더 낫겠다는 생각이 드네요. 없애버리는게 나을까요? 아니면 check list에 추가하는게 나을까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저도 빨리 해야겠다는 동기부여를 받게 되네요. util 컨벤션만 확인 부탁드려요. 고생 많으셨습니다!
@poiu694 꼼꼼한 리뷰 정말정말 감사해요!! 🥹
|
@Pridesd 확인 감사합니다!! 저도 아직 스토리북 세팅 끝내려면 한참 남았어요 ㅋㅋ 유틸 코드 컨벤션 맞춰서 수정해둘게요~! |
The CJS build of Vite's Node API is deprecated. See https://vitejs.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated for more details.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨슴니다~
Issue Number
#2
Description
To Reviewers
다음 문서를 참고해서 Vitest를 설정했습니다.
__tests__
폴더 밑에 테스트할 코드를 모두 넣는 구조로 작성해보았습니다.__tests__
라는 이름은 jest의 컨벤션입니다. https://jestjs.io/docs/configuration대신
__tests__
하위 폴더 구조를 src의 폴더 구조랑 같게끔(테스트할 코드가 위치한 폴더만 추가하는 방식) 하면 어떨까 합니다!추가하고 싶은 설정이 있다면 알려주세요!
Snapshot이나 Mocking 샘플은 차차 올리도록 하겠습니다...!
Node 버전은 다음 경고 때문에 LTS로 변경했습니다.
(node:30570) [DEP0040] DeprecationWarning: The
punycode
module is deprecated. Please use a userland alternative instead.editorconfig는 extension을 설치하고
npm install -g editorconfig
를 입력하셔야 정상 작동합니다.Cf. https://stackoverflow.com/a/48523398/11625302
Checklist
feat: add login page
)