🚀 2단계 - GitHub(HTTP) #105

KwonDae · 2023-08-31T15:49:46Z

안녕하세요 리뷰어님.
2단계 미션 리뷰 요청드립니다.

MockServer를 모듈로 생성해서 따로 관리할 경우는 테스트 범위가 넓을 때 사용할 수 있을까요?
언제 모듈로 관리해야 효율적인지 궁금합니다.

이번 미션 TODO

Repository 목록을 가져오는 기능은 data 모듈에 구현되어야 한다. 3b2fdcd, 0c87b22
data 모듈의 구현체는 모두 internal class로 선언한다. 3b2fdcd
HTTP 요청을 통해 가져오는 구현체에 대한 테스트 코드를 작성, OkHttp MockWebServer 이용. 75765ee

namjackson

안녕하세요 대원님!
2단계 고생하셨습니다!
잘작성해주셨네요!
몇가지 고민할법한 포인트들 코멘트로 남겼으니, 확인해주세요!
변경사항은 다음 미션 진행하면서 적용해주세요 😄

namjackson · 2023-09-01T12:58:51Z

data/src/main/java/camp/nextstep/edu/github/data/retrofit/NetworkResult.kt

+internal inline fun <T> networkResult(transform: () -> T): Result<T> = try {
+    Result.Success(transform.invoke())
+} catch (e: Exception) {
+    Result.Error(e)
+}


기존에 존재하는 runCatching 와 동일한 구조네요!
굳이 새로 만들필요는 없을거같아요!

namjackson · 2023-09-01T13:37:38Z

data/src/main/java/camp/nextstep/edu/github/data/retrofit/RetrofitNetwork.kt

+import retrofit2.Retrofit
+import retrofit2.converter.moshi.MoshiConverterFactory
+
+const val BASE_URL = "https://api.github.com/"


외부에서 노출되기보다 내부적으로만 활용되도 좋을거 같아요!

namjackson · 2023-09-01T13:42:31Z

data/src/main/java/camp/nextstep/edu/github/data/retrofit/RetrofitNetwork.kt

+
+const val BASE_URL = "https://api.github.com/"
+
+internal object RetrofitNetwork {


Retrofit이 아닌 다른 네트워크라이브러리를 사용한다면, 네이밍도 변경해야할까요?
좀더 추상적인 네이밍을 사용해도 좋을거같아요

namjackson · 2023-09-01T13:52:43Z

data/src/test/kotlin/camp/nextstep/edu/github/data/GithubServiceTest.kt

+    }
+
+    @Test
+    fun `test_json_요청`() = runTest {


의미있는 테스트 케이스를 작성해도 좋을거 같아요!

namjackson · 2023-09-01T13:56:13Z

data/src/test/kotlin/camp/nextstep/edu/github/data/GithubServiceTest.kt

+    }
+
+    @Test
+    fun `test_json_요청2`() = runTest {


중복되서 불필요한 테스트 같아요!
차라리 emptyList 같은 케이스를 추가해보면 어떨까요?

namjackson · 2023-09-01T13:58:13Z

data/src/test/kotlin/camp/nextstep/edu/github/data/GithubServiceTest.kt

+    @Test
+    fun `test_json_요청`() = runTest {
+        // given
+        val response = MockResponse().setBody(File("src/test/resources/repositories.json").readText())


src/test/resources/repositories/200.json src/test/resources/repositories/404.json src/test/resources/repositories/�emptyList.json

같이 테스트별로 여러 케이스의 json파일을 활용해봐도 좋아요!

jayden.d and others added 13 commits August 31, 2023 23:26

app 모듈은 domain 모듈에 의존해야 한다.

be88d43

step2 README 작성

fc477b5

domain 모듈 Model, Repository 생성

5c853e0

domain - 네트워크 결과 핸들링 로직 생성

d41d689

data - Response 생성

882e300

data - Service 생성

f8ca1cc

data - 기본 Retrofit, Moshi Converter 설정

0c87b22

data - RetrofitProvider 생성

3649fdc

data - 네트워크 에러 핸들링 로직 생성

6438cda

data - DefaultRepository 구현

3b2fdcd

data - mapper 생성

387b56b

data - MockWebServer를 이용한 HTTP 요청 테스트

75765ee

data - 구현체 internal

10bc813

namjackson approved these changes Sep 1, 2023

View reviewed changes

namjackson merged commit 67ec1bf into next-step:kwondae Sep 1, 2023
1 check passed

KwonDae deleted the step2 branch September 1, 2023 14:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 2단계 - GitHub(HTTP) #105

🚀 2단계 - GitHub(HTTP) #105

KwonDae commented Aug 31, 2023

namjackson left a comment

namjackson Sep 1, 2023

namjackson Sep 1, 2023

namjackson Sep 1, 2023

namjackson Sep 1, 2023

namjackson Sep 1, 2023

namjackson Sep 1, 2023


		const val BASE_URL = "https://api.github.com/"

		internal object RetrofitNetwork {

🚀 2단계 - GitHub(HTTP) #105

🚀 2단계 - GitHub(HTTP) #105

Conversation

KwonDae commented Aug 31, 2023

이번 미션 TODO

namjackson left a comment

Choose a reason for hiding this comment

namjackson Sep 1, 2023

Choose a reason for hiding this comment

namjackson Sep 1, 2023

Choose a reason for hiding this comment

namjackson Sep 1, 2023

Choose a reason for hiding this comment

namjackson Sep 1, 2023

Choose a reason for hiding this comment

namjackson Sep 1, 2023

Choose a reason for hiding this comment

namjackson Sep 1, 2023

Choose a reason for hiding this comment