diff --git a/backend/src/main/java/com/woowacourse/zzimkkong/domain/TimeSlot.java b/backend/src/main/java/com/woowacourse/zzimkkong/domain/TimeSlot.java index be4e78ad3..068d788c8 100644 --- a/backend/src/main/java/com/woowacourse/zzimkkong/domain/TimeSlot.java +++ b/backend/src/main/java/com/woowacourse/zzimkkong/domain/TimeSlot.java @@ -46,6 +46,10 @@ public static void validateStartEndTime(final LocalTime startTime, final LocalTi if (TimeUnit.cannotDivideByMinimumTimeUnit(startTime.getMinute())) { throw new IllegalTimeUnitValueException(startTime.getMinute()); } + + if (TimeUnit.cannotDivideByMinimumTimeUnit(endTime.getMinute())) { + throw new IllegalTimeUnitValueException(endTime.getMinute()); + } } public boolean isNotDivisibleBy(final TimeUnit timeUnit) { diff --git a/backend/src/test/java/com/woowacourse/zzimkkong/domain/TimeSlotTest.java b/backend/src/test/java/com/woowacourse/zzimkkong/domain/TimeSlotTest.java index be8b32cd3..3584d83df 100644 --- a/backend/src/test/java/com/woowacourse/zzimkkong/domain/TimeSlotTest.java +++ b/backend/src/test/java/com/woowacourse/zzimkkong/domain/TimeSlotTest.java @@ -26,7 +26,7 @@ void validateStartEndTime_endTimeEqualOrShorterThanStartTime(final LocalTime sta } @ParameterizedTest - @DisplayName("startTime이 5의 배수가 아니면 에러를 반환한다") + @DisplayName("startTime이나 endTime의 분 단위가 5의 배수가 아니면 에러를 반환한다") @MethodSource("provideStartAndEndTime_notDivisibleByMinimumTimeUnit") void validateStartEndTime_startTimeAndEndTimeNotDivisibleByMinimumTimeUnit(final LocalTime startTime, final LocalTime endTime) { assertThatThrownBy(() -> TimeSlot.validateStartEndTime(startTime, endTime)) @@ -107,9 +107,9 @@ private static Stream provideStartAndEndTime_endTimeEqualOrShorterTha private static Stream provideStartAndEndTime_notDivisibleByMinimumTimeUnit() { return Stream.of( - Arguments.of(LocalTime.of(10, 1), LocalTime.of(10, 5)), + Arguments.of(LocalTime.of(10, 0), LocalTime.of(10, 1)), Arguments.of(LocalTime.of(12, 4), LocalTime.of(12, 5)), - Arguments.of(LocalTime.of(13, 6), LocalTime.of(13, 10)) + Arguments.of(LocalTime.of(13, 6), LocalTime.of(13, 11)) ); }