Skip to content
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

[Feature] Chip 컴포넌트 테스트 코드 작성 #90

Closed
wants to merge 13 commits into from

Conversation

eugene028
Copy link
Collaborator

🎉 변경 사항

  • Chip 컴포넌트에 대한 테스트 코드를 작성했어요.(토글, 랜더링, 외부 컨트롤)
  • 테스트 코드를 작성함에 있어서 Chip 컴포넌트에 접근성 관련된 스펙을 빼먹은 것들이 있어 몇가지 추가했어요!

🙏 여기는 꼭 봐주세요!

@ghdtjgus76
Switch 컴포넌트와 Chip 컴포넌트가 유사한게 굉장히 많아서 코드를 많이 참고했는데요,
확인해보니 testing library의 fireEvent에 대한 사용을 많이 하셨더라구요,
어떤 아티클을 읽어보니까 , fireEvent에서 userEvent로 마이그레이션을 하시길래 그 이유를 찾아보았습니다.

  • fireEvent.click은 단순히 클릭 이벤트만 일으키지만, userEvent.click은 어떤 element에서 발생한 이벤트인지 체크하여, 클릭이벤트가 호출될 때 브라우저에서 발생하는 모든 이벤트들을 함께 발생시켜준다.(DOM Event 형식으로 발생)
  • rtl 공식 홈페이지에서도 이렇게 추천하고 있기도 하구영!
스크린샷 2024-07-19 오후 12 19 12

Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

Copy link
Contributor

@eugene028 eugene028 closed this Jul 20, 2024
@ghdtjgus76 ghdtjgus76 deleted the feature/chip-test-code branch July 25, 2024 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant