[1 - 3단계 방탈출 사용자 예약] 커찬(이충안) 미션 제출합니다. #2

leegwichan · 2024-05-02T08:45:12Z

주어진 3일동안 정신없이 하다 보니, 아직 생각하지 못한 부분도 많네요;;
요구사항을 만족하는 것도 시간에 쫓기면서 하다 보니, 부족한 점이 많을 수 있습니다. 리뷰 부탁드리겠습니다!

아래에는 제 Spring 경험, 신경 쓴 점과 신경 쓰지 못한 점을 생각나는 대로 정리해 보았습니다!

리뷰 링크

초기 코드를 제외한 변경 사항 PR 입니다!

링크

이전 Spring 경험

6개월 동안 Java/Spring을 배우는 부트캠프를 수료한 적이 있습니다. (2022.02 ~ 2022.08)
- 부트캠프 기간 중에 Spring을 이용하여 서비스를 배포해 본 경험이 있습니다.

신경 쓴 점

유효성 검사 위치

도메인을 최대한 안전하게 사용하기 위해서, 모든 유효성 검사를 도메인 객체에 위치시켰습니다. 처음에는 null 값이 들어오는 것 등은 DTO에 있어도 되지 않을까 생각했지만, 페어의 말에 설득 당해(?) 도메인에 위치시켰습니다. 이를 통해 도메인을 더 안전하게 사용할 수 있었습니다!

신경쓰지 못한 점

각 계층별 테스트

Service, Repository를 각각 테스트 해보려고 했었는데, 어떻게 테스트할지 고민하다가 구현 이후로 미뤘습니다.시간이 부족해 테스트를 해보지 못했습니다. 코멘트 이후에 한 번 해보도록 하겠습니다!

Co-authored-by: leegwichan <[email protected]>

…를 조회할 수 있다 Co-authored-by: leegwichan <[email protected]>

Co-authored-by: leegwichan <[email protected]>

Libienz

커찬 하이 👋

내가 잘 모르기도 해서 커찬에게 도움이 될 수 있는 코멘트를 남겼는지는 의문이지만 열심히 남겨보았다우

테스트 관련해서는 아직 작성하지 못한 것 같은데 나중에 꼭 커찬의 테스트 확인해보고 싶네
관련해서 같이 얘기도 나눠보고 싶고 👍

1차 제출 수고했으 👊👊
남은 미션 화이팅!

Libienz · 2024-05-03T04:23:52Z

src/main/java/roomescape/controller/ReservationController.java

    @DeleteMapping("/{id}")
-    public ResponseEntity<Void> deleteReservation(@PathVariable long id) {
+    public ResponseEntity<Void> deleteReservation(@PathVariable Long id) {
        service.deleteReservation(id);
        return ResponseEntity.noContent().build();
    }


id에 해당하는 예약을 삭제하는 요청에서 id를 Wrapper 클래스로 받았네!

널이 들어올 수 있다고 생각한 걸까?
그렇다면 그렇게 생각한 이유가 궁금!

null이 들어올 수 있다기보다는, 모든 곳에서 id의 타입을 Long으로 통일해서 사용해주고 싶었어!

Libienz · 2024-05-03T04:32:03Z

src/main/java/roomescape/dao/ThemeDao.java

+        String sql = """
+                SELECT theme.id, theme.name, theme.description, theme.thumbnail, COUNT(reservation.theme_id) AS reservation_count
+                FROM theme
+                JOIN reservation
+                ON reservation.theme_id = theme.id
+                WHERE reservation.date >= ? AND reservation.date <= ?
+                GROUP BY theme.id, theme.name, theme.description, theme.thumbnail
+                ORDER BY reservation_count DESC
+                LIMIT 10
+                """;


startDate과 endDate을 파라미터로써 동작하도록 할 수 있구나

나는 쿼리안에 모든 내용(일주일, Limit 10)이 들어있어

내 코드에서 맥락을 떼어내는 개선이 필요하다고 커찬의 코드를 통해 알게되었네 땡큐 ㅎ

나도 내 코드에서 10은 빼야 할 듯;;

Libienz · 2024-05-03T04:35:07Z

src/main/java/roomescape/domain/Name.java

+public record Name(String value) {
+    public Name {
+        Objects.requireNonNull(value, "인자 중 null 값이 존재합니다.");
+        if (value.isEmpty()) {
+            throw new IllegalArgumentException("이름은 한글자 이상이어야 합니다.");
+        }
+    }
+}


도메인에 레코드 사용하는 것 커찬은 괜찮다고 보는걸까?

원래의 레코드의 설계의의는 데이터 전송 객체를 위한 것이라고 들었어
커찬의 의견 궁금 👀

아 이정도는 사용해도 되지 않을까? 싶음.

만약에 Name에 로직을 실행하는 public method가 들어있다면, class를 고려해볼 것 같아.
추후 다른 로직이 추가된다면, class로 바꿔줄 것 같아!

Libienz · 2024-05-03T04:35:44Z

src/main/java/roomescape/domain/Reservation.java

+    public boolean isBefore(LocalDateTime currentDateTime) {
+        LocalDate currentDate = currentDateTime.toLocalDate();
+        if (date.isBefore(currentDate)) {
+            return true;
+        }
+        if (date.isAfter(currentDate)) {
+            return false;
+        }
+        return time.isBefore(currentDateTime.toLocalTime());
+    }


꼼꼼한 처리 굳 👍

Libienz · 2024-05-03T04:36:12Z

src/main/java/roomescape/domain/ReservationTime.java

+    public ReservationTime(LocalTime startAt) {
+        this.id = null;
+        this.startAt = Objects.requireNonNull(startAt, "인자 중 null 값이 존재합니다.");
+    }
+
+    public ReservationTime(Long id, LocalTime startAt) {
+        String errorMessage = "인자 중 null 값이 존재합니다.";
+        this.id = Objects.requireNonNull(id, errorMessage);
+        this.startAt = Objects.requireNonNull(startAt, errorMessage);
+    }


주생성자 부생성자 개념 적용해보면 좋을 듯!

그러고 싶었지만, 골치 아파지는 코드 떄문에 그럴 수 없었다... 코드 전반적으로 개선해보면서 할 듯!

Libienz · 2024-05-03T04:41:18Z

src/main/resources/schema.sql

+    name     VARCHAR(255) NOT NULL,
+    date     VARCHAR(255) NOT NULL,


date나 time은 varchar로 관리되어도 필요한 경우 변환이 스무스 하더라

하지만 데이터 무결성을 위해서 다른 타입으로 수정해보는 것이 좋을 듯!?

나도 그렇게 생각해. 이번 기회에 형식 바꿔봐야 겠다.

Libienz · 2024-05-03T04:42:49Z

src/main/java/roomescape/controller/AdminController.java

-public class LoadController {
+public class AdminController {


AdminController 충분히 좋은 이름일수도 있지만 다른 RestController와 다른 맥락이라는 의미에서 AdminViewController라는 이름은 어때?

Libienz · 2024-05-03T04:43:47Z

src/main/java/roomescape/controller/ReservationController.java

+    @ExceptionHandler({IllegalArgumentException.class, NullPointerException.class})
+    public ResponseEntity<ErrorResponse> handleException(RuntimeException e) {
+        return ResponseEntity.badRequest()
+                .body(new ErrorResponse(e.getMessage()));
+    }


전역적인 ExceptionHandler를 사용하지 않은 이유 궁금!
다른 컨트롤러에서도 똑같이 생긴 코드가 나올 수 밖에 없기에 전역적인 처리가 가질 수 있는 장점을 나는 높게 평가하는 편이야

이거... IllegalArgumentException.class, NullPointerException.class과 같이 공용적인 예외를 사용하는 상황에서는, 각 컨트롤러가 가지고 있는 것이 맞다고 생각했었어. 특정 부분에서는 내가 생각하지도 못하는 상황에서 예외가 터질 수 있으니까.

근데 이정도로 중복되면, 그냥 Advice로 묶는게 좋을 것 같아;;

Libienz · 2024-05-03T04:44:48Z

src/main/java/roomescape/controller/ThemeController.java

+    @GetMapping("/popular")
+    public ResponseEntity<List<ThemeResponse>> readPopularThemes() {
+        List<ThemeResponse> response = themeService.readPopularThemes();
+        return ResponseEntity.ok(response);
+    }


엔드포인트를 다음과 같이 설계한 크루도 있더라

/themes?show-popular=true
커찬은 어떤 생각을 가지고 API 설계했어?

기존에 REST API에 대해서 공부한 적이 있어서, /themes/popular로 설계했던 것 같아. (리소스를 명시하는 느낌으로)

그런데 정말로 내 엔드포인트가 REST API를 따랐다고는 볼 수 없을 것 같아. 그리고 /themes?show-popular=true가 그렇게 어색하다고 보기도 힘들 것 같아;; 오늘 수업 떄 말한 브리 말처럼 다른 사이트에서 어떻게 사용하는지 봐야 할 듯;;

Libienz · 2024-05-03T04:46:29Z

src/main/java/roomescape/controller/TimeController.java

+    @GetMapping("/available")
+    public ResponseEntity<List<AvailableTimeResponse>> readAvailableTimes(@RequestParam String date, @RequestParam Long themeId) {
+        List<AvailableTimeResponse> response = service.readAvailableTimes(date, themeId);
+        return ResponseEntity.ok(response);
    }


엔드포인트 설계에 대해서 궁금

available은 리소스를 의미하기 보다는 필터링 하는 조건을 의미하는 것 같은데
그렇다면 쿼리 파라미터가 좀 더 적절하지 않을까 하는 개인적인 생각!

/times?available=true

robinjoon · 2024-05-03T04:11:34Z

src/main/java/roomescape/dao/ReservationDao.java

                new ReservationTime(
                        resultSet.getLong("time_id"),
-                        resultSet.getString("start_at"))
+                        resultSet.getObject("start_at", LocalTime.class)),


이런 방법도 있구나!

robinjoon · 2024-05-03T05:17:01Z

src/main/java/roomescape/dto/ReservationResponse.java

+
+public record ReservationResponse(Long id,
+                                  String name,
+                                  @JsonFormat(pattern = "yyyy-MM-dd") LocalDate date,


JsonFormat 이 반복해서 등장하는데 전역으로 처리할 수 있는 방법을 찾아보는 것도 좋을 것 같아요!

싫어요.

목적은 각 record 응답마다 어떤 형식으로 변환하는지 명시해주기 위해서 그랬어! (전역으로 처리할 수 있는 방법이 있는지도 몰랐다는 사실을 비밀!)

해당 방법을 찾아보고, 한번 시도해보도록 할께! (근데 시간을 요청, 응답하는 객체가 더 많아진다면 확실이 고려해봐야 할 듯;;)

robinjoon · 2024-05-03T05:18:31Z

src/main/resources/schema.sql

+    time_id  BIGINT,
+    theme_id BIGINT,


요구사항 상 not null이 추가되어야 할 것 같아요!

엥 이거 왜 없지? 구현 끝나고 붙이려 했는데... TODO 써 놓을 걸 ㅠ_ㅠ

robinjoon · 2024-05-03T05:19:56Z

src/main/java/roomescape/controller/ThemeController.java

+    @ExceptionHandler({IllegalArgumentException.class, NullPointerException.class})
+    public ResponseEntity<ErrorResponse> handleException(RuntimeException e) {
+        return ResponseEntity.badRequest()
+                .body(new ErrorResponse(e.getMessage()));
+    }


중복된 코드로 보이는데 제거할 수 있을 것 같아요! @ControllerAdvice 등

이거... IllegalArgumentException.class, NullPointerException.class과 같이 공용적인 예외를 사용하는 상황에서는, 각 컨트롤러가 가지고 있는 것이 맞다고 생각했었어. 특정 부분에서는 내가 생각하지도 못하는 상황에서 예외가 터질 수 있으니까.

근데 이정도로 중복되면, 그냥 Advice로 묶는게 좋을 것 같아;;

robinjoon · 2024-05-03T05:20:56Z

src/test/java/roomescape/integrated/AdminIntegratedTest.java

+import org.springframework.boot.test.web.server.LocalServerPort;
+
+@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
+class AdminIntegratedTest {


Service 레이어나 dao 레이어의 테스트틑 일부러 작성하지 않은건가요?

이번 리뷰어에게 받은 테스트 관련 리뷰 요약

robinjoon · 2024-05-03T05:22:18Z

src/main/java/roomescape/dao/ReservationDao.java

-        long id = keyHolder.getKey().longValue();
-        return reservation.changeId(id);
+        Long id = keyHolder.getKey().longValue();
+        return readReservationById(id).orElseThrow();


long 대신 Long 으로 바꾼 이유가 있나요?

모든 id 값들은 전부 Long으로 통일하기 위해서!

long으로 사용하다 보니까 Long -> long 될 때 NullPointerException이 나는 게 별로였음

leegwichan

Hi 로빈, 리비!

일단은 생각 나는대로 즉흥적으로 답변 달아봤어!
혹시나 의문 사항이 생기면 즉시 연락해 줘!

leegwichan · 2024-05-03T06:09:51Z

src/main/java/roomescape/controller/ReservationController.java

+    @ExceptionHandler({IllegalArgumentException.class, NullPointerException.class})
+    public ResponseEntity<ErrorResponse> handleException(RuntimeException e) {
+        return ResponseEntity.badRequest()
+                .body(new ErrorResponse(e.getMessage()));
+    }


이거... IllegalArgumentException.class, NullPointerException.class과 같이 공용적인 예외를 사용하는 상황에서는, 각 컨트롤러가 가지고 있는 것이 맞다고 생각했었어. 특정 부분에서는 내가 생각하지도 못하는 상황에서 예외가 터질 수 있으니까.

근데 이정도로 중복되면, 그냥 Advice로 묶는게 좋을 것 같아;;

leegwichan · 2024-05-03T06:10:59Z

src/main/java/roomescape/controller/ThemeController.java

+    @ExceptionHandler({IllegalArgumentException.class, NullPointerException.class})
+    public ResponseEntity<ErrorResponse> handleException(RuntimeException e) {
+        return ResponseEntity.badRequest()
+                .body(new ErrorResponse(e.getMessage()));
+    }


이거... IllegalArgumentException.class, NullPointerException.class과 같이 공용적인 예외를 사용하는 상황에서는, 각 컨트롤러가 가지고 있는 것이 맞다고 생각했었어. 특정 부분에서는 내가 생각하지도 못하는 상황에서 예외가 터질 수 있으니까.

근데 이정도로 중복되면, 그냥 Advice로 묶는게 좋을 것 같아;;

leegwichan · 2024-05-03T06:14:05Z

src/main/java/roomescape/controller/ThemeController.java

+    @GetMapping("/popular")
+    public ResponseEntity<List<ThemeResponse>> readPopularThemes() {
+        List<ThemeResponse> response = themeService.readPopularThemes();
+        return ResponseEntity.ok(response);
+    }


기존에 REST API에 대해서 공부한 적이 있어서, /themes/popular로 설계했던 것 같아. (리소스를 명시하는 느낌으로)

그런데 정말로 내 엔드포인트가 REST API를 따랐다고는 볼 수 없을 것 같아. 그리고 /themes?show-popular=true가 그렇게 어색하다고 보기도 힘들 것 같아;; 오늘 수업 떄 말한 브리 말처럼 다른 사이트에서 어떻게 사용하는지 봐야 할 듯;;

leegwichan · 2024-05-03T06:14:46Z

src/main/java/roomescape/dao/ThemeDao.java

+        String sql = """
+                SELECT theme.id, theme.name, theme.description, theme.thumbnail, COUNT(reservation.theme_id) AS reservation_count
+                FROM theme
+                JOIN reservation
+                ON reservation.theme_id = theme.id
+                WHERE reservation.date >= ? AND reservation.date <= ?
+                GROUP BY theme.id, theme.name, theme.description, theme.thumbnail
+                ORDER BY reservation_count DESC
+                LIMIT 10
+                """;


나도 내 코드에서 10은 빼야 할 듯;;

leegwichan · 2024-05-03T06:16:32Z

src/main/java/roomescape/domain/Name.java

+public record Name(String value) {
+    public Name {
+        Objects.requireNonNull(value, "인자 중 null 값이 존재합니다.");
+        if (value.isEmpty()) {
+            throw new IllegalArgumentException("이름은 한글자 이상이어야 합니다.");
+        }
+    }
+}


아 이정도는 사용해도 되지 않을까? 싶음.

만약에 Name에 로직을 실행하는 public method가 들어있다면, class를 고려해볼 것 같아.
추후 다른 로직이 추가된다면, class로 바꿔줄 것 같아!

leegwichan · 2024-05-03T06:19:17Z

src/main/java/roomescape/dto/ReservationResponse.java

+
+public record ReservationResponse(Long id,
+                                  String name,
+                                  @JsonFormat(pattern = "yyyy-MM-dd") LocalDate date,


싫어요.

목적은 각 record 응답마다 어떤 형식으로 변환하는지 명시해주기 위해서 그랬어! (전역으로 처리할 수 있는 방법이 있는지도 몰랐다는 사실을 비밀!)

해당 방법을 찾아보고, 한번 시도해보도록 할께! (근데 시간을 요청, 응답하는 객체가 더 많아진다면 확실이 고려해봐야 할 듯;;)

leegwichan · 2024-05-03T06:20:05Z

src/main/java/roomescape/service/ThemeService.java

+        LocalDate currentDate = LocalDate.now();
+        LocalDate startDate = currentDate.minusDays(7);
+        LocalDate endDate = currentDate.minusDays(1);


안그래도 해당 피드백을 리뷰어한테 받았어! 바쁘게 구현하느라 미처 생각 못함;;
상수 추출 해보도록 할께!

leegwichan · 2024-05-03T06:20:59Z

src/main/resources/schema.sql

+    name     VARCHAR(255) NOT NULL,
+    date     VARCHAR(255) NOT NULL,


나도 그렇게 생각해. 이번 기회에 형식 바꿔봐야 겠다.

leegwichan · 2024-05-03T06:21:33Z

src/main/resources/schema.sql

+    time_id  BIGINT,
+    theme_id BIGINT,


엥 이거 왜 없지? 구현 끝나고 붙이려 했는데... TODO 써 놓을 걸 ㅠ_ㅠ

leegwichan · 2024-05-03T06:23:33Z

src/test/java/roomescape/integrated/AdminIntegratedTest.java

+import org.springframework.boot.test.web.server.LocalServerPort;
+
+@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
+class AdminIntegratedTest {


이번 리뷰어에게 받은 테스트 관련 리뷰 요약

kyum-q and others added 30 commits April 30, 2024 13:36

migrate first mission code

e53540e

docs(README): 1단계 기능 요구사항 작성

6bcc1f8

Co-authored-by: leegwichan <[email protected]>

refactor(Reservation): date를 LocalDate 타입으로 변환

94fce36

Co-authored-by: leegwichan <[email protected]>

feat(Reservation): 날짜가 형식에 맞지 않는 경우, 예외를 던진다.

c39da94

Co-authored-by: leegwichan <[email protected]>

feat(Reservation): 날짜/시간id/예약자명 중 하나라도 비어있을 때 예외를 던진다.

9dd6082

Co-authored-by: leegwichan <[email protected]>

feat(ReservationTime): 시간이 형식이 맞지 않을 때, 예외를 던진다

0f486ca

Co-authored-by: leegwichan <[email protected]>

feat(Reservation): 지나간 시간대에 예약 생성이 불가하도록 구현

5e619bc

Co-authored-by: leegwichan <[email protected]>

feat(ReservationService): 해당 시간id가 시간 테이블에 없을 때 예외를 던진다

6969681

Co-authored-by: leegwichan <[email protected]>

style(ReservationControllerTest): 사용하지 않는 import문 제거

59e8ced

Co-authored-by: leegwichan <[email protected]>

feat(TimeService): 특정 시간에 대한 예약이 존재하는데, 그 시간을 삭제하려 할 때, 예외를 던진다

2894382

Co-authored-by: leegwichan <[email protected]>

fix(TimeService): 직렬화 에러 및 null 참조 에러 해결

816fd09

Co-authored-by: leegwichan <[email protected]>

feat(TimeService): 이미 존재하는 시간을 추가할 경우, 예외를 던진다

67615d2

Co-authored-by: leegwichan <[email protected]>

feat(ReservationService): 이미 존재하는 시간대에 예약을 추가할 경우, 예외를 던진다

0461ad0

Co-authored-by: leegwichan <[email protected]>

feat(ExceptionHandler): 예외가 발생할 경우, 400 Bad Request를 응답

bd8f9df

Co-authored-by: leegwichan <[email protected]>

feat(Reservation, ReservationTime): 도메인 객체 생성자 추가

34e77a5

Co-authored-by: leegwichan <[email protected]>

fix(ReservationService): 현재 시간 이후에 예약할 수 있도록 변경

4885469

Co-authored-by: leegwichan <[email protected]>

refactor(ReservationCreateRequest): id 타입을 long -> Long로 변환

dace2fe

Co-authored-by: leegwichan <[email protected]>

docs(README): 2단계 기능 요구 사항 작성

8ff1c18

Co-authored-by: leegwichan <[email protected]>

feat(schema.sql): DB DDL 변경

3a2bd95

Co-authored-by: leegwichan <[email protected]>

feat(LoadController): 테마 관리 페이지 응답

ce7d63f

Co-authored-by: leegwichan <[email protected]>

feat(Theme): 테마 조회 기능 구현

69ed1b8

Co-authored-by: leegwichan <[email protected]>

feat(Theme): 테마 생성 기능 구현

ca4f84b

Co-authored-by: leegwichan <[email protected]>

refactor(Controller): Created 응답 리팩터링

a87c72e

Co-authored-by: leegwichan <[email protected]>

feat(Theme): 테마 삭제 기능 구현

b83cf51

Co-authored-by: leegwichan <[email protected]>

refactor(id): long 타입을 Long 타입으로 변환

9fd8e48

Co-authored-by: leegwichan <[email protected]>

feat(Theme): id를 제외한 인자 중 null이 있으면 예외를 던진다

c4686fd

Co-authored-by: leegwichan <[email protected]>

feat(ThemeService): 중복된 이름의 테마 생성 불가

ee088fb

Co-authored-by: leegwichan <[email protected]>

feat(ThemeService): 이미 예약된 테마 삭제 불가

1d54328

Co-authored-by: leegwichan <[email protected]>

feat(Reservation): Reservation에 Theme 추가

f52e195

Co-authored-by: leegwichan <[email protected]>

fix(ReservationResponse): 예약 응답에 테마 추가

88297e6

Co-authored-by: leegwichan <[email protected]>

kyum-q and others added 19 commits May 1, 2024 18:27

feat(LoadController): 방탈출 예약 페이지 변경

662af18

Co-authored-by: leegwichan <[email protected]>

style(Dao): 메서드 명 변경

fcede7d

Co-authored-by: leegwichan <[email protected]>

refactor(Service): 도매인 객체 대신 Response 객체 반환

a046959

Co-authored-by: leegwichan <[email protected]>

docs(README): 3단계 기능 요구 사항 작성

a72799b

Co-authored-by: leegwichan <[email protected]>

feat(UserController): 사용자 페이지 응답 구현

aece0f9

Co-authored-by: leegwichan <[email protected]>

feat(ReservationService): 중복 예약(시간, 날짜와 테마가 동일할 때)은 불가

8853af8

Co-authored-by: leegwichan <[email protected]>

feat(TimeController): 날짜와 테마가 주어지면, 해당 날짜에 예약 가능 시간 조회 기능 구현

823ede2

Co-authored-by: leegwichan <[email protected]>

feat(user-reservation.js): 날짜 테마 선택에 따른 가능한 시간만 선택되도록 수정

dd71878

Co-authored-by: leegwichan <[email protected]>

style(TimeService): 변수명 변경

4fdbf9e

Co-authored-by: leegwichan <[email protected]>

feat(TimeController): 현재 날짜 이전 일주일을 기준으로, 해당 기간 내에 방문하는 예약이 많은 테마 10개…

430b82e

…를 조회할 수 있다 Co-authored-by: leegwichan <[email protected]>

feat(ranking.js): 인기 테마 목록 조회 API 호출 후 렌더링

cec843e

Co-authored-by: leegwichan <[email protected]>

feat(Name): 이름은 빈칸이거나 null일 수 없다

888953e

Co-authored-by: leegwichan <[email protected]>

style: 클래스 공백 기준 통일 및 사용하지 않는 import문 제거

b9fa540

Co-authored-by: leegwichan <[email protected]>

refactor(ReservationTime): startAt을 모두 LocalTime로 관리

2d19d33

Co-authored-by: leegwichan <[email protected]>

refactor(Reservation): date을 모두 LocalDate로 관리

d8c8a27

Co-authored-by: leegwichan <[email protected]>

refactor(test): port 설정을 @beforeeach로 설정

bbccce8

Co-authored-by: leegwichan <[email protected]>

refactor(test): SQL을 사용하여 테스트 격리

8e8c87d

Co-authored-by: leegwichan <[email protected]>

style: 코드 컨벤션 정리

bcb70ce

Co-authored-by: leegwichan <[email protected]>

style: 통합 테스트 네이밍 변경

b6b3393

Co-authored-by: leegwichan <[email protected]>

Libienz reviewed May 3, 2024

View reviewed changes

robinjoon reviewed May 3, 2024

View reviewed changes

leegwichan commented May 3, 2024

View reviewed changes

[1 - 3단계 방탈출 사용자 예약] 커찬(이충안) 미션 제출합니다. #2

Are you sure you want to change the base?

[1 - 3단계 방탈출 사용자 예약] 커찬(이충안) 미션 제출합니다. #2

Conversation

leegwichan commented May 2, 2024 • edited Loading

리뷰 링크

이전 Spring 경험

신경 쓴 점

유효성 검사 위치

신경쓰지 못한 점

각 계층별 테스트

Libienz left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

leegwichan left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

leegwichan commented May 2, 2024 •

edited

Loading