refactor: 사용하지 않는 Repository 로직 제거

woowacourse-teams · Mar 29, 2024 · 92cae38 · 92cae38
1 parent db8721d
commit 92cae38
Show file tree

Hide file tree

Showing 9 changed files with 1 addition and 41 deletions.
diff --git a/backend/src/main/java/com/festago/festival/repository/FestivalInfoRepository.java b/backend/src/main/java/com/festago/festival/repository/FestivalInfoRepository.java
@@ -1,7 +1,6 @@
 package com.festago.festival.repository;
 
 import com.festago.festival.domain.FestivalQueryInfo;
-import java.util.List;
 import java.util.Optional;
 import org.springframework.data.repository.Repository;
 
@@ -12,7 +11,5 @@ public interface FestivalInfoRepository extends Repository<FestivalQueryInfo, Lo
     Optional<FestivalQueryInfo> findByFestivalId(Long festivalId);
 
     void deleteByFestivalId(Long festivalId);
-
-    List<FestivalQueryInfo> findAll();
 }
 
diff --git a/backend/src/main/java/com/festago/festival/repository/FestivalRepository.java b/backend/src/main/java/com/festago/festival/repository/FestivalRepository.java
@@ -3,7 +3,6 @@
 import com.festago.common.exception.ErrorCode;
 import com.festago.common.exception.NotFoundException;
 import com.festago.festival.domain.Festival;
-import java.util.List;
 import java.util.Optional;
 import org.springframework.data.repository.Repository;
 
@@ -17,10 +16,8 @@ default Festival getOrThrow(Long festivalId) {
     boolean existsBySchoolId(Long schoolId);
 
     Festival save(Festival festival);
-    
+
     Optional<Festival> findById(Long festivalId);
 
     void deleteById(Long festivalId);
-
-    List<Festival> findAll();
 }
diff --git a/backend/src/main/java/com/festago/stage/repository/StageArtistRepository.java b/backend/src/main/java/com/festago/stage/repository/StageArtistRepository.java
@@ -18,6 +18,4 @@ public interface StageArtistRepository extends Repository<StageArtist, Long> {
     Set<Long> findAllArtistIdByStageIdIn(@Param("stageIds") List<Long> stageIds);
 
     void deleteByStageId(Long stageId);
-
-    List<StageArtist> findAll();
 }
diff --git a/backend/src/main/java/com/festago/stage/repository/StageQueryInfoRepository.java b/backend/src/main/java/com/festago/stage/repository/StageQueryInfoRepository.java
@@ -1,7 +1,6 @@
 package com.festago.stage.repository;
 
 import com.festago.stage.domain.StageQueryInfo;
-import java.util.List;
 import java.util.Optional;
 import org.springframework.data.repository.Repository;
 
@@ -12,6 +11,4 @@ public interface StageQueryInfoRepository extends Repository<StageQueryInfo, Lon
     Optional<StageQueryInfo> findByStageId(Long stageId);
 
     void deleteByStageId(Long stageId);
-
-    List<StageQueryInfo> findAll();
 }
diff --git a/backend/src/main/java/com/festago/stage/repository/StageRepository.java b/backend/src/main/java/com/festago/stage/repository/StageRepository.java
@@ -18,7 +18,4 @@ public interface StageRepository extends Repository<Stage, Long>, StageRepositor
     boolean existsByFestivalId(Long festivalId);
 
     List<Stage> findAllByFestivalId(Long festivalId);
-
-    List<Stage> findAll();
-
 }
diff --git a/backend/src/test/java/com/festago/festival/repository/MemoryFestivalQueryInfoRepository.java b/backend/src/test/java/com/festago/festival/repository/MemoryFestivalQueryInfoRepository.java
@@ -2,8 +2,6 @@
 
 import com.festago.festival.domain.FestivalQueryInfo;
 import java.lang.reflect.Field;
-import java.util.ArrayList;
-import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
 import java.util.concurrent.ConcurrentHashMap;
@@ -41,9 +39,4 @@ public Optional<FestivalQueryInfo> findByFestivalId(Long festivalId) {
     public void deleteByFestivalId(Long festivalId) {
         memory.entrySet().removeIf(it -> Objects.equals(it.getValue().getFestivalId(), festivalId));
     }
-
-    @Override
-    public List<FestivalQueryInfo> findAll() {
-        return new ArrayList<>(memory.values());
-    }
 }
diff --git a/backend/src/test/java/com/festago/festival/repository/MemoryFestivalRepository.java b/backend/src/test/java/com/festago/festival/repository/MemoryFestivalRepository.java
@@ -2,8 +2,6 @@
 
 import com.festago.festival.domain.Festival;
 import java.lang.reflect.Field;
-import java.util.ArrayList;
-import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
 import java.util.concurrent.ConcurrentHashMap;
@@ -45,9 +43,4 @@ public Optional<Festival> findById(Long festivalId) {
     public void deleteById(Long festivalId) {
         memory.remove(festivalId);
     }
-
-    @Override
-    public List<Festival> findAll() {
-        return new ArrayList<>(memory.values());
-    }
 }
diff --git a/backend/src/test/java/com/festago/stage/repository/MemoryStageArtistRepository.java b/backend/src/test/java/com/festago/stage/repository/MemoryStageArtistRepository.java
@@ -4,7 +4,6 @@
 
 import com.festago.stage.domain.StageArtist;
 import java.lang.reflect.Field;
-import java.util.ArrayList;
 import java.util.List;
 import java.util.Objects;
 import java.util.Set;
@@ -52,9 +51,4 @@ public Set<Long> findAllArtistIdByStageIdIn(List<Long> stageIds) {
     public void deleteByStageId(Long stageId) {
         memory.entrySet().removeIf(entry -> Objects.equals(entry.getValue().getStageId(), stageId));
     }
-
-    @Override
-    public List<StageArtist> findAll() {
-        return new ArrayList<>(memory.values());
-    }
 }
diff --git a/backend/src/test/java/com/festago/stage/repository/MemoryStageRepository.java b/backend/src/test/java/com/festago/stage/repository/MemoryStageRepository.java
@@ -2,7 +2,6 @@
 
 import com.festago.stage.domain.Stage;
 import java.lang.reflect.Field;
-import java.util.ArrayList;
 import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
@@ -58,11 +57,6 @@ public List<Stage> findAllByFestivalId(Long festivalId) {
             .toList();
     }
 
-    @Override
-    public List<Stage> findAll() {
-        return new ArrayList<>(memory.values());
-    }
-
     @Override
     public List<Stage> findAllDetailByFestivalId(Long festivalId) {
         return findAllByFestivalId(festivalId);