Merge pull request #182 from twenty-three-23/feature/TT-118-Redis-Set…

…ting TT-118 redis setting
twenty-three-23 · Jul 10, 2024 · e154e28 · e154e28
2 parents ff61449 + cd567ff
commit e154e28
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 5 deletions.
diff --git a/src/main/java/com/twentythree/peech/script/cache/CacheService.java b/src/main/java/com/twentythree/peech/script/cache/CacheService.java
@@ -14,6 +14,4 @@ public interface CacheService {
     List<Long> findAllByUserKey(String userKey);
 
     RedisSentenceDTO findByKey(Long sentenceId);
-
-    void setKeySentenceIdValueSentenceInformations(Long sentenceId, RedisSentenceDTO redisSentence);
 }
diff --git a/src/main/java/com/twentythree/peech/script/cache/RedisTemplateImpl.java b/src/main/java/com/twentythree/peech/script/cache/RedisTemplateImpl.java
@@ -57,5 +57,15 @@ public void saveSentenceInfo(Long sentenceId, RedisSentenceDTO redisSentence){
             throw new RuntimeException("Error saving redisSentence List: " + sentenceId);
         }
     }
+
+    @Override
+    public List<Long> findAllByUserKey(String userKey) {
+        return List.of();
+    }
+
+    @Override
+    public RedisSentenceDTO findByKey(Long sentenceId) {
+        return null;
+    }
 }
 
diff --git a/src/main/java/com/twentythree/peech/script/service/ScriptService.java b/src/main/java/com/twentythree/peech/script/service/ScriptService.java
@@ -2,6 +2,7 @@
 
 import com.twentythree.peech.common.dto.request.GPTRequest;
 import com.twentythree.peech.common.dto.response.GPTResponse;
+import com.twentythree.peech.script.cache.CacheService;
 import com.twentythree.peech.script.domain.*;
 import com.twentythree.peech.script.dto.*;
 import com.twentythree.peech.script.dto.response.MajorScriptsResponseDTO;
@@ -38,7 +39,7 @@ public class ScriptService {
     private final ThemeRepository themeRepository;
     private final VersionRepository versionRepository;
     private final SentenceRepository sentenceRepository;
-    private final ScriptInMemoryRepository scriptRedisRepository;
+    private final CacheService scriptRedisRepository;
 
     @Transactional
     public SaveScriptDTO saveInputScript(Long themeId, String[] paragraphs) {
@@ -206,11 +207,11 @@ public ModifyScriptResponseDTO modifyScriptService(List<ParagraphDTO> modifiedPa
                 Long newSentenceId = redisSentenceMap.getKey().getSentenceId();
                 RedisSentenceDTO newSentence = redisSentenceMap.getValue();
 
-                scriptRedisRepository.setKeySentenceIdValueSentenceInformations(newSentenceId, newSentence);
+                scriptRedisRepository.saveSentenceInfo(newSentenceId, newSentence);
                 newSentenceIds.add(newSentenceId);
 
             }
-            scriptRedisRepository.setKeyUserValueSentenceIds("user"+userId, newSentenceIds);
+            scriptRedisRepository.saveSentencesIdList("user"+userId, newSentenceIds);
 
         }
         return new ModifyScriptResponseDTO(modifiedParagraphList);

diff --git a/src/test/java/com/twentythree/peech/script/service/ScriptRedisTemplateImplTest.java b/src/test/java/com/twentythree/peech/script/service/ScriptRedisTemplateImplTest.java
@@ -2,6 +2,7 @@
 
 import com.twentythree.peech.script.cache.RedisTemplateImpl;
 import com.twentythree.peech.script.dto.SaveRedisSentenceInfoDto;
+import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.context.SpringBootTest;
@@ -39,6 +40,7 @@ public class ScriptRedisTemplateImplTest {
     }
 
     @Test
+    @Disabled
     public void 문장_정보_저장() throws Exception {
         List<SaveRedisSentenceInfoDto> sentencesInfoList = List.of(
                 new SaveRedisSentenceInfoDto(1L, 1L, 1L, "sentence1", 1L, LocalTime.of(0, 0, 12), false),