From 59ba0874c99cb5024dd8ff75fc0a2013edd9f49a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=EC=B5=9C=EC=A4=80=ED=98=B8?= <junho5336@gmail.com>
Date: Thu, 19 Oct 2023 22:38:52 +0900
Subject: [PATCH] =?UTF-8?q?fix:=20=EC=84=B8=EC=85=98=20=EC=A1=B0=ED=9A=8C?=
 =?UTF-8?q?=20=EB=A1=9C=EC=A7=81=20=EB=B3=80=EA=B2=BD=20(#465)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../pium/repository/SessionGroupRepository.java      |  2 +-
 .../official/pium/service/SessionGroupService.java   |  4 ++--
 .../db/migration/V8__session_group_drop_unique.sql   |  1 +
 .../pium/repository/SessionGroupRepositoryTest.java  | 12 ++++++------
 4 files changed, 10 insertions(+), 9 deletions(-)
 create mode 100644 backend/pium/src/main/resources/db/migration/V8__session_group_drop_unique.sql

diff --git a/backend/pium/src/main/java/com/official/pium/repository/SessionGroupRepository.java b/backend/pium/src/main/java/com/official/pium/repository/SessionGroupRepository.java
index 3977aeb2..17adec16 100644
--- a/backend/pium/src/main/java/com/official/pium/repository/SessionGroupRepository.java
+++ b/backend/pium/src/main/java/com/official/pium/repository/SessionGroupRepository.java
@@ -10,5 +10,5 @@ public interface SessionGroupRepository extends JpaRepository<SessionGroup, Long
 
     void deleteBySessionValue(String sessionValue);
 
-    boolean existsBySessionIdAndSessionKey(String sessionId, String key);
+    boolean existsBySessionIdAndSessionValue(String sessionId, String key);
 }
diff --git a/backend/pium/src/main/java/com/official/pium/service/SessionGroupService.java b/backend/pium/src/main/java/com/official/pium/service/SessionGroupService.java
index 0991780c..47f68207 100644
--- a/backend/pium/src/main/java/com/official/pium/service/SessionGroupService.java
+++ b/backend/pium/src/main/java/com/official/pium/service/SessionGroupService.java
@@ -45,7 +45,7 @@ private void extendsSession(SessionGroup sessionGroup, LocalDateTime currentTime
 
     @Transactional
     public void add(String sessionId, String key, String value) {
-        if (sessionGroupRepository.existsBySessionIdAndSessionKey(sessionId, key)) {
+        if (sessionGroupRepository.existsBySessionIdAndSessionValue(sessionId, value)) {
             return;
         }
 
@@ -53,7 +53,7 @@ public void add(String sessionId, String key, String value) {
                 .sessionId(sessionId)
                 .sessionKey(key)
                 .sessionValue(value)
-                .expireTime(LocalDateTime.now().plusMinutes(EXTEND_EXPIRED_DAY))
+                .expireTime(LocalDateTime.now().plusDays(EXTEND_EXPIRED_DAY))
                 .build();
         sessionGroupRepository.save(sessionGroup);
     }
diff --git a/backend/pium/src/main/resources/db/migration/V8__session_group_drop_unique.sql b/backend/pium/src/main/resources/db/migration/V8__session_group_drop_unique.sql
new file mode 100644
index 00000000..7d5a7597
--- /dev/null
+++ b/backend/pium/src/main/resources/db/migration/V8__session_group_drop_unique.sql
@@ -0,0 +1 @@
+ALTER TABLE session_group DROP INDEX uniq_sessions;
diff --git a/backend/pium/src/test/java/com/official/pium/repository/SessionGroupRepositoryTest.java b/backend/pium/src/test/java/com/official/pium/repository/SessionGroupRepositoryTest.java
index dc83b241..6cd84620 100644
--- a/backend/pium/src/test/java/com/official/pium/repository/SessionGroupRepositoryTest.java
+++ b/backend/pium/src/test/java/com/official/pium/repository/SessionGroupRepositoryTest.java
@@ -88,22 +88,22 @@ class SessionGroupRepositoryTest extends RepositoryTest {
                 .sessionId("UUID.randomUUID().toString()")
                 .sessionKey("KAKAO_ID")
                 .sessionValue("1234546426")
-                .expireTime(LocalDateTime.now().plusMinutes(30))
+                .expireTime(LocalDateTime.now().plusDays(30))
                 .build();
         SessionGroup savedSessionGroup = sessionGroupRepository.save(sessionGroup);
 
-        boolean existsBySessionIdAndSessionKey = sessionGroupRepository.existsBySessionIdAndSessionKey(
+        boolean existsBySessionIdAndSessionValue = sessionGroupRepository.existsBySessionIdAndSessionValue(
                 savedSessionGroup.getSessionId(),
-                savedSessionGroup.getSessionKey()
+                savedSessionGroup.getSessionValue()
         );
 
-        assertThat(existsBySessionIdAndSessionKey).isTrue();
+        assertThat(existsBySessionIdAndSessionValue).isTrue();
     }
 
     @Test
     void 기존_세션이_존재하지않으면_False() {
-        boolean existsBySessionIdAndSessionKey = sessionGroupRepository.existsBySessionIdAndSessionKey(
-                "id", "sessionKey"
+        boolean existsBySessionIdAndSessionKey = sessionGroupRepository.existsBySessionIdAndSessionValue(
+                "id", "sessionValue"
         );
 
         assertThat(existsBySessionIdAndSessionKey).isFalse();