Merge pull request #24 from Nexters/main

release: 0.0.3
Nexters · Feb 8, 2025 · 71da202 · 71da202
2 parents 6b0ae8f + 3d37760
commit 71da202
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/src/main/kotlin/me/misik/api/app/GetReviewFacade.kt b/src/main/kotlin/me/misik/api/app/GetReviewFacade.kt
@@ -1,5 +1,6 @@
 package me.misik.api.app
 
+import kotlinx.coroutines.delay
 import kotlinx.coroutines.runBlocking
 import kotlinx.coroutines.withTimeout
 import me.misik.api.core.GracefulShutdownDispatcher
@@ -16,8 +17,17 @@ class GetReviewFacade(
     fun getReview(id: Long): Review {
         return runBlocking(GracefulShutdownDispatcher.dispatcher) {
             withTimeout(60.seconds) {
-                reviewService.getReview(id)
-            }.get()
+                var result: Review? = null
+                while (result == null) {
+                    delay(500)
+                    reviewService.getById(id)
+                        .takeIf { it.isCompleted }
+                        .let {
+                            result = it
+                        }
+                }
+                return@withTimeout result!!
+            }
         }
     }
 }
diff --git a/src/main/kotlin/me/misik/api/domain/ReviewService.kt b/src/main/kotlin/me/misik/api/domain/ReviewService.kt
@@ -46,7 +46,4 @@ class ReviewService(
 
     fun getById(id: Long): Review = reviewRepository.findByIdOrNull(id)
         ?: throw IllegalArgumentException("Cannot find review by id \"$id\"")
-
-    fun getReview(id: Long) = reviewRepository.findById(id)
-        ?: throw IllegalArgumentException("Cannot find review by id \"$id\"")
 }