refactor: Login 뷰에서 진행하는 네트워크 작업 코루틴으로 변경

android/app/src/main/java/com/now/naaga/data/remote/retrofit/service/AuthService.kt

@@ -2,13 +2,13 @@ package com.now.naaga.data.remote.retrofit.service
 
 import com.now.naaga.data.remote.dto.NaagaAuthDto
 import com.now.naaga.data.remote.dto.PlatformAuthDto
-import retrofit2.Call
+import retrofit2.Response
 import retrofit2.http.Body
 import retrofit2.http.POST
 
 interface AuthService {
     @POST("/auth")
-    fun requestAuth(
+    suspend fun requestAuth(
         @Body platformAuthDto: PlatformAuthDto,
-    ): Call<NaagaAuthDto>
+    ): Response<NaagaAuthDto>
 }

android/app/src/main/java/com/now/naaga/data/repository/DefaultAuthRepository.kt

@@ -4,22 +4,27 @@ import com.now.domain.model.PlatformAuth
 import com.now.domain.repository.AuthRepository
 import com.now.naaga.data.local.AuthDataSource
 import com.now.naaga.data.mapper.toDto
-import com.now.naaga.data.remote.retrofit.ServicePool
-import com.now.naaga.data.remote.retrofit.fetchResponse
+import com.now.naaga.data.remote.retrofit.ServicePool.authService
+import com.now.naaga.util.getValueOrThrow
+import kotlinx.coroutines.CoroutineDispatcher
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 
-class DefaultAuthRepository(private val authDataSource: AuthDataSource) : AuthRepository {
-    override fun getToken(
-        platformAuth: PlatformAuth,
-        callback: (Result<Boolean>) -> Unit,
-    ) {
-        val call = ServicePool.authService.requestAuth(platformAuth.toDto())
-        call.fetchResponse(
-            onSuccess = { naagaAuthDto ->
+class DefaultAuthRepository(
+    private val authDataSource: AuthDataSource,
+    private val dispatcher: CoroutineDispatcher = Dispatchers.IO
+) : AuthRepository {
+
+    override suspend fun getToken(platformAuth: PlatformAuth): Boolean {
+        return withContext(dispatcher) {
+            val response = authService.requestAuth(platformAuth.toDto())
+            runCatching {
+                val naagaAuthDto = response.getValueOrThrow()
                 authDataSource.setAccessToken(naagaAuthDto.accessToken)
                 authDataSource.setRefreshToken(naagaAuthDto.refreshToken)
-                callback(Result.success(true))
-            },
-            onFailure = { callback(Result.failure(it)) },
-        )
+                return@withContext true
+            }
+            return@withContext false
+        }
     }
 }

android/app/src/main/java/com/now/naaga/presentation/login/LoginViewModel.kt

@@ -3,10 +3,12 @@ package com.now.naaga.presentation.login
 import androidx.lifecycle.LiveData
 import androidx.lifecycle.MutableLiveData
 import androidx.lifecycle.ViewModel
+import androidx.lifecycle.viewModelScope
 import com.now.domain.model.AuthPlatformType
 import com.now.domain.model.PlatformAuth
 import com.now.domain.repository.AuthRepository
 import com.now.naaga.data.throwable.DataThrowable
+import kotlinx.coroutines.launch
 
 class LoginViewModel(
     private val authRepository: AuthRepository,
@@ -18,10 +20,14 @@ class LoginViewModel(
     val throwable: LiveData<DataThrowable> = _throwable
 
     fun signIn(token: String, platformType: AuthPlatformType) {
-        authRepository.getToken(PlatformAuth(token, platformType)) { result ->
-            result
-                .onSuccess { _isLoginSucceed.value = it }
-                .onFailure { _throwable.value = it as DataThrowable }
+        viewModelScope.launch {
+            runCatching {
+                authRepository.getToken(PlatformAuth(token, platformType))
+            }.onSuccess { status ->
+                _isLoginSucceed.value = status
+            }.onFailure {
+                _throwable.value = it as DataThrowable
+            }
         }
     }
 }

android/domain/src/main/java/com/now/domain/repository/AuthRepository.kt

@@ -3,8 +3,7 @@ package com.now.domain.repository
 import com.now.domain.model.PlatformAuth
 
 interface AuthRepository {
-    fun getToken(
+    suspend fun getToken(
         platformAuth: PlatformAuth,
-        callback: (Result<Boolean>) -> Unit,
-    )
+    ): Boolean
 }