refactore/KD-50 Schedule title 필드 제거 및 일정 수정 API 수정 (#292)

dkdltm221 · web-flow · commit 22514001b05f · 2025-11-28T14:33:50.000+09:00
* refactore/KD-50 : Schedule title 필드 제거 및 일정 수정 API  수정

* refactore/KD-50 : 코드래빗 리뷰 반영
diff --git a/aics-admin/src/main/java/kgu/developers/admin/schedule/application/ScheduleAdminFacade.java b/aics-admin/src/main/java/kgu/developers/admin/schedule/application/ScheduleAdminFacade.java
@@ -22,7 +22,6 @@ public class ScheduleAdminFacade {
     public SchedulePersistResponse createSchedule(ScheduleCreateRequest request) {
         Long id = scheduleCommandService.createSchedule(
                 request.submissionType(),
-                request.title(),
                 request.content(),
                 request.startDate(),
                 request.endDate()
@@ -34,8 +33,6 @@ public void updateSchedule(Long scheduleId, ScheduleUpdateRequest request) {
         Schedule schedule = scheduleQueryService.getScheduleManagement(scheduleId);
         scheduleCommandService.updateSchedule(
                 schedule,
-                request.submissionType(),
-                request.title(),
                 request.startDate(),
                 request.endDate()
         );
diff --git a/aics-admin/src/main/java/kgu/developers/admin/schedule/presentation/ScheduleAdminController.java b/aics-admin/src/main/java/kgu/developers/admin/schedule/presentation/ScheduleAdminController.java
@@ -43,7 +43,7 @@ ResponseEntity<SchedulePersistResponse> createSchedule(
 	@ApiResponse(responseCode = "204")
 	ResponseEntity<Void> updateSchedule(
 			@Parameter(
-					description = "이 API는 제출 유형/제목/기간을 수정합니다.",
+					description = "이 API는 일정 별 기간을 수정합니다.",
 					example = "1",
 					required = true
 			)@Positive @PathVariable Long scheduleId,
diff --git a/aics-admin/src/main/java/kgu/developers/admin/schedule/presentation/request/ScheduleCreateRequest.java b/aics-admin/src/main/java/kgu/developers/admin/schedule/presentation/request/ScheduleCreateRequest.java
@@ -11,9 +11,6 @@ public record ScheduleCreateRequest(
         @Schema(description = "제출 유형", example = "SUBMITTED", requiredMode = REQUIRED)
         @NotNull SubmissionType submissionType,
 
-        @Schema(description = "일정 제목", example = "중간논문 제출 안내",requiredMode = REQUIRED)
-        @NotBlank @Size(max=100,message = "일정 제목은 100자 이내여야 합니다.") String title,
-
         @Schema(description = "일정 내용", example = "매학기 개강 후 2주 이내에 신청서를 작성하여 접수해야합니다.",requiredMode = REQUIRED)
         @NotBlank String content,
 
diff --git a/aics-admin/src/main/java/kgu/developers/admin/schedule/presentation/request/ScheduleUpdateRequest.java b/aics-admin/src/main/java/kgu/developers/admin/schedule/presentation/request/ScheduleUpdateRequest.java
@@ -10,13 +10,6 @@
 import static io.swagger.v3.oas.annotations.media.Schema.RequiredMode.REQUIRED;
 @Builder
 public record ScheduleUpdateRequest(
-        @Schema(description = "제출 유형", example = "SUBMITTED", requiredMode = REQUIRED)
-        @NotNull
-        SubmissionType submissionType,
-
-        @Schema(description = "일정 제목", example = "중간논문 제출 안내",requiredMode = REQUIRED)
-        @NotBlank @Size(max=100,message = "일정 제목은 100자 이내여야 합니다.")
-        String title,
 
 
         @Schema(description = "시작 일시", example = "2025-04-15T00:00:00", requiredMode = REQUIRED)
diff --git a/aics-admin/src/testFixtures/java/schedule/application/ScheduleAdminFacadeTest.java b/aics-admin/src/testFixtures/java/schedule/application/ScheduleAdminFacadeTest.java
@@ -46,7 +46,6 @@ public void init() {
 		Schedule savedSchedule = fakeScheduleRepository.save(
 			Schedule.create(
 				SUBMITTED,
-				"신청 접수",
 				"본문",
 				DEFAULT_START_DATE,
 				DEFAULT_END_DATE
@@ -61,7 +60,6 @@ void createSchedule_Success() {
 		// given
 		ScheduleCreateRequest request = ScheduleCreateRequest.builder()
 			.submissionType(MIDTHESIS)
-			.title("중간 논문 일정")
 			.content("중간 논문 본문")
 			.startDate(DEFAULT_START_DATE.plusMonths(1))
 			.endDate(DEFAULT_END_DATE.plusMonths(1))
@@ -81,7 +79,6 @@ void createSchedule_DuplicatedSubmissionType_ThrowsException() {
 		// given
 		ScheduleCreateRequest request = ScheduleCreateRequest.builder()
 			.submissionType(SUBMITTED)
-			.title("중복 일정")
 			.content("본문")
 			.startDate(DEFAULT_START_DATE)
 			.endDate(DEFAULT_END_DATE)
@@ -97,8 +94,6 @@ void createSchedule_DuplicatedSubmissionType_ThrowsException() {
 	void updateSchedule_Success() {
 		// given
 		ScheduleUpdateRequest request = ScheduleUpdateRequest.builder()
-			.submissionType(CERTIFICATE)
-			.title("수정 일정")
 			.startDate(DEFAULT_START_DATE.plusDays(3))
 			.endDate(DEFAULT_END_DATE.plusDays(5))
 			.build();
@@ -108,45 +103,15 @@ void updateSchedule_Success() {
 
 		// then
 		Schedule updated = fakeScheduleRepository.findById(savedScheduleId).orElseThrow();
-		assertEquals(CERTIFICATE, updated.getSubmissionType());
-		assertEquals("수정 일정", updated.getTitle());
 		assertEquals(DEFAULT_START_DATE.plusDays(3), updated.getStartDate());
 		assertEquals(DEFAULT_END_DATE.plusDays(5), updated.getEndDate());
 	}
 
-	@Test
-	@DisplayName("다른 일정의 제출 유형으로 수정 시 DuplicateScheduleTypeException을 발생시킨다")
-	void updateSchedule_DuplicatedSubmissionType_ThrowsException() {
-		// given
-		fakeScheduleRepository.save(
-			Schedule.create(
-				MIDTHESIS,
-				"다른 일정",
-				"본문",
-				DEFAULT_START_DATE.plusMonths(2),
-				DEFAULT_END_DATE.plusMonths(2)
-			)
-		);
-
-		ScheduleUpdateRequest request = ScheduleUpdateRequest.builder()
-			.submissionType(MIDTHESIS)
-			.title("중복 수정")
-			.startDate(DEFAULT_START_DATE)
-			.endDate(DEFAULT_END_DATE)
-			.build();
-
-		// when & then
-		assertThatThrownBy(() -> scheduleAdminFacade.updateSchedule(savedScheduleId, request))
-			.isInstanceOf(DuplicateScheduleTypeException.class);
-	}
-
 	@Test
 	@DisplayName("존재하지 않는 id로 일정 수정 시 ScheduleNotFoundException을 발생시킨다")
 	void updateSchedule_NotFound_ThrowsException() {
 		// given
 		ScheduleUpdateRequest request = ScheduleUpdateRequest.builder()
-			.submissionType(SUBMITTED)
-			.title("수정 실패")
 			.startDate(DEFAULT_START_DATE)
 			.endDate(DEFAULT_END_DATE)
 			.build();
diff --git a/aics-api/src/main/java/kgu/developers/api/schedule/presentation/response/ScheduleListResponse.java b/aics-api/src/main/java/kgu/developers/api/schedule/presentation/response/ScheduleListResponse.java
@@ -16,7 +16,6 @@ public record ScheduleListResponse(
         [{
             "id": 1,
             "submissionType": "MIDTHESIS",
-            "title": "중간논문 제출 안내",
             "startDate": "2025-05-01",
             "endDate": "2025-12-31",
             "status": "IN_PROGRESS"
diff --git a/aics-api/src/main/java/kgu/developers/api/schedule/presentation/response/ScheduleSummaryResponse.java b/aics-api/src/main/java/kgu/developers/api/schedule/presentation/response/ScheduleSummaryResponse.java
@@ -18,9 +18,6 @@ public record ScheduleSummaryResponse(
     @Schema(description = "제출 유형", example = "MIDTHESIS", requiredMode = REQUIRED)
     String submissionType,
 
-    @Schema(description = "일정 제목", example = "중간논문 제출 안내", requiredMode = REQUIRED)
-    String title,
-
     @Schema(description = "시작일", example = "2025-05-01", requiredMode = REQUIRED)
     @DateTimeFormat(pattern = "yyyy-MM-dd")
     String startDate,
@@ -38,7 +35,6 @@ public static ScheduleSummaryResponse from(Schedule schedule, LocalDateTime refe
         return ScheduleSummaryResponse.builder()
                 .id(schedule.getId())
                 .submissionType(schedule.getSubmissionType().name())
-                .title(schedule.getTitle())
                 .startDate(schedule.getStartDate().format(DATE_FORMATTER))
                 .endDate(schedule.getEndDate().format(DATE_FORMATTER))
                 .status(schedule.determineStatusAt(referenceTime).name())
diff --git a/aics-api/src/main/resources/db/data.sql b/aics-api/src/main/resources/db/data.sql
@@ -142,16 +142,16 @@ VALUES
     ('최동욱', 'dongwook@kyonggi.ac.kr', '이교수', '자격증', '2027-08-31', NULL, NULL, 2, '202512348', CURRENT_TIMESTAMP, CURRENT_TIMESTAMP);
 
 -- schedule
-INSERT INTO schedule (submission_type, title, content, start_date, end_date, created_at, updated_at)
-VALUES ('SUBMITTED', '신청 접수', '학부생 졸업 논문 신청을 접수합니다.', '2025-02-24 09:00:00', '2025-03-10 18:00:00',
+INSERT INTO schedule (submission_type,  content, start_date, end_date, created_at, updated_at)
+VALUES ('SUBMITTED',  '학부생 졸업 논문 신청을 접수합니다.', '2025-02-24 09:00:00', '2025-03-10 18:00:00',
         CURRENT_TIMESTAMP, CURRENT_TIMESTAMP),
-       ('MIDTHESIS', '중간 보고서', '중간 논문 제출 및 심사 기간입니다.', '2025-04-01 09:00:00', '2025-04-12 18:00:00',
+       ('MIDTHESIS',  '중간 논문 제출 및 심사 기간입니다.', '2025-04-01 09:00:00', '2025-04-12 18:00:00',
         CURRENT_TIMESTAMP, CURRENT_TIMESTAMP),
-       ('FINALTHESIS', '최종 보고서', '최종 논문 제출 및 심사 기간입니다.', '2025-05-20 09:00:00', '2025-06-05 18:00:00',
+       ('FINALTHESIS',  '최종 논문 제출 및 심사 기간입니다.', '2025-05-20 09:00:00', '2025-06-05 18:00:00',
         CURRENT_TIMESTAMP, CURRENT_TIMESTAMP),
-       ('CERTIFICATE', '제안서', '취득한 자격증 제출 및 심사 기간입니다.', '2025-03-15 09:00:00', '2025-03-29 18:00:00',
+       ('CERTIFICATE',  '취득한 자격증 제출 및 심사 기간입니다.', '2025-03-15 09:00:00', '2025-03-29 18:00:00',
         CURRENT_TIMESTAMP, CURRENT_TIMESTAMP),
-       ('APPROVED', '최종 통과', '최종 승인 기간입니다.', '2025-06-10 09:00:00', '2025-06-14 18:00:00',
+       ('APPROVED',  '최종 승인 기간입니다.', '2025-06-10 09:00:00', '2025-06-14 18:00:00',
         CURRENT_TIMESTAMP, CURRENT_TIMESTAMP),
-       ('OTHER', '기타 일정 안내', '기타 일정 입니다.', '2025-03-01 09:00:00', '2025-12-31 18:00:00',
+       ('OTHER',  '기타 일정 입니다.', '2025-03-01 09:00:00', '2025-12-31 18:00:00',
         CURRENT_TIMESTAMP, CURRENT_TIMESTAMP);
diff --git a/aics-api/src/main/resources/db/schema.sql b/aics-api/src/main/resources/db/schema.sql
@@ -173,7 +173,6 @@ CREATE TABLE schedule
         CONSTRAINT schedule_submission_type_check
             CHECK ((submission_type)::TEXT = ANY
         (ARRAY ['SUBMITTED', 'MIDTHESIS', 'FINALTHESIS', 'CERTIFICATE', 'APPROVED', 'OTHER'])),
-    title            VARCHAR(100) NOT NULL,
     content          TEXT         NOT NULL,
     start_date       TIMESTAMP(6) NOT NULL,
     end_date         TIMESTAMP(6) NOT NULL,
diff --git a/aics-domain/src/main/java/kgu/developers/domain/schedule/application/command/ScheduleCommandService.java b/aics-domain/src/main/java/kgu/developers/domain/schedule/application/command/ScheduleCommandService.java
@@ -17,24 +17,16 @@
 public class ScheduleCommandService {
     private final ScheduleRepository scheduleRepository;
 
-    public Long createSchedule(SubmissionType submissionType, String title, String content , LocalDateTime startDate, LocalDateTime endDate) {
+    public Long createSchedule(SubmissionType submissionType,String content , LocalDateTime startDate, LocalDateTime endDate) {
         scheduleRepository.findBySubmissionType(submissionType).ifPresent(existing -> {
             throw new DuplicateScheduleTypeException();
         });
-        Schedule schedule = Schedule.create(submissionType,title,content,startDate,endDate);
+        Schedule schedule = Schedule.create(submissionType,content,startDate,endDate);
 
         return scheduleRepository.save(schedule).getId();
     }
     @Transactional
-    public void updateSchedule(Schedule schedule, SubmissionType submissionType , String title, LocalDateTime startDate, LocalDateTime endDate) {
-        if(!schedule.getSubmissionType().equals(submissionType)) {
-            scheduleRepository.findBySubmissionType(submissionType)
-                    .filter(other -> !other.getId().equals(schedule.getId()))
-                    .ifPresent(existing -> {
-                        throw new DuplicateScheduleTypeException();});
-        }
-        schedule.updateSubmissionType(submissionType);
-        schedule.updateTitle(title);
+    public void updateSchedule(Schedule schedule,  LocalDateTime startDate, LocalDateTime endDate) {
         schedule.updateStartDate(startDate);
         schedule.updateEndDate(endDate);
 
diff --git a/aics-domain/src/main/java/kgu/developers/domain/schedule/domain/Schedule.java b/aics-domain/src/main/java/kgu/developers/domain/schedule/domain/Schedule.java
@@ -15,7 +15,6 @@ public class Schedule {
 
     private Long id;
     private SubmissionType submissionType;
-    private String title;
     private String content;
     private LocalDateTime startDate;
     private LocalDateTime endDate;
@@ -25,10 +24,9 @@ public class Schedule {
     private LocalDateTime deletedAt;
 
 
-    public static Schedule create(SubmissionType submissionType, String title,String content, LocalDateTime startDate, LocalDateTime endDate) {
+    public static Schedule create(SubmissionType submissionType,String content, LocalDateTime startDate, LocalDateTime endDate) {
         return Schedule.builder()
                 .submissionType(submissionType)
-                .title(title)
                 .content(content)
                 .startDate(startDate)
                 .endDate(endDate)
@@ -46,9 +44,6 @@ public ScheduleStatus determineStatusAt(LocalDateTime referenceTime) {
     public void updateSubmissionType(SubmissionType submissionType) {
         this.submissionType = submissionType;
     }
-    public void updateTitle(String title) {
-        this.title = title;
-    }
     public void updateContent(String content) {this.content = content;}
     public void updateStartDate(LocalDateTime startDate) {
         this.startDate = startDate;
diff --git a/aics-domain/src/main/java/kgu/developers/domain/schedule/infrastructure/entity/ScheduleJpaEntity.java b/aics-domain/src/main/java/kgu/developers/domain/schedule/infrastructure/entity/ScheduleJpaEntity.java
@@ -31,9 +31,6 @@ public class ScheduleJpaEntity extends BaseTimeEntity {
     @Enumerated(STRING)
     private SubmissionType submissionType;
 
-    @Column(nullable = false)
-    private String title;
-
     @Column(nullable = false, columnDefinition = "text")
     private String content;
 
@@ -51,7 +48,6 @@ public static ScheduleJpaEntity toEntity(final Schedule schedule) {
         return ScheduleJpaEntity.builder()
                 .id(schedule.getId())
                 .submissionType(schedule.getSubmissionType())
-                .title(schedule.getTitle())
                 .content(schedule.getContent())
                 .startDate(schedule.getStartDate())
                 .endDate(schedule.getEndDate())
@@ -61,7 +57,6 @@ public Schedule toDomain(){
         return Schedule.builder()
                 .id(id)
                 .submissionType(submissionType)
-                .title(title)
                 .startDate(startDate)
                 .content(content)
                 .endDate(endDate)
diff --git a/aics-domain/src/testFixtures/java/mock/repository/FakeScheduleRepository.java b/aics-domain/src/testFixtures/java/mock/repository/FakeScheduleRepository.java
@@ -22,7 +22,6 @@ public Schedule save(Schedule schedule) {
 		ScheduleJpaEntity entity = ScheduleJpaEntity.builder()
 			.id(id)
 			.submissionType(schedule.getSubmissionType())
-			.title(schedule.getTitle())
 			.content(schedule.getContent())
 			.startDate(schedule.getStartDate())
 			.endDate(schedule.getEndDate())
@@ -60,4 +59,9 @@ public Optional<Schedule> findBySubmissionType(SubmissionType submissionType) {
 				.findFirst()
 				.map(ScheduleJpaEntity :: toDomain);
 	}
+
+	@Override
+	public boolean existsById(Long id) {
+		return data.stream().anyMatch(schedule -> schedule.getId().equals(id));
+	}
 }
diff --git a/aics-domain/src/testFixtures/java/schedule/application/ScheduleCommandServiceTest.java b/aics-domain/src/testFixtures/java/schedule/application/ScheduleCommandServiceTest.java
@@ -33,7 +33,6 @@ void createSchedule_Success(){
         //given
         Long id = scheduleCommandService.createSchedule(
                 SubmissionType.SUBMITTED,
-                "신청 일정",
                 "신청 일정 본문",
                 startDate,
                 endDate
@@ -43,7 +42,6 @@ void createSchedule_Success(){
 
         //then
         assertEquals(1L,id);
-        assertEquals("신청 일정", saved.getTitle());
         assertEquals("신청 일정 본문", saved.getContent());
     }
 
@@ -52,13 +50,12 @@ void createSchedule_Success(){
     void createSchedule_DuplicateSubmissionType(){
         //given
         fakeScheduleRepository.save(
-                Schedule.create(SubmissionType.SUBMITTED,"기존 일정","내용",startDate,endDate)
+                Schedule.create(SubmissionType.SUBMITTED,"내용",startDate,endDate)
         );
         //when & then
         assertThrows(DuplicateScheduleTypeException.class, () ->
                 scheduleCommandService.createSchedule(
                         SubmissionType.SUBMITTED,
-                        "중복 일정",
                         "다른 내용",
                         startDate.plusDays(1),
                         endDate.plusDays(1)
@@ -67,57 +64,32 @@ void createSchedule_DuplicateSubmissionType(){
     }
 
     @Test
-    @DisplayName("updateSchedule은 제출타입과 기간, 제목을 갱신한다.")
+    @DisplayName("updateSchedule은 기간을 갱신한다.")
     void updateSchedule_Success(){
         //given
         Schedule schedule = fakeScheduleRepository.save(
-                Schedule.create(SubmissionType.SUBMITTED, "접수", "내용", startDate, endDate)
+                Schedule.create(SubmissionType.SUBMITTED, "접수",  startDate, endDate)
         );
         scheduleCommandService.updateSchedule(
                 schedule,
-                SubmissionType.MIDTHESIS,
-                "중간 점검",
                 startDate.plusWeeks(1),
                 endDate.plusWeeks(1)
         );
         //when
         Schedule updated = fakeScheduleRepository.findById(schedule.getId()).orElseThrow();
 
         //then
-        assertEquals(SubmissionType.MIDTHESIS, updated.getSubmissionType());
-        assertEquals("중간 점검", updated.getTitle());
         assertEquals(startDate.plusWeeks(1), updated.getStartDate());
 
     }
-    @Test
-    @DisplayName("이미 사용 중인 제출타입으로 updateSchedule을 호출하면 예외를 던진다")
-    void updateSchedule_DuplicateSubmissionTypeThrows() {
-        //given
-        Schedule first = fakeScheduleRepository.save(
-                Schedule.create(SubmissionType.SUBMITTED, "접수", "내용", startDate, endDate)
-        );
-        fakeScheduleRepository.save(
-                Schedule.create(SubmissionType.MIDTHESIS, "중간", "내용", startDate.plusDays(10), endDate.plusDays(10))
-        );
 
-        //when & then
-        assertThrows(DuplicateScheduleTypeException.class, () ->
-                scheduleCommandService.updateSchedule(
-                        first,
-                        SubmissionType.MIDTHESIS,
-                        "중간으로 변경",
-                        startDate.plusDays(1),
-                        endDate.plusDays(1)
-                )
-        );
-    }
 
     @Test
     @DisplayName("updateScheduleContent는 내용을 수정한다")
     void updateScheduleContent_Success() {
         //given
         Schedule schedule = fakeScheduleRepository.save(
-                Schedule.create(SubmissionType.SUBMITTED, "접수", "내용", startDate, endDate)
+                Schedule.create(SubmissionType.SUBMITTED, "내용", startDate, endDate)
         );
         //when
         scheduleCommandService.updateScheduleContent(schedule, "콘텐츠 업데이트");
@@ -131,7 +103,7 @@ void updateScheduleContent_Success() {
     void deleteSchedule_Success() {
         //given
         Schedule schedule = fakeScheduleRepository.save(
-                Schedule.create(SubmissionType.SUBMITTED, "접수", "내용", startDate, endDate)
+                Schedule.create(SubmissionType.SUBMITTED,  "내용", startDate, endDate)
         );
         //when
         scheduleCommandService.deleteSchedule(schedule.getId());
diff --git a/aics-domain/src/testFixtures/java/schedule/application/ScheduleQueryServiceTest.java b/aics-domain/src/testFixtures/java/schedule/application/ScheduleQueryServiceTest.java
diff --git a/aics-domain/src/testFixtures/java/schedule/domain/ScheduleDomainTest.java b/aics-domain/src/testFixtures/java/schedule/domain/ScheduleDomainTest.java
