[feat #56] 질문글 수정 API #57
Merged
Conversation
This file contains 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
Test Results84 tests 79 ✅ 9s ⏱️ Results for commit 4a90375. ♻️ This comment has been updated with latest results. |
hyun2371
commented
Aug 19, 2024
| private void updateQuestionPostImages(QuestionPost questionPost, List<String> imageUrls) { | ||
| if (imageUrls != null) { // 수정 사항 존재 | ||
| deleteImages(questionPost); // 기존 이미지 객체 삭제 (새로 비우기 || 수정할 값 존재) | ||
| if (!imageUrls.isEmpty()) { //수정할 값 담아보냄 |
There was a problem hiding this comment.
요청에서 이미지가 빈 리스트-> 기존 이미지 삭제를 단위 테스트 하려고 했습니다. 그런데 repository를 활용헤 deleteAll() 하는거라 값이 비워지지 않았습니다. given.willReturn 을 사용하기에는 void 메서드라 stubbing을 하지 못했어요.. 그래서 단위 테스트 작성 대신 통합테스트에서 검증했습니다
dudxo
approved these changes
Aug 19, 2024
src/main/java/com/dnd/gongmuin/question_post/dto/QuestionPostMapper.java
Outdated
Show resolved
Hide resolved
src/test/java/com/dnd/gongmuin/question_post/controller/QuestionPostControllerTest.java
Show resolved
Hide resolved
dudxo
pushed a commit
that referenced
this pull request
Aug 20, 2024
* [chore] : 질문글 추천 API SWAGGER 명세 * [feat] : 질문 수정 응답 dto 추가 * [rename] : 질문 수정 응답 dto 컨벤션에 맞게 네이밍 변경 * [refactor] : dto 팩토리 메서드 제거 * [feat] : 질문글 업데이트 dto 추가 * [fix] : dto mapper에 이미지, url null일 때 검증 로직 추가 * [feat] : 이미지 일괄 삭제를 위한 repository 추가 * [feat] : 업데이트 dto mapper 함수 추가 * [feat] : 엔티티 내 필드 업데이트 로직 추가 * [feat] : 질문글 업데이트 비즈니스 로직 작성 * [feat] : 질문글 업데이트 API 메서드 작성 * [test] : 요구사항 validation 맞게 fixture 수정 * [fix] : 이미지 삭제 시 질문글 내 이미지 리스트 비우는 로직 추가 * [fix] : 컨트롤러 누락된 어노테이션 추가 * [fix] : 리스트 데이터 변경 불가 예외 해결 * [test] : 질문글 업데이트 단위 테스트 작성 * [test] : 질문글 업데이트 통합 테스트 작성 * [style] : 코드 리포멧팅 * [feat] : null 대신 새 ArrayList 할당
dudxo
added a commit
that referenced
this pull request
Aug 20, 2024
* [fix] : 토큰 재발급 보안 위험을 제거하기 위한 바디 응답 -> 헤더 응답 변경 * [fix] : JWT-Cookie 방식으로 변경 및 로그인 후 리다이렉트 주소(프론트 테스트 환경) 변경 * [fix] : JWT 인증 필터 헤더가 아닌 쿠키에서 꺼내오도록 수정 * [feat] : Cookie 생성 및 분해 처리 Uitl.Class 추가 * [fix] : Jwt-Cookie 변경에 따른 로그아웃/토큰 재발급 로직 수정 * [fix] : 카카오 로그인 flow 변경에 따른 api 삭제 * [rename] : 인증 로직 파일 구조 변경 (Member -> Auth) * [test] : Auth <-> Member Service 레이어 책임에 따른 로직 이동으로 인한 테스트 코드 위치 수정 * [test] : AuthFixture 추가 * [fix] : 공무원 이메일 존재 검증 메서드 접근제어자 변경 * [test] : 팩토리 메서드가 아닌 Fixture 사용 변경 * [test] : disable 제거 및 통합테스트 어노테이션 제거 * [test] : ApiTestSupport JWT 생성 시 쿠키 필드로 저장 * [test] : JWT-Cookie 방식 변경으로 인한 컨트롤러 통합테스트 요청 수정(header -> cookie) * [sytle] : 코드 포맷팅 * [fix] : findBy를 통해 찾은 엔티티가 없을 때도 항상 실행되는 orElse() 문제 orElseGet() 변경으로 해결 * [rename] : 변수/메서드 네이밍 변경 * [feat] : 토큰 재발급 API 인가 허용 * [fix] : swagger 쿠키 사용으로 인한 인가 관련 Bean 삭제 * [fix] : swagger 임시 회원가입 API 응답 로직 변경(JWT return String -> return response.addCookie) * [feat] : 토큰 검증 시 기존 예외처리 외 나머지 예외에 대한 예외처리 추가 * [feat] : 인가 API 수정 및 일부 API 시큐리티 필터 통과하도록 변경 * [test] : 본 로직 수정으로 필요 없어진 stub 제거 * [feat] : 로그아웃 API 기존 쿠키 삭제 로직 추가 * [feat] : 쿠키 저장된 토큰 삭제 로직 추가 * [test] : 로그아웃 시 기존 쿠키가 삭제되는지 검증 로직 추가 * [style] : 스타일 변경 * [style] : 스타일 변경 * [test] : CI 오류 해결을 위한 disable * [feat #56] 질문글 수정 API (#57) * [chore] : 질문글 추천 API SWAGGER 명세 * [feat] : 질문 수정 응답 dto 추가 * [rename] : 질문 수정 응답 dto 컨벤션에 맞게 네이밍 변경 * [refactor] : dto 팩토리 메서드 제거 * [feat] : 질문글 업데이트 dto 추가 * [fix] : dto mapper에 이미지, url null일 때 검증 로직 추가 * [feat] : 이미지 일괄 삭제를 위한 repository 추가 * [feat] : 업데이트 dto mapper 함수 추가 * [feat] : 엔티티 내 필드 업데이트 로직 추가 * [feat] : 질문글 업데이트 비즈니스 로직 작성 * [feat] : 질문글 업데이트 API 메서드 작성 * [test] : 요구사항 validation 맞게 fixture 수정 * [fix] : 이미지 삭제 시 질문글 내 이미지 리스트 비우는 로직 추가 * [fix] : 컨트롤러 누락된 어노테이션 추가 * [fix] : 리스트 데이터 변경 불가 예외 해결 * [test] : 질문글 업데이트 단위 테스트 작성 * [test] : 질문글 업데이트 통합 테스트 작성 * [style] : 코드 리포멧팅 * [feat] : null 대신 새 ArrayList 할당 * [test] : CI 오류 해결 * Test 충돌 해결을 위한 dev pull ---------
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.
관련 이슈
📑 작업 상세 내용
💫 작업 요약
🔍 중점적으로 리뷰 할 부분