fix: Apply 상태값(JOINED) 의미 불일치 해결 및 checkApplyStatus 로직 개선 (#375)

* fix: 지원 상태 설명에서 JOINED 상태의 정의 수정

* refactor: ApplyStatusResponse에서 step 필드 및 관련 팩토리 메서드 제거

* refactor: 지원 상태 조회 로직 간소화 및 예외 처리 추가

* test: 지원 상태 조회 시 프로필 미작성 예외 처리 테스트 추가 및 기존 테스트 수정

Author: 023-dev

src/main/java/org/ject/support/domain/apply/controller/ApplyApiSpec.java

@@ -44,9 +44,9 @@ void submitApplication(@AuthPrincipal Long memberId,
             summary = "지원 상태 확인",
             description = """
                     지원자의 지원서 제출 여부를 확인합니다.
+                    - JOINED: 프로필을 저장한 경우
                     - TEMP_SAVED: 작성 중인 지원서가 있는 경우
                     - SUBMITTED: 이미 지원서를 제출한 경우
-                    - JOINED: 합격하여 팀에 합류한 경우
                     """)
     ApplyStatusResponse checkApplyStatus(@RequestParam @Email String email);
 

src/main/java/org/ject/support/domain/apply/dto/ApplyStatusResponse.java

@@ -1,25 +1,12 @@
 package org.ject.support.domain.apply.dto;
 
-import com.fasterxml.jackson.annotation.JsonInclude;
+import org.ject.support.domain.apply.domain.Apply;
 import org.ject.support.domain.apply.domain.Apply.Status;
 
-@JsonInclude(JsonInclude.Include.NON_NULL)
 public record ApplyStatusResponse(
-        Status status,
-        String step
+        Status status
 ) {
-
-    public static ApplyStatusResponse of(Status status) {
-        return new ApplyStatusResponse(status, null);
-    }
-
-    // 프로필 작성을 하지 않았을 경우
-    public static ApplyStatusResponse tempSavedProfile() {
-        return new ApplyStatusResponse(Status.TEMP_SAVED, "PROFILE");
-    }
-
-    // 프로필 작성 이후, 지원서 작성 중인 경우
-    public static ApplyStatusResponse tempSavedApply() {
-        return new ApplyStatusResponse(Status.TEMP_SAVED, "APPLY");
+    public static ApplyStatusResponse of(Apply apply) {
+        return new ApplyStatusResponse(apply.getStatus());
     }
 }

src/main/java/org/ject/support/domain/apply/service/ApplyService.java

@@ -174,27 +174,9 @@ public ApplyStatusResponse checkApplyStatus(String email) {
         Member member = memberRepository.findByEmail(email)
                 .orElseThrow(() -> new MemberException(MemberErrorCode.NOT_FOUND_MEMBER));
 
-        if (!member.isProfileComplete()) {
-            return ApplyStatusResponse.tempSavedProfile();
-        }
-        // 3. 지원 내역 조회
-        //    - 없으면: 임시 저장 지원 상태 반환
-        //    - 있으면: 상태 값에 따라 분기
-        Optional<Apply> optionalApply = applyRepository.findByMemberIdInActiveRecruit(member.getId(), LocalDateTime.now());
-
-        if (optionalApply.isEmpty()) {
-            return ApplyStatusResponse.tempSavedApply();
-        }
-
-        Apply apply = optionalApply.get();
-
-        // 4. 지원서가 임시 저장 상태인 경우
-        if (apply.isTempSaved()) {
-            return ApplyStatusResponse.tempSavedApply();
-        }
-
-        // 5. 그 외에는 실제 지원 상태 반환
-        return ApplyStatusResponse.of(apply.getStatus());
+        return applyRepository.findByMemberIdInActiveRecruit(member.getId(), LocalDateTime.now())
+                .map(ApplyStatusResponse::of)
+                .orElseThrow(() -> new ApplyException(NOT_FOUND_APPLY));
     }
 
     @Override

src/test/java/org/ject/support/domain/apply/service/ApplyServiceTest.java

@@ -191,19 +191,21 @@ class ApplyServiceTest extends UnitTestSupport {
     }
 
     @Test
-    void 지원단계중_프로필작성을_하지_않았을_경우_STEP을_PROFILE로_STATUS를_TEMP_SAVED_반환() {
+    void 지원상태_조회_시_프로필작성을_하지_않았을_경우_예외발생() {
         // given
         String email = "test@example.com";
         Member member = Member.builder()
                 .build();
         given(memberRepository.findByEmail(email))
                 .willReturn(Optional.of(member));
+        given(applyRepository.findByMemberIdInActiveRecruit(eq(member.getId()), any()))
+                .willReturn(Optional.empty());
 
-        // when
-        ApplyStatusResponse result = applyService.checkApplyStatus(email);
-
-        // then
-        assertThat(result).isEqualTo(ApplyStatusResponse.tempSavedProfile());
+        // expected
+        assertThatThrownBy(() -> applyService.checkApplyStatus(email))
+                .isInstanceOf(ApplyException.class)
+                .extracting("errorCode")
+                .isEqualTo(ApplyErrorCode.NOT_FOUND_APPLY);
     }
 
     @Test
@@ -229,7 +231,7 @@ class ApplyServiceTest extends UnitTestSupport {
         ApplyStatusResponse result = applyService.checkApplyStatus(email);
 
         // then
-        assertThat(result).isEqualTo(ApplyStatusResponse.tempSavedApply());
+        assertThat(result.status()).isEqualTo(TEMP_SAVED);
     }
 
     @Test
@@ -255,7 +257,7 @@ class ApplyServiceTest extends UnitTestSupport {
         ApplyStatusResponse result = applyService.checkApplyStatus(email);
 
         // then
-        assertThat(result).isEqualTo(ApplyStatusResponse.of(SUBMITTED));
+        assertThat(result.status()).isEqualTo(SUBMITTED);
     }
 
     @Test
@@ -562,23 +564,6 @@ class ApplyServiceTest extends UnitTestSupport {
         assertThat(member.getExperiencePeriod()).isEqualTo(request.experiencePeriod());
     }
 
-    @Test
-    void 지원상태확인_지원내역없음_신규지원가능() {
-        // given
-        String email = "re-apply@example.com";
-        Member member = getApplicant(1L, email);
-        member.edit(member.toEditor().name("Test").phoneNumber("010-0000-0000").build()); // 프로필 작성 완료 상태로
-
-        given(memberRepository.findByEmail(email)).willReturn(Optional.of(member));
-        given(applyRepository.findByMemberIdInActiveRecruit(eq(member.getId()), any())).willReturn(Optional.empty());
-
-        // when
-        ApplyStatusResponse result = applyService.checkApplyStatus(email);
-
-        // then
-        assertThat(result).isEqualTo(ApplyStatusResponse.tempSavedApply());
-    }
-
     private Recruit getActiveRecruit(JobFamily jobFamily, List<Question> questions) {
         return Recruit.builder()
                 .id(1L)