chore: master hotfix를 develop에 sync (#694)

* fix: /applications 중복 활성 지원서 조회 시 500 오류 방지

- 활성 지원서 조회를 최신 1건 기준 메서드로 변경

- 지원서 제출 로직과 테스트 코드를 변경된 조회 메서드명에 맞춰 동기화

* fix: 활성 지원서 중복 문제의 근본 해결 TODO 명시

- 최신 1건 선택 로직이 임시 회피임을 주석으로 명확화

- DB 유일성 제약 도입 후 우회 조회 제거 필요사항을 기록

* hotfix: 대학 조회 캐싱 임시 제거

* hotfix: 대학 조회 캐싱 임시 제거

---------

Co-authored-by: Wibaek Park <34394229+devMuromi@users.noreply.github.com>
Co-authored-by: 이세원 <107756067+leesewon00@users.noreply.github.com>
Co-authored-by: Wibaek Park <34394229+wibaek@users.noreply.github.com>
Co-authored-by: Yeongseo Na <panda0329@naver.com>
Co-authored-by: Wibaek Park <devmuromi@gmail.com>
Co-authored-by: 황규혁 <126947828+Gyuhyeok99@users.noreply.github.com>
Co-authored-by: Gyuhyeok99 <ghkdrbgur13@naver.com>
Co-authored-by: Yeon <84384499+lsy1307@users.noreply.github.com>
Co-authored-by: hyungjun <115551339+sukangpunch@users.noreply.github.com>
Co-authored-by: 정재희 <y2hjjh@naver.com>
Co-authored-by: Yeonri <lsy1307@inha.edu>

Author: 8 people

src/main/java/com/example/solidconnection/application/repository/ApplicationRepository.java

@@ -27,17 +27,13 @@ public interface ApplicationRepository extends JpaRepository<Application, Long>
            """)
     List<Application> findAllByUnivApplyInfoIds(@Param("univApplyInfoIds") List<Long> univApplyInfoIds, @Param("status") VerifyStatus status, @Param("termId") long termId);
 
-    @Query("""
-           SELECT a
-           FROM Application a
-           WHERE a.siteUserId = :siteUserId
-               AND a.termId = :termId
-               AND a.isDelete = false
-           """)
-    Optional<Application> findBySiteUserIdAndTermId(@Param("siteUserId") long siteUserId, @Param("termId") long termId);
+    // TODO: 근본 해결 필요
+    // 지원서 유일성은 DB 제약으로 강제하고
+    // 이 조회는 임시 회피 로직을 제거하는 방향으로 수정 필요.
+    Optional<Application> findTopBySiteUserIdAndTermIdAndIsDeleteFalseOrderByIdDesc(long siteUserId, long termId);
 
     default Application getApplicationBySiteUserIdAndTermId(long siteUserId, long termId) {
-        return findBySiteUserIdAndTermId(siteUserId, termId)
+        return findTopBySiteUserIdAndTermIdAndIsDeleteFalseOrderByIdDesc(siteUserId, termId)
                 .orElseThrow(() -> new CustomException(APPLICATION_NOT_FOUND));
     }
 

src/main/java/com/example/solidconnection/application/service/ApplicationSubmissionService.java

@@ -62,7 +62,8 @@ public ApplicationSubmissionResponse apply(long siteUserId, ApplyRequest applyRe
         Long secondChoiceUnivApplyInfoId = univApplyInfoChoiceRequest.secondChoiceUnivApplyInfoId();
         Long thirdChoiceUnivApplyInfoId = univApplyInfoChoiceRequest.thirdChoiceUnivApplyInfoId();
 
-        Optional<Application> existingApplication = applicationRepository.findBySiteUserIdAndTermId(siteUser.getId(), term.getId());
+        Optional<Application> existingApplication =
+                applicationRepository.findTopBySiteUserIdAndTermIdAndIsDeleteFalseOrderByIdDesc(siteUser.getId(), term.getId());
         int updateCount = existingApplication
                 .map(application -> {
                     validateUpdateLimitNotExceed(application);

src/main/java/com/example/solidconnection/university/service/UnivApplyInfoQueryService.java

@@ -42,7 +42,8 @@ public UnivApplyInfoDetailResponse getUnivApplyInfoDetail(Long univApplyInfoId)
     }
 
     @Transactional(readOnly = true)
-    @ThunderingHerdCaching(key = "univApplyInfoTextSearch:{0}", cacheManager = "customCacheManager", ttlSec = 86400)
+    // todo: 현재 레디스 관련 에러 발생중으로 임시 주석처리, 추후 원인 분석 후 적용 필요
+    // @ThunderingHerdCaching(key = "univApplyInfoTextSearch:{0}", cacheManager = "customCacheManager", ttlSec = 86400)
     public UnivApplyInfoPreviewResponses searchUnivApplyInfoByText(String text) {
         Term term = termRepository.findByIsCurrentTrue()
                 .orElseThrow(() -> new CustomException(CURRENT_TERM_NOT_FOUND));

src/test/java/com/example/solidconnection/application/service/ApplicationSubmissionServiceTest.java

@@ -89,7 +89,9 @@ void setUp() {
         ApplicationSubmissionResponse response = applicationSubmissionService.apply(user.getId(), request);
 
         // then
-        Application savedApplication = applicationRepository.findBySiteUserIdAndTermId(user.getId(), term.getId()).orElseThrow();
+        Application savedApplication = applicationRepository
+                .findTopBySiteUserIdAndTermIdAndIsDeleteFalseOrderByIdDesc(user.getId(), term.getId())
+                .orElseThrow();
         assertAll(
                 () -> assertThat(response.totalApplyCount())
                         .isEqualTo(APPLICATION_UPDATE_COUNT_LIMIT),

src/test/java/com/example/solidconnection/university/service/UnivApplyInfoQueryServiceTest.java

@@ -192,7 +192,8 @@ class 각각의_검색_대상에_대해_검색한다 {
                     );
         }
 
-        @Test
+        // todo: 현재 레디스 관련 에러 발생중으로 임시 주석처리, 추후 원인 분석 후 적용 필요
+        //  @Test
         void 캐시가_적용된다() {
             // given
             String text = "Guam";