MongoService에서 coroutine 제거 (#113)

* MongoService CoroutineScope 제거

* Mock MongoService in test

Author: asp345

core/build.gradle.kts

@@ -20,8 +20,6 @@ dependencies {
 
     implementation("org.springframework.boot:spring-boot-starter-data-redis")
 
-    implementation("org.jetbrains.kotlinx:kotlinx-coroutines-core")
-
     runtimeOnly("com.mysql:mysql-connector-j")
     kapt("com.querydsl:querydsl-apt::jakarta")
 }

core/src/main/kotlin/com/wafflestudio/snuttev/core/domain/mongo/MongoService.kt

@@ -1,9 +1,6 @@
 package com.wafflestudio.snuttev.core.domain.mongo
 
 import com.wafflestudio.snuttev.core.domain.lecture.model.LectureRatingDao
-import kotlinx.coroutines.CoroutineScope
-import kotlinx.coroutines.Dispatchers
-import kotlinx.coroutines.launch
 import org.springframework.data.mongodb.core.MongoTemplate
 import org.springframework.data.mongodb.core.query.Criteria
 import org.springframework.data.mongodb.core.query.Query
@@ -15,15 +12,13 @@ class MongoService(
     private val mongoTemplate: MongoTemplate,
 ) {
     fun updateEvInfoToSnuttIds(snuttIds: List<String>, evInfo: LectureRatingDao?) =
-        CoroutineScope(Dispatchers.IO).launch {
-            runCatching {
-                mongoTemplate.updateMulti(
-                    Query(Criteria.where("_id").`in`(snuttIds)),
-                    Update().set("evInfo.evId", evInfo?.id)
-                        .set("evInfo.avgRating", evInfo?.avgRating)
-                        .set("evInfo.count", evInfo?.count),
-                    "lectures",
-                )
-            }
+        runCatching {
+            mongoTemplate.updateMulti(
+                Query(Criteria.where("_id").`in`(snuttIds)),
+                Update().set("evInfo.evId", evInfo?.id)
+                    .set("evInfo.avgRating", evInfo?.avgRating)
+                    .set("evInfo.count", evInfo?.count),
+                "lectures",
+            )
         }
 }

core/src/test/kotlin/com/wafflestudio/snuttev/core/domain/evaluation/service/EvaluationServiceTest.kt

@@ -14,13 +14,15 @@ import com.wafflestudio.snuttev.core.domain.lecture.model.Lecture
 import com.wafflestudio.snuttev.core.domain.lecture.model.SemesterLecture
 import com.wafflestudio.snuttev.core.domain.lecture.repository.LectureRepository
 import com.wafflestudio.snuttev.core.domain.lecture.repository.SemesterLectureRepository
+import com.wafflestudio.snuttev.core.domain.mongo.MongoService
 import org.assertj.core.api.Assertions.assertThat
 import org.assertj.core.api.Assertions.assertThatNoException
 import org.assertj.core.api.Assertions.assertThatThrownBy
 import org.junit.jupiter.api.BeforeEach
 import org.junit.jupiter.api.Test
 import org.springframework.beans.factory.annotation.Autowired
 import org.springframework.boot.test.context.SpringBootTest
+import org.springframework.boot.test.mock.mockito.MockBean
 import org.springframework.data.repository.findByIdOrNull
 import org.springframework.transaction.annotation.Transactional
 import kotlin.random.Random
@@ -34,6 +36,7 @@ class EvaluationServiceTest @Autowired constructor(
     private val lectureRepository: LectureRepository,
     private val semesterLectureRepository: SemesterLectureRepository,
     private val evaluationLikeRepository: EvaluationLikeRepository,
+    @MockBean private val mongoService: MongoService,
 ) {
     @BeforeEach
     fun setup() {