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/#313] MainScreenTest 수정 #319

Merged
merged 5 commits into from
Jun 3, 2024

Conversation

l2hyunwoo
Copy link
Contributor

@l2hyunwoo l2hyunwoo commented Jun 1, 2024

Issue

Overview (Required)

TL;DR

Issues

  1. 원래 이슈가 발생되는 원인이 HiltAndroidRule이 적용되지 않은것으로 파악(Hilt 컴포넌트가 주입될 수 있는 액티비티가 없다고 하니..), HiltAndroidRule 적용하기 위한 DroidKnightsTestRunner 적용 + 기존 String Route 기반 테스트 Type 기반 테스트로 변경
  2. 적용후에 META-INF/LICENSE.md 패키지 옵션 이슈 발생해서 피쳐 모듈 전체 packaging option 추가
  3. Screen Test 도중 인터넷 퍼미션 에러 발생하여 :feature:main 모듈의 AndroidManifest에 internet permission 추가
  4. 그래도 Hilt 관련 에러 떠서 무슨 이슈인지 파악하기 위해 Now In Android 프로젝트 들어가보니 ui-test-hilt-manifest라는 모듈 발견해서 해당 모듈내에 HiltComponentActivity 클래스 발견.
  5. 해당 파일에 링크된 이슈를 보니 이미 NowInAndroid에서도 발생한 이슈였음. 드나 레포지터리에도 이를 적용하여 이슈 해결

Links

Screenshot

스크린샷 2024-06-01 오후 7 17 35

Copy link

github-actions bot commented Jun 1, 2024

Test Results

18 tests   18 ✅  5s ⏱️
11 suites   0 💤
11 files     0 ❌

Results for commit 3a362d9.

@taehwandev taehwandev merged commit 335bf23 into droidknights:main Jun 3, 2024
2 checks passed
@l2hyunwoo l2hyunwoo deleted the feature/#313 branch June 3, 2024 00:51
@taehwandev taehwandev added the 2024 label Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI 테스트] 네비게이션 테스트 수정
2 participants