[feat]: 공연 단건 정보 조회 API 추가 #33

ssssujini99 · 2023-12-24T09:03:11Z

closed [Feat] : 사용자는 한 공연에 대한 세부 정보를 조회할 수 있다 #22

📄 무엇을 개발했나요?

공연 단건 정보 조회 controller, application, repository 구현 및 각 레이어 별 테스트까지 구현 완료

🍸 무엇을 집중적으로 리뷰해야할까요?

네이밍, 컨벤션 확인
application 레이어 및 mapper 에서 제대로 매핑되었는지 한번 더 확인 부탁드립니다

ssssujini99 · 2023-12-24T09:03:40Z

conflict 는 이따 밥먹고 제가 해결해놓겠습니다!

github-actions · 2023-12-24T09:36:58Z

🦞 Test Coverage Report

Overall Project	NaN% `NaN%`	🟢

There is no coverage information present for the Files changed

EunChanNam

리뷰 마쳤구요 고생하셨습니다~~ 👏
제 의견 관련해서 코멘트 몇개 달았구요
전체적으로 @AllArgumentCustructor 를 사용하셨는데, 컨벤션에서 안쓰기로 한 부분인것같아서 없애고 별도로 생성자로 만드는게 좋을 것 같아요!
단순히 테스트용으로 사용하는 용도로 특별한 검증없는 단순한 생성자여도 상관없을 것 같아요!
저는 이런 테스트용 생성자정도는 열려있어도 무방하다고 생각합니다 ㅎㅎ

그리고 탭 인덴트가 안맞는거같은데 확인 부탁드려요 ㅜㅜ 😢

EunChanNam · 2023-12-24T09:45:19Z

core/src/main/java/dev/hooon/show/application/ShowService.java

+		Optional<Show> showOptional = showRepository.findById(showId);
+		if (showOptional.isEmpty()) {
+			throw new NotFoundException(ShowErrorCode.SHOW_NOT_FOUND);
+		}


orElseThrow() 로 처리하는게 어떨까요? 더 깔끔할거같아요 ㅎㅎ

EunChanNam · 2023-12-24T09:47:05Z

core/src/main/java/dev/hooon/show/domain/entity/Show.java

 import lombok.Getter;
 import lombok.NoArgsConstructor;

 @Entity
 @Getter
 @Table(name = "show_table")
 @NoArgsConstructor
+@AllArgsConstructor


저희 @AllArgsConstructor 사용 안하기로 컨벤션 정했던거같아요!
그리고 이 생성자가 꼭 필요한가요?? 만약 필요하다면 별도로 생성자 구현하는게 좋을 것 같아요!

테스트 시 데이터 insert를 위해서 필요해서 만들었습니다..!!

넵 이거 생성자 추가로 변경하겠습니당

EunChanNam · 2023-12-24T09:57:58Z

core/src/test/java/dev/hooon/show/application/ShowServiceTest.java

+
+		// when
+
+		// then
+		assertThrows(
+			NotFoundException.class,
+			() -> showService.getShowDetailInfo(1L)
+		);


비어있는 given 은 없애도 될 것 같고, then 부분에서 when 행위도 하니까 // when, then 이렇게 묶는게 어떨까요?
그리고 검증에 예외 메세지도 같이 검증해주면 좋을거같아요! 아마 해당 메소드도 메세지 필드추가할 수 있는거로 압니당 ㅎㅎ

EunChanNam · 2023-12-24T09:59:07Z

core/src/test/java/dev/hooon/show/infrastructure/adaptor/ShowRepositoryAdaptorTest.java

저는 findById 같이 DataJpa 가 짜주는 쿼리정도는 테스트 안해도 된다고 생각해요! 물론 열심히 짜신건 고생하셨습니다...!

EunChanNam · 2023-12-24T10:00:35Z

core/src/main/java/dev/hooon/show/dto/ShowMapper.java

+			show.getShowPeriod().getStartDate(),
+			show.getShowPeriod().getEndDate(),
+			show.getShowTime().getTotalMinutes(),
+			show.getShowTime().getIntermissionMinutes(),
+			show.getShowAgeLimit(),
+			new PlaceDetailsInfo(
+				show.getPlace().getName(),
+				show.getPlace().getContactInfo(),
+				show.getPlace().getAddress(),
+				show.getPlace().getPlaceUrl()


. 에 . 찍으면서 호출하는 패턴 대신 별도의 getter 를 구현하는게 좋다고 생각을 했었는데... 이건 별도로 구현할 게터가 너무 많아보이네요... 그냥 이대로 써도 괜찮을 것 같아요 ㅎㅎ

EunChanNam · 2023-12-24T10:03:36Z

core/src/testFixtures/resources/sql/show_dummy.sql

별도의 sql �문이 있어야 테스트가 가능한 상황일까요?? (사용해도 상관은 없는데 궁금해서 물어봐용 ㅎㅎ)

이거는 controller test 에서 미리 세팅된 데이터가 필요해서 넣었습니다!

github-actions · 2023-12-24T15:29:06Z

🦞 Test Coverage Report

Overall Project	NaN% `NaN%`	🟢

There is no coverage information present for the Files changed

github-actions · 2023-12-24T15:33:57Z

🦞 Test Coverage Report

Overall Project	NaN% `NaN%`	🟢

There is no coverage information present for the Files changed

github-actions · 2023-12-24T16:09:47Z

🦞 Test Coverage Report

Overall Project	NaN% `NaN%`	🟢

There is no coverage information present for the Files changed

ssssujini99 · 2023-12-24T16:11:08Z

코드 인덴트까지 모두 마쳤습니다! 마지막 검토 부탁드려용

EunChanNam

고생하셨어요~ 꼼꼼하게 수정 잘 해주신거같아요 👍

@AllArgsConstructor

* [test]: 공연 정보 조회 API controller test 추가 * [feat]: 공연 정보 조회 API 추가 * [chore]: core모듈에 jackson 라이브러리 추가 * [test]: 공연 정보 조회 Application test 추가 * [feat]: 공연 정보 조회 Application, 응답 DTO 추가 * [feat]: NotFoundException 공통 예외 추가 * [feat]: ShowErrorCode - Show 도메인 에러 코드 추가 * [refactor]: Show 모든 도메인에 AllArgsConstructor 추가 * [test]: ShowRepositoryAdaptorTest 추가 * [feat]: ShowRepository, ShowRepositoryAdaptor 추가 * [feat]: PlaceRepository, PlaceRepositoryAdaptor 추가 * [feat]: ApiExceptionHandler에 NotFoundException 추가 * [docs]: show_dummy.sql 더미데이터 파일 추가 * [refactor]: DataJpaTestSupport 옵션 설정 추가 * [fix]: NotFoundException, ExceptionHandler 수정 * [fix]: conflict 해결 * [refactor]: Optional orElseThrow로 처리 * [refactor]: 에러 메시지 검증 추가 * [refactor]: @AllArgsConstructor 대신 생성자로 리팩토링

ssssujini99 added 14 commits December 24, 2023 17:38

[test]: 공연 정보 조회 API controller test 추가

0c07585

[feat]: 공연 정보 조회 API 추가

1ca4e5b

[chore]: core모듈에 jackson 라이브러리 추가

b9b2d95

[test]: 공연 정보 조회 Application test 추가

f6f57c9

[feat]: 공연 정보 조회 Application, 응답 DTO 추가

7dda6b5

[feat]: NotFoundException 공통 예외 추가

2f14b7c

[feat]: ShowErrorCode - Show 도메인 에러 코드 추가

789c88d

[refactor]: Show 모든 도메인에 AllArgsConstructor 추가

12f1aff

[test]: ShowRepositoryAdaptorTest 추가

e9ab71a

[feat]: ShowRepository, ShowRepositoryAdaptor 추가

6638025

[feat]: PlaceRepository, PlaceRepositoryAdaptor 추가

210a340

[feat]: ApiExceptionHandler에 NotFoundException 추가

5eee542

[docs]: show_dummy.sql 더미데이터 파일 추가

07ed441

[refactor]: DataJpaTestSupport 옵션 설정 추가

334fabc

ssssujini99 added the ✏️ Feat 기능 개발 label Dec 24, 2023

ssssujini99 added this to the 1차 스프린트 milestone Dec 24, 2023

ssssujini99 self-assigned this Dec 24, 2023

ssssujini99 and others added 3 commits December 24, 2023 18:14

[fix]: NotFoundException, ExceptionHandler 수정

eb5218a

Merge branch 'dev' into feat/#22/show-details-api

caa42ae

[fix]: conflict 해결

babb762

EunChanNam reviewed Dec 24, 2023

View reviewed changes

[refactor]: Optional orElseThrow로 처리

6c88622

[refactor]: 에러 메시지 검증 추가

23b8021

[refactor]: @AllArgsConstructor 대신 생성자로 리팩토링

300c112

EunChanNam approved these changes Dec 24, 2023

View reviewed changes

EunChanNam merged commit a6fe876 into dev Dec 24, 2023
2 checks passed

ssssujini99 linked an issue Dec 26, 2023 that may be closed by this pull request

[Feat] : 사용자는 한 공연에 대한 세부 정보를 조회할 수 있다 #22

Closed

ssssujini99 deleted the feat/#22/show-details-api branch January 2, 2024 04:46

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat]: 공연 단건 정보 조회 API 추가 #33

[feat]: 공연 단건 정보 조회 API 추가 #33

ssssujini99 commented Dec 24, 2023

ssssujini99 commented Dec 24, 2023

github-actions bot commented Dec 24, 2023

EunChanNam left a comment •

edited

Loading

EunChanNam Dec 24, 2023

EunChanNam Dec 24, 2023

ssssujini99 Dec 24, 2023

ssssujini99 Dec 24, 2023

EunChanNam Dec 24, 2023

EunChanNam Dec 24, 2023

EunChanNam Dec 24, 2023

EunChanNam Dec 24, 2023

ssssujini99 Dec 24, 2023

github-actions bot commented Dec 24, 2023

github-actions bot commented Dec 24, 2023

github-actions bot commented Dec 24, 2023

ssssujini99 commented Dec 24, 2023

EunChanNam left a comment

[feat]: 공연 단건 정보 조회 API 추가 #33

[feat]: 공연 단건 정보 조회 API 추가 #33

Conversation

ssssujini99 commented Dec 24, 2023

📄 무엇을 개발했나요?

🍸 무엇을 집중적으로 리뷰해야할까요?

ssssujini99 commented Dec 24, 2023

github-actions bot commented Dec 24, 2023

🦞 Test Coverage Report

EunChanNam left a comment • edited Loading

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

github-actions bot commented Dec 24, 2023

🦞 Test Coverage Report

github-actions bot commented Dec 24, 2023

🦞 Test Coverage Report

github-actions bot commented Dec 24, 2023

🦞 Test Coverage Report

ssssujini99 commented Dec 24, 2023

EunChanNam left a comment

Choose a reason for hiding this comment

EunChanNam left a comment •

edited

Loading