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

kuring-101 FeedbackViewModel unit test 수정, MainDispatcherRule 추가 #86

Merged
merged 1 commit into from
Jan 24, 2024

Conversation

yeon-kyu
Copy link
Member

@yeon-kyu yeon-kyu commented Jan 22, 2024

kuring-101 작업의 연장선에서 FeedbackViewModelTest 가 fail 나는것을 수정합니다.

변경 사항

  • FeedbackViewModelTest 수정
  • Feedback Feature 모듈에 MainDispatcherRule 을 추가합니다. (테스트에 코루틴 dispatcher가 1개만 있도록 세팅해주는 rule)

추후 대응

testFixture 가 kotlin을 지원할수 있게 되면 MainDispatcherRule을 util 모듈로 이동합니다. (다른 방안으로는 test-util 모듈을 따로 만들거나 이 클래스만 util 모듈의 실제코드로 이동하여 해결하는것도 가능할듯 합니다)

@auto-assign auto-assign bot requested a review from mwy3055 January 22, 2024 14:46
Copy link
Member

@mwy3055 mwy3055 left a comment

Choose a reason for hiding this comment

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

아 이거군요
확인

@yeon-kyu yeon-kyu merged commit c5dcba3 into 2.0/kuring-101_yk Jan 24, 2024
1 check passed
@github-actions github-actions bot deleted the 2.0/kuring-101_test_yk branch April 8, 2024 12:18
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.

2 participants