Skip to content

Conversation

jajugoguma
Copy link
Contributor

Please check if the PR fulfills these requirements

  • It's submitted to right branch according to our branching model
  • It's right issue type on title
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes/features)
  • Docs have been added/updated (for bug fixes/features)
  • It does not introduce a breaking change or has description for the breaking change

Description

  • remove karma-jasmine test environment
  • set jest test environment
  • tests migrate from jasmine to jest

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

Copy link
Contributor

@lja1018 lja1018 left a 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',

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

여기도 공백라인은 없어도 될 것 같아요!

Comment on lines +518 to +521
{
_id: 4,
value: 2
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

린트 룰에 안맞지 않나요?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

린트 룰(prettier/prettier)에 의해서 해당 형태로 자동 변환됩니다!

Comment on lines 85 to 86
var spy,
spyMock = jest.fn();
Copy link
Contributor

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();

이 좋을 것 같습니다.

Comment on lines 215 to 216
var view2,
spy = jest.fn();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

여기도요!

Comment on lines 232 to 234
var view2,
view3,
spy = jest.fn();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

여기도 입니다!

Copy link

@adhrinae adhrinae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리뷰완료합니다 👏
jest.fn 의 모킹 구현에 대한 짤막한 코멘트가 있습니다만, 테스트 동작에 문제가 없다면 바로 수정할 필요는 없어보입니다.

@jajugoguma jajugoguma merged commit e79c52e into master Jun 18, 2021
@jajugoguma jajugoguma deleted the chore/karma-to-jest branch June 18, 2021 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants