test: 테스트 수정

Author: devmizz

src/main/kotlin/co/yappuworld/operation/client/application/GenerationActiveStateManager.kt

@@ -4,6 +4,7 @@ import co.yappuworld.global.exception.BusinessException
 import co.yappuworld.operation.client.dto.param.GenerationActivationControlResult
 import co.yappuworld.operation.domain.GenerationEntity
 import co.yappuworld.operation.domain.OperationError
+import co.yappuworld.operation.infrastructure.GenerationFindService
 import co.yappuworld.operation.infrastructure.GenerationRepository
 import io.github.oshai.kotlinlogging.KotlinLogging
 import org.springframework.data.repository.findByIdOrNull
@@ -14,6 +15,7 @@ private val logger = KotlinLogging.logger { }
 
 @Component
 class GenerationActiveStateManager(
+    private val generationFindService: GenerationFindService,
     private val generationRepository: GenerationRepository
 ) {
 
@@ -39,9 +41,9 @@ class GenerationActiveStateManager(
     fun getActiveGenerationOrNull(): Int? = generationRepository.getGenerationOrNullByIsActiveIsTrue()?.value
 
     private fun deactivateGeneration(targetGenerationValue: Int? = null): Int? {
-        if (!generationRepository.existsGenerationByIsActiveIsTrue()) return null
+        if (!generationFindService.existsActiveGeneration()) return null
 
-        val activeGenerations = generationRepository.findAllByIsActiveIsTrue()
+        val activeGenerations = generationFindService.findAllActiveGeneration()
         checkDeactivatingConsistency(activeGenerations, targetGenerationValue)
 
         return activeGenerations

src/main/kotlin/co/yappuworld/operation/infrastructure/GenerationFindService.kt

@@ -20,5 +20,21 @@ class GenerationFindService(
 
     fun existsGeneration(value: Int): Boolean = generationRepository.existsById(value)
 
+    fun existsActiveGeneration(): Boolean =
+        generationRepository
+            .findAll(limit = 1) {
+                select(intLiteral(1))
+                    .from(entity(GenerationEntity::class))
+                    .where(path(GenerationEntity::isActive).equal(true))
+            }.isNotEmpty()
+
     fun findGenerations(values: List<Int>): List<GenerationEntity> = generationRepository.findAllByValueIn(values)
+
+    fun findAllActiveGeneration(): List<GenerationEntity> =
+        generationRepository
+            .findAll {
+                select(entity(GenerationEntity::class))
+                    .from(entity(GenerationEntity::class))
+                    .where(path(GenerationEntity::isActive).equal(true))
+            }.filterNotNull()
 }

src/main/kotlin/co/yappuworld/operation/infrastructure/GenerationRepository.kt

@@ -1,11 +1,12 @@
 package co.yappuworld.operation.infrastructure
 
 import co.yappuworld.operation.domain.GenerationEntity
+import com.linecorp.kotlinjdsl.support.spring.data.jpa.repository.KotlinJdslJpqlExecutor
 import org.springframework.data.jpa.repository.JpaRepository
 
-interface GenerationRepository : JpaRepository<GenerationEntity, Int> {
-
-    fun existsGenerationByIsActiveIsTrue(): Boolean
+interface GenerationRepository :
+    JpaRepository<GenerationEntity, Int>,
+    KotlinJdslJpqlExecutor {
 
     fun getGenerationOrNullByIsActiveIsTrue(): GenerationEntity?
 

src/main/resources/application-local.yaml

@@ -16,13 +16,13 @@ spring:
 
     sql:
         init:
-            mode: never
+            mode: always
 
     jpa:
         open-in-view: false
         database-platform: org.hibernate.dialect.OracleDialect
         hibernate:
-            ddl-auto: create-drop
+            ddl-auto: none
         show-sql: true
 
     h2:

src/main/resources/application-test.yaml

@@ -17,7 +17,7 @@ spring:
     jpa:
         database-platform: org.hibernate.dialect.OracleDialect
         hibernate:
-            ddl-auto: create-drop
+            ddl-auto: none
         show-sql: true
 
 jwt:

src/main/resources/schema.sql

@@ -21,7 +21,7 @@ CREATE TABLE users
     role         varchar(16) NOT NULL,
     gender       varchar(8),
     phone_number varchar(16),
-    is_active    BOOLEAN  NOT NULL
+    is_active    NUMBER(1)  NOT NULL
 );
 
 DROP TABLE IF EXISTS sign_up_application;
@@ -54,8 +54,8 @@ CREATE TABLE user_alarm_settings
     created_at TIMESTAMP,
     updated_at TIMESTAMP,
     user_id    binary(16) NOT NULL,
-    device     BOOLEAN  NOT NULL,
-    master     BOOLEAN  NOT NULL
+    device     NUMBER(1)  NOT NULL,
+    master     NUMBER(1)  NOT NULL
 );
 
 DROP TABLE IF EXISTS user_devices;
@@ -74,18 +74,18 @@ CREATE TABLE schedules
     id           binary(16) PRIMARY KEY,
     created_at   TIMESTAMP,
     updated_at   TIMESTAMP,
-    is_deleted   BOOLEAN  NOT NULL,
+    is_deleted   NUMBER(1)  NOT NULL,
     name         varchar(32) NOT NULL,
     description  varchar(256),
     place        varchar(32),
     address      varchar(64),
     longitude    double,
     latitude     double,
-    date         date        NOT NULL,
-    end_date     date        NOT NULL,
-    time         varchar(32),
+    start_date   varchar(10) NOT NULL,
+    end_date     varchar(10) NOT NULL,
+    start_time   varchar(32),
     end_time     varchar(32),
-    is_all_day   BOOLEAN  NOT NULL,
+    is_all_day   NUMBER(1)  NOT NULL,
     generation   int,
     type         varchar(32) NOT NULL,
     session_type varchar(32)
@@ -104,17 +104,17 @@ CREATE TABLE posts
     content_summary varchar(255),
     display_target  varchar(255),
     writer_id       binary(16) NOT NULL,
-    is_active       BOOLEAN NOT NULL,
+    is_active       NUMBER(1) NOT NULL,
     session_id      binary(16)
 );
 
 DROP TABLE IF EXISTS generations;
 CREATE TABLE generations
 (
     value      int PRIMARY KEY,
-    start_date date,
-    end_date   date,
-    is_active  BOOLEAN  NOT NULL
+    start_date varchar(10),
+    end_date   varchar(10),
+    is_active  NUMBER(1)  NOT NULL
 );
 
 drop table if exists attendances;
@@ -156,8 +156,8 @@ CREATE TABLE team_services
     id              binary(16) PRIMARY KEY,
     created_at      TIMESTAMP,
     updated_at      TIMESTAMP,
-    has_app         BOOLEAN NOT NULL DEFAULT 0,
-    has_web         BOOLEAN NOT NULL DEFAULT 0,
+    has_app         NUMBER(1) NOT NULL DEFAULT 0,
+    has_web         NUMBER(1) NOT NULL DEFAULT 0,
     name            varchar(255) DEFAULT NULL,
     service_links   json DEFAULT NULL,
     team_id         binary(16) NOT NULL

src/test/kotlin/co/yappuworld/operation/client/application/GenerationActiveStateManagerTest.kt

@@ -2,6 +2,7 @@ package co.yappuworld.operation.client.application
 
 import co.yappuworld.global.exception.BusinessException
 import co.yappuworld.operation.domain.GenerationEntity
+import co.yappuworld.operation.infrastructure.GenerationFindService
 import co.yappuworld.operation.infrastructure.GenerationRepository
 import co.yappuworld.support.environment.CustomDataJpaTestFeatureSpec
 import io.kotest.assertions.throwables.shouldThrow
@@ -17,7 +18,8 @@ class GenerationActiveStateManagerTest @Autowired constructor(
     private val repository: GenerationRepository
 ) : CustomDataJpaTestFeatureSpec({
 
-        val generationActiveStateManager = GenerationActiveStateManager(repository)
+        val generationFindService = GenerationFindService(repository)
+        val generationActiveStateManager = GenerationActiveStateManager(generationFindService, repository)
 
         beforeEach {
             fun saveNotExist(generation: GenerationEntity) {