Feature/69 기관 프로필 조회 컨트롤러 구현 #74
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 기관 프로필 조회 엔드포인트를 위한 컨트롤러 구현 - center의 PK값에 columnDefinition = "BINARY(16)" 옵션을 넣어 타입을 더 명시적으로 나타냄 - 테스트 코드 작성및 검증완료
1 task
m-a-king
reviewed
Nov 29, 2024
Collaborator
m-a-king
left a comment
m-a-king
left a comment
There was a problem hiding this comment.
수고하셨습니다. 컨트롤러 테스트 이렇게 따라가보게씁니다
| .andDo(print()) | ||
| .andExpect(status().isBadRequest()) | ||
| .andExpect(jsonPath("$.status").value("400")) | ||
| .andExpect(jsonPath("$.detail").value("존재하지 않는 기관 ID 입니다.")); |
Collaborator
There was a problem hiding this comment.
존재하지 않는 기관입니다 로 바꾸는 것은 어떨까요?
m-a-king
approved these changes
Nov 29, 2024
leebs0521
approved these changes
Nov 29, 2024
| @GeneratedValue(strategy = GenerationType.UUID) | ||
| @Column(nullable = false, length = 16) | ||
| @Column(nullable = false, columnDefinition = "BINARY(16)") | ||
| private UUID id; |
Collaborator
There was a problem hiding this comment.
이렇게 바꾸신 이유가 혹시 오류 떄문인가요?
Collaborator
Author
There was a problem hiding this comment.
BINARY(16)으로 id값을 압축했다는걸 명시적으로 보여주고 싶었어요
- 기관 프로필 조회 엔드포인트를 위한 컨트롤러 구현 - center의 PK값에 columnDefinition = "BINARY(16)" 옵션을 넣어 타입을 더 명시적으로 나타냄 - 테스트 코드 작성및 검증완료
- 예외 메세지 변경 - 컨트롤러와 API 엔드포인트에 스웨거 어노테이션 추가 - 응답 DTO에 스웨거 스키마 추
…ller' into feature/69-add-center-api-controller # Conflicts: # src/main/java/com/somemore/center/controller/CenterQueryApiController.java
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
github-actions
bot
changed the title
github-actions
bot
changed the title
- CenterQueryApiControllerTest 예외 테스트의 예외 메세지를 수정
7zrv
changed the title
|
leebs0521
pushed a commit
that referenced
this pull request
Nov 29, 2024
* feat: 기관 프로필 조회 컨트롤러 구현 - 기관 프로필 조회 엔드포인트를 위한 컨트롤러 구현 - center의 PK값에 columnDefinition = "BINARY(16)" 옵션을 넣어 타입을 더 명시적으로 나타냄 - 테스트 코드 작성및 검증완료 * feat: 기관 프로필 조회 컨트롤러 구현 - 기관 프로필 조회 엔드포인트를 위한 컨트롤러 구현 - center의 PK값에 columnDefinition = "BINARY(16)" 옵션을 넣어 타입을 더 명시적으로 나타냄 - 테스트 코드 작성및 검증완료 * chore: 코드 리뷰 사항 반영 - 예외 메세지 변경 - 컨트롤러와 API 엔드포인트에 스웨거 어노테이션 추가 - 응답 DTO에 스웨거 스키마 추 * fix: 테스트 코드 수정 - CenterQueryApiControllerTest 예외 테스트의 예외 메세지를 수정
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
resolved : #69
📌 과제 설명
기관 프로필 조회의 컨트롤러 구현과 테스트 코드 입니다.
👩💻 요구 사항과 구현 내용
✅ PR 포인트 & 궁금한 점
presentation 레이어 테스트를 작성하는데 많이 어색한 느낌이었습니다.
테스트 작성이 잘 되었는지 여쭤보고 싶어요