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

Step3 미션 제출합니다. #7

Open
wants to merge 4 commits into
base: Larry7939
Choose a base branch
from

Conversation

Larry7939
Copy link

3-1 단계 미션입니다.
수업 중 들은 내용으로는 안드로이드 테스트는 junit4만 지원하고 있어서, Junit5를 사용하려면 클래스를 직접 구현해줘야한다는 의미로 이해했는데... Junit5 클래스를 직접 구현해서 해결하는 미션이 맞을까요?!
제가 수행한 미션은 Junit4에 Truth를 사용한 방식인 것 같은데, 제대로 맞게 한 건지를 모르겠네요..ㅎ
매우 간단한 올바른_패키지_명을_반환한다 라는 테스트를 만들었습니다!

Copy link
Member

@malibinYun malibinYun left a comment

Choose a reason for hiding this comment

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

네! 말씀대로 안드로이드는 공식적으로 Junit4만을 지원합니다.
여러가지 단점으로 Junit5를 활용하는 게 더 좋다는 것을 알게되셨을거예요!
그래서 이 프로젝트에서는 Junit5를 안드로이드 테스트에 활용할 수 있는 환경을 세팅해두었어요.
이전에 수행한 미션은 모두 Junit5 + Truth를 활용 하셨습니다.

Junit5에서 만들어주어야하는 클래스는,
Junit4 만을 지원해주는 안드로이드에서 만든 테스트 Rule 등이에요.

Rule은 Extension으로 변경되었는데, (Junit4 -> Junit5)
안드로이드에서 Junit5를 공식 지원하지 않으니, Extension 클래스가 없습니다. 그래서 기존 Rule 객체를 보고 Extension 객체를 직접 마이그레이션 해야한다는 의미였어요 :)

다시 Junit5 + Truth로 테스트를 구성해보시고 리뷰 요청 주세요! 😊

Comment on lines 5 to -7
import org.junit.Test
import org.junit.runner.RunWith
Copy link
Member

Choose a reason for hiding this comment

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

org.junit.Test 는 Junit4이에요.
Junit5를 활용해서 테스트를 구성해주세요 :)

Comment on lines +16 to +17
//when
val actualResult = InstrumentationRegistry.getInstrumentation().targetContext
Copy link
Member

Choose a reason for hiding this comment

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

ApplicationProvider.getApplicationContext() 를 활용해볼 수 있어요 😊

Comment on lines +19 to 20
assertEquals("com.malibin.study.github", actualResult.packageName)
}
Copy link
Member

Choose a reason for hiding this comment

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

이 부분도 Truth를 활용해서 테스트를 검증해보세요 :)

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