refs #130 テストフィクスチャをbuilder経由で作成する

averak · Jan 31, 2024 · 1ad4057 · 1ad4057
1 parent f8c4987
commit 1ad4057
Show file tree

Hide file tree

Showing 19 changed files with 253 additions and 89 deletions.
diff --git a/...rc/test/groovy/net/averak/gsync/adapter/handler/player_api/PlayerStorageHandler_IT.groovy b/...rc/test/groovy/net/averak/gsync/adapter/handler/player_api/PlayerStorageHandler_IT.groovy
@@ -1,52 +1,32 @@
 package net.averak.gsync.adapter.handler.player_api
 
-import net.averak.gsync.adapter.dao.dto.base.PlayerDto
-import net.averak.gsync.adapter.dao.dto.base.PlayerStorageEntryDto
-import net.averak.gsync.adapter.dao.dto.base.PlayerStorageRevisionDto
 import net.averak.gsync.adapter.handler.player_api.pbconv.PlayerStorageConverter
+import net.averak.gsync.core.game_context.GameContext
 import net.averak.gsync.domain.model.PlayerStorageEntry
 import net.averak.gsync.schema.protobuf.player_api.PlayerStorageClearV1
 import net.averak.gsync.schema.protobuf.player_api.PlayerStorageSearchV1
 import net.averak.gsync.schema.protobuf.player_api.PlayerStorageSetV1
 import net.averak.gsync.schema.protobuf.resource.player_storage.Criteria
 import net.averak.gsync.testkit.AbstractDatabaseSpec
 import net.averak.gsync.testkit.Faker
-import net.averak.gsync.testkit.Fixture
+import net.averak.gsync.testkit.fixture.builder.player.PlayerBuilder
+import net.averak.gsync.testkit.fixture.builder.player.PlayerStorageBuilder
 
 class PlayerStorageHandler_SearchV1_IT extends AbstractDatabaseSpec {
 
     def "正常系: パターンにマッチしたエントリを取得できる"() {
         given:
-        Fixture.setup(
-            Faker.fake(PlayerDto, [
-                "playerId": Faker.uuidv5("p1").toString(),
-            ]),
-        )
-        Fixture.setup(
-            Faker.fake(PlayerStorageRevisionDto, [
-                "playerId": Faker.uuidv5("p1").toString(),
-                "gameId"  : Faker.uuidv5("g1").toString(),
-            ]),
-        )
-        Fixture.setup(
-            Faker.fake(PlayerStorageEntryDto, [
-                "playerId": Faker.uuidv5("p1").toString(),
-                "gameId"  : Faker.uuidv5("g1").toString(),
-                "key"     : "group1#key1",
-                "value"   : "value1".bytes,
-            ]),
-            Faker.fake(PlayerStorageEntryDto, [
-                "playerId": Faker.uuidv5("p1").toString(),
-                "gameId"  : Faker.uuidv5("g1").toString(),
-                "key"     : "group1#key2",
-                "value"   : "value2".bytes,
-            ]),
-            Faker.fake(PlayerStorageEntryDto, [
-                "playerId": Faker.uuidv5("p1").toString(),
-                "gameId"  : Faker.uuidv5("g1").toString(),
-                "key"     : "group2#key1",
-                "value"   : "value3".bytes,
-            ]),
+        final gctx = Faker.fake(GameContext)
+        playerUp.setup(
+            gctx,
+            new PlayerBuilder(Faker.uuidv5("p1")).build(),
+            new PlayerStorageBuilder(Faker.uuidv5("p1"), Faker.uuidv5("g1"))
+                .entries(
+                    new PlayerStorageEntry("group1#key1", "value1".bytes),
+                    new PlayerStorageEntry("group1#key2", "value2".bytes),
+                    new PlayerStorageEntry("group2#key1", "value3".bytes),
+                )
+                .build(),
         )
 
         when:
@@ -60,7 +40,7 @@ class PlayerStorageHandler_SearchV1_IT extends AbstractDatabaseSpec {
                 ).build(),
         ) {
             it.session(Faker.uuidv5("p1"), Faker.uuidv5("g1"))
-            it.spoofingMasterVersion(Faker.uuidv4())
+            it.spoofingMasterVersion(gctx.masterVersion)
         }
 
         then:
@@ -73,17 +53,14 @@ class PlayerStorageHandler_SetV1_IT extends AbstractDatabaseSpec {
 
     def "正常系: パターンにマッチしたエントリを作成できる"() {
         given:
-        Fixture.setup(
-            Faker.fake(PlayerDto, [
-                "playerId": Faker.uuidv5("p1").toString(),
-            ]),
-        )
-        Fixture.setup(
-            Faker.fake(PlayerStorageRevisionDto, [
-                "playerId"               : Faker.uuidv5("p1").toString(),
-                "gameId"                 : Faker.uuidv5("g1").toString(),
-                "playerStorageRevisionId": Faker.uuidv5("r1").toString(),
-            ]),
+        final gctx = Faker.fake(GameContext)
+        playerUp.setup(
+            gctx,
+            new PlayerBuilder(Faker.uuidv5("p1")).build(),
+            new PlayerStorageBuilder(Faker.uuidv5("p1"), Faker.uuidv5("g1"))
+                .revision(Faker.uuidv5("r1"))
+                .entries()
+                .build(),
         )
 
         final entry = Faker.fake(PlayerStorageEntry)
@@ -97,7 +74,7 @@ class PlayerStorageHandler_SetV1_IT extends AbstractDatabaseSpec {
                 .build(),
         ) {
             it.session(Faker.uuidv5("p1"), Faker.uuidv5("g1"))
-            it.spoofingMasterVersion(Faker.uuidv4())
+            it.spoofingMasterVersion(gctx.masterVersion)
         }
 
         then:
@@ -120,37 +97,18 @@ class PlayerStorageHandler_ClearV1_IT extends AbstractDatabaseSpec {
 
     def "正常系: パターンにマッチしたエントリを削除できる"() {
         given:
-        Fixture.setup(
-            Faker.fake(PlayerDto, [
-                "playerId": Faker.uuidv5("p1").toString(),
-            ]),
-        )
-        Fixture.setup(
-            Faker.fake(PlayerStorageRevisionDto, [
-                "playerId"               : Faker.uuidv5("p1").toString(),
-                "gameId"                 : Faker.uuidv5("g1").toString(),
-                "playerStorageRevisionId": Faker.uuidv5("current revision").toString(),
-            ]),
-        )
-        Fixture.setup(
-            Faker.fake(PlayerStorageEntryDto, [
-                "playerId": Faker.uuidv5("p1").toString(),
-                "gameId"  : Faker.uuidv5("g1").toString(),
-                "key"     : "group1#key1",
-                "value"   : "value1".bytes,
-            ]),
-            Faker.fake(PlayerStorageEntryDto, [
-                "playerId": Faker.uuidv5("p1").toString(),
-                "gameId"  : Faker.uuidv5("g1").toString(),
-                "key"     : "group1#key2",
-                "value"   : "value2".bytes,
-            ]),
-            Faker.fake(PlayerStorageEntryDto, [
-                "playerId": Faker.uuidv5("p1").toString(),
-                "gameId"  : Faker.uuidv5("g1").toString(),
-                "key"     : "group2#key1",
-                "value"   : "value3".bytes,
-            ]),
+        final gctx = Faker.fake(GameContext)
+        playerUp.setup(
+            gctx,
+            new PlayerBuilder(Faker.uuidv5("p1")).build(),
+            new PlayerStorageBuilder(Faker.uuidv5("p1"), Faker.uuidv5("g1"))
+                .revision(Faker.uuidv5("current revision"))
+                .entries(
+                    new PlayerStorageEntry("group1#key1", "value1".bytes),
+                    new PlayerStorageEntry("group1#key2", "value2".bytes),
+                    new PlayerStorageEntry("group2#key1", "value3".bytes),
+                )
+                .build(),
         )
 
         when:
@@ -165,7 +123,7 @@ class PlayerStorageHandler_ClearV1_IT extends AbstractDatabaseSpec {
                 ).build(),
         ) {
             it.session(Faker.uuidv5("p1"), Faker.uuidv5("g1"))
-            it.spoofingMasterVersion(Faker.uuidv4())
+            it.spoofingMasterVersion(gctx.masterVersion)
         }
 
         then:

diff --git a/...et/averak/gsync/adapter/handler/player_api/interceptor/ClientVersionInterceptor_UT.groovy b/...et/averak/gsync/adapter/handler/player_api/interceptor/ClientVersionInterceptor_UT.groovy
@@ -8,7 +8,7 @@ import net.averak.gsync.schema.protobuf.player_api.EchoEchoV1
 import net.averak.gsync.testkit.AbstractDatabaseSpec
 import net.averak.gsync.testkit.Assert
 import net.averak.gsync.testkit.Faker
-import net.averak.gsync.testkit.Fixture
+import net.averak.gsync.testkit.fixture.Fixture
 import org.springframework.beans.factory.annotation.Autowired
 
 class ClientVersionInterceptor_UT extends AbstractDatabaseSpec {

diff --git a/...apter/src/test/groovy/net/averak/gsync/adapter/handler/utils/MasterVersionUtils_UT.groovy b/...apter/src/test/groovy/net/averak/gsync/adapter/handler/utils/MasterVersionUtils_UT.groovy
@@ -6,7 +6,7 @@ import net.averak.gsync.core.exception.GsyncException
 import net.averak.gsync.testkit.AbstractDatabaseSpec
 import net.averak.gsync.testkit.Assert
 import net.averak.gsync.testkit.Faker
-import net.averak.gsync.testkit.Fixture
+import net.averak.gsync.testkit.fixture.Fixture
 import org.springframework.beans.factory.annotation.Autowired
 
 class MasterVersionUtils_UT extends AbstractDatabaseSpec {

diff --git a/app/adapter/src/test/groovy/net/averak/gsync/adapter/repository/EchoRepository_UT.groovy b/app/adapter/src/test/groovy/net/averak/gsync/adapter/repository/EchoRepository_UT.groovy
@@ -6,7 +6,7 @@ import net.averak.gsync.domain.model.Echo
 import net.averak.gsync.testkit.AbstractDatabaseSpec
 import net.averak.gsync.testkit.Assert
 import net.averak.gsync.testkit.Faker
-import net.averak.gsync.testkit.Fixture
+import net.averak.gsync.testkit.fixture.Fixture
 import org.springframework.beans.factory.annotation.Autowired
 import spock.lang.Shared
 

diff --git a/app/adapter/src/test/groovy/net/averak/gsync/adapter/repository/FriendRepository_UT.groovy b/app/adapter/src/test/groovy/net/averak/gsync/adapter/repository/FriendRepository_UT.groovy
@@ -10,7 +10,7 @@ import net.averak.gsync.domain.repository.exception.MasterDataNotFoundException
 import net.averak.gsync.testkit.AbstractDatabaseSpec
 import net.averak.gsync.testkit.Assert
 import net.averak.gsync.testkit.Faker
-import net.averak.gsync.testkit.Fixture
+import net.averak.gsync.testkit.fixture.Fixture
 import org.springframework.beans.factory.annotation.Autowired
 
 import java.time.LocalDateTime

diff --git a/app/adapter/src/test/groovy/net/averak/gsync/adapter/repository/GameRepository_UT.groovy b/app/adapter/src/test/groovy/net/averak/gsync/adapter/repository/GameRepository_UT.groovy
@@ -5,7 +5,7 @@ import net.averak.gsync.core.game_context.GameContext
 import net.averak.gsync.domain.model.Game
 import net.averak.gsync.testkit.AbstractDatabaseSpec
 import net.averak.gsync.testkit.Faker
-import net.averak.gsync.testkit.Fixture
+import net.averak.gsync.testkit.fixture.Fixture
 import org.springframework.beans.factory.annotation.Autowired
 
 class GameRepository_UT extends AbstractDatabaseSpec {

diff --git a/app/adapter/src/test/groovy/net/averak/gsync/adapter/repository/OperatorRepository_UT.groovy b/app/adapter/src/test/groovy/net/averak/gsync/adapter/repository/OperatorRepository_UT.groovy
@@ -6,7 +6,7 @@ import net.averak.gsync.core.game_context.GameContext
 import net.averak.gsync.domain.model.Operator
 import net.averak.gsync.testkit.AbstractDatabaseSpec
 import net.averak.gsync.testkit.Faker
-import net.averak.gsync.testkit.Fixture
+import net.averak.gsync.testkit.fixture.Fixture
 import org.springframework.beans.factory.annotation.Autowired
 
 class OperatorRepository_UT extends AbstractDatabaseSpec {

diff --git a/app/adapter/src/test/groovy/net/averak/gsync/adapter/repository/PlayerRepository_UT.groovy b/app/adapter/src/test/groovy/net/averak/gsync/adapter/repository/PlayerRepository_UT.groovy
@@ -10,7 +10,7 @@ import net.averak.gsync.domain.model.PlayerProfile
 import net.averak.gsync.testkit.AbstractDatabaseSpec
 import net.averak.gsync.testkit.Assert
 import net.averak.gsync.testkit.Faker
-import net.averak.gsync.testkit.Fixture
+import net.averak.gsync.testkit.fixture.Fixture
 import org.springframework.beans.factory.annotation.Autowired
 import spock.lang.Shared
 

diff --git a/...ter/src/test/groovy/net/averak/gsync/adapter/repository/PlayerStorageRepository_UT.groovy b/...ter/src/test/groovy/net/averak/gsync/adapter/repository/PlayerStorageRepository_UT.groovy
@@ -10,7 +10,7 @@ import net.averak.gsync.domain.repository.IPlayerStorageRepository
 import net.averak.gsync.domain.repository.exception.AlreadyDoneException
 import net.averak.gsync.testkit.AbstractDatabaseSpec
 import net.averak.gsync.testkit.Faker
-import net.averak.gsync.testkit.Fixture
+import net.averak.gsync.testkit.fixture.Fixture
 import org.springframework.beans.factory.annotation.Autowired
 import spock.lang.Shared
 

diff --git a/testkit/src/main/groovy/net/averak/gsync/testkit/AbstractDatabaseSpec.groovy b/testkit/src/main/groovy/net/averak/gsync/testkit/AbstractDatabaseSpec.groovy
@@ -3,6 +3,9 @@ package net.averak.gsync.testkit
 import groovy.sql.Sql
 import jakarta.annotation.PostConstruct
 import net.averak.gsync.infrastructure.redis.RedisClient
+import net.averak.gsync.testkit.fixture.Fixture
+import net.averak.gsync.testkit.fixture.setupper.MasterUp
+import net.averak.gsync.testkit.fixture.setupper.PlayerUp
 import org.springframework.beans.factory.annotation.Autowired
 import org.springframework.boot.autoconfigure.EnableAutoConfiguration
 import spock.lang.Shared
@@ -18,6 +21,14 @@ abstract class AbstractDatabaseSpec extends AbstractSpec {
     @Shared
     RedisClient redis
 
+    @Autowired
+    @Shared
+    MasterUp masterUp
+
+    @Autowired
+    @Shared
+    PlayerUp playerUp
+
     @PostConstruct
     private void init() {
         Fixture.init(sql)

diff --git a/testkit/src/main/kotlin/net/averak/gsync/testkit/api/grpc/GrpcTester.kt b/testkit/src/main/kotlin/net/averak/gsync/testkit/api/grpc/GrpcTester.kt
@@ -4,6 +4,7 @@ import com.google.protobuf.AbstractMessage
 import io.grpc.*
 import jakarta.annotation.PostConstruct
 import net.averak.gsync.core.config.Config
+import net.averak.gsync.core.game_context.GameContext
 import net.averak.gsync.domain.model.Os
 import net.averak.gsync.infrastructure.grpc.player_api.metadata.IncomingHeaderKey
 import net.averak.gsync.schema.protobuf.player_api.EchoGrpc
@@ -74,6 +75,10 @@ class GrpcTester(
             metadata[IncomingHeaderKey.GAME_ID] = value.toString()
         }
 
+        fun idempotencyKey(value: UUID) {
+            metadata[IncomingHeaderKey.IDEMPOTENCY_KEY] = value.toString()
+        }
+
         fun spoofingPlayerID(value: UUID) {
             metadata[IncomingHeaderKey.DEBUG_SPOOFING_PLAYER_ID] = value.toString()
         }
@@ -85,6 +90,12 @@ class GrpcTester(
         fun spoofingCurrentTime(value: LocalDateTime) {
             metadata[IncomingHeaderKey.DEBUG_SPOOFING_CURRENT_TIME] = value.toString()
         }
+
+        fun gameContext(gctx: GameContext) {
+            idempotencyKey(gctx.idempotencyKey)
+            spoofingMasterVersion(gctx.masterVersion)
+            spoofingCurrentTime(gctx.currentTime)
+        }
     }
 
     private class MetadataInterceptor : ClientInterceptor {

diff --git a/...otlin/net/averak/gsync/testkit/Fixture.kt → ...t/averak/gsync/testkit/fixture/Fixture.kt b/...otlin/net/averak/gsync/testkit/Fixture.kt → ...t/averak/gsync/testkit/fixture/Fixture.kt
@@ -1,4 +1,4 @@
-package net.averak.gsync.testkit
+package net.averak.gsync.testkit.fixture
 
 import com.google.common.base.CaseFormat
 import groovy.sql.Sql
@@ -19,7 +19,7 @@ class Fixture {
          */
         @JvmStatic
         fun init(sql: Sql) {
-            Fixture.sql = sql
+            Companion.sql = sql
         }
 
         /**

diff --git a/...t/src/main/kotlin/net/averak/gsync/testkit/fixture/builder/master/FriendSettingBuilder.kt b/...t/src/main/kotlin/net/averak/gsync/testkit/fixture/builder/master/FriendSettingBuilder.kt
@@ -0,0 +1,29 @@
+package net.averak.gsync.testkit.fixture.builder.master
+
+import net.averak.gsync.domain.model.FriendSetting
+
+class FriendSettingBuilder(
+    private var data: FriendSetting,
+) {
+
+    constructor() : this(
+        FriendSetting(
+            maxFriendCount = 100,
+            maxFriendRequestCount = 100,
+        ),
+    )
+
+    fun maxFriendCount(maxFriendCount: Int): FriendSettingBuilder {
+        data = data.copy(maxFriendCount = maxFriendCount)
+        return this
+    }
+
+    fun maxFriendRequestCount(maxFriendRequestCount: Int): FriendSettingBuilder {
+        data = data.copy(maxFriendRequestCount = maxFriendRequestCount)
+        return this
+    }
+
+    fun build(): Registry {
+        return Registry(friendSetting = data)
+    }
+}
diff --git a/testkit/src/main/kotlin/net/averak/gsync/testkit/fixture/builder/master/Registry.kt b/testkit/src/main/kotlin/net/averak/gsync/testkit/fixture/builder/master/Registry.kt
@@ -0,0 +1,7 @@
+package net.averak.gsync.testkit.fixture.builder.master
+
+import net.averak.gsync.domain.model.FriendSetting
+
+data class Registry(
+    val friendSetting: FriendSetting? = null,
+)
diff --git a/testkit/src/main/kotlin/net/averak/gsync/testkit/fixture/builder/player/PlayerBuilder.kt b/testkit/src/main/kotlin/net/averak/gsync/testkit/fixture/builder/player/PlayerBuilder.kt
@@ -0,0 +1,33 @@
+package net.averak.gsync.testkit.fixture.builder.player
+
+import net.averak.gsync.domain.model.Player
+import net.averak.gsync.domain.model.PlayerProfile
+import net.averak.gsync.testkit.Faker
+import java.util.*
+
+class PlayerBuilder(
+    private var data: Player,
+) {
+
+    constructor(playerID: UUID) : this(
+        Player(
+            id = playerID,
+            friendID = UUID.randomUUID(),
+            isBanned = false,
+            profile = PlayerProfile(
+                nickname = Faker.alphanumeric(10),
+                iconID = Faker.uuidv4().toString(),
+            ),
+            login = null,
+        ),
+    )
+
+    fun profile(profile: PlayerProfile): PlayerBuilder {
+        data = data.copy(profile = profile)
+        return this
+    }
+
+    fun build(): Registry {
+        return Registry(player = data)
+    }
+}