-
Notifications
You must be signed in to change notification settings - Fork 21
chore: migrate tests from karma to jest #52
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
Conversation
chore: remove karma-jasmine test environment
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.
리뷰완료합니다.
jest.config.js
Outdated
module.exports = { | ||
moduleFileExtensions: ['js'], | ||
testEnvironment: 'jsdom', | ||
|
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.
여기도 공백라인은 없어도 될 것 같아요!
{ | ||
_id: 4, | ||
value: 2 | ||
}, |
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.
린트 룰에 안맞지 않나요?
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.
린트 룰(prettier/prettier)에 의해서 해당 형태로 자동 변환됩니다!
test/core/view.spec.js
Outdated
var spy, | ||
spyMock = jest.fn(); |
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.
여긴
var spy;
var spyMock = jest.fn();
이 좋을 것 같습니다.
test/core/view.spec.js
Outdated
var view2, | ||
spy = jest.fn(); |
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.
여기도요!
test/core/view.spec.js
Outdated
var view2, | ||
view3, | ||
spy = jest.fn(); |
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.
여기도 입니다!
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.
리뷰완료합니다 👏
jest.fn
의 모킹 구현에 대한 짤막한 코멘트가 있습니다만, 테스트 동작에 문제가 없다면 바로 수정할 필요는 없어보입니다.
Please check if the PR fulfills these requirements
fix #xxx[,#xxx]
, where "xxx" is the issue number)Description
Thank you for your contribution to TOAST UI product. 🎉 😘 ✨