This repository has been archived by the owner on Aug 13, 2022. It is now read-only.
[#43] 추가된 단위 테스트에 따른 Service Layer 로직 수정 #44
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
- 각 DTO 별로 필드 유효성 제약 조건 추가 - PostController 사용자가 존재하지 않는 게시물이나 존재하지 않는 사용자의 피드 조회를 요청하면 Not Found 응답을 반환하도록 로직 수정
f-lab-dev
reviewed
Nov 17, 2020
|
|
||
| return new ResponseEntity<>(posts, HttpStatus.OK); | ||
| return new ResponseEntity<>(posts, HttpStatus.OK); |
There was a problem hiding this comment.
이거는 팩토리메소드로 대체할 수 있을 것 같습니다~
ResponseEntity.ok()
f-lab-dev
reviewed
Nov 24, 2020
|
|
||
| return RESPONSE_OK; | ||
| } catch (AlreadyLoginException e) { | ||
| return RESPONSE_BAD_REQUEST; |
There was a problem hiding this comment.
이건 사용자가 잘못된 요청을 보냈다기보다는 이미 로그인한 상태이기 때문에 에러가 발생하는거라 다른 코드를 쓰는게 나을 것 같아요~
그리고 꼭 에러를 보내줘야할지도 생각해봐야겠네요
There was a problem hiding this comment.
로그인한 유저가 다시 로그인을 시도한다고 해서 굳이 에러를 보내줄 필요는 없을 것 같아요~
이거는 상황에 따라 다르겠지만 정상 상황으로 처리해줘도 될 것 같아서 굳이 바꾸진 않더라도 한번 고민만 해보시면 될 것 같네요~
|
테스트도 깨지는 것 같은데 이것도 확인 부탁드려요~ |
f-lab-dev
approved these changes
Dec 4, 2020
|
|
||
| return RESPONSE_OK; | ||
| } catch (AlreadyLoginException e) { | ||
| return RESPONSE_BAD_REQUEST; |
There was a problem hiding this comment.
로그인한 유저가 다시 로그인을 시도한다고 해서 굳이 에러를 보내줄 필요는 없을 것 같아요~
이거는 상황에 따라 다르겠지만 정상 상황으로 처리해줘도 될 것 같아서 굳이 바꾸진 않더라도 한번 고민만 해보시면 될 것 같네요~
cyj199637
added a commit
that referenced
this pull request
Dec 18, 2020
* [#29] Service Layer 단위테스트 작성 - UserServiceTest의 일부 테스트 메소드 로직 수정 - PostServiceTest 작성 - SessionLoginServiceTest 작성 * [#31] 팔로우 / 언팔로우 기능 구현 - 팔로우 기능 구현 - 언팔로우 기능 구현 * [#31] 알림 추가 기능 구현 - 팔로잉된 사용자에게 알림이 가도록 FollowController의 followUser()에 알림 추가 기능을 추가 * [#31] Follow 테이블 컬럼 변경된 이름 반영 * [#31] 피드백 반영 - FollowController URL 수정 - FollowService에서 알림 기능을 처리하도록 수정 - FollowService 인터페이스 제거 * [#29] 피드백 반영 - 실패 시나리오에 대한 테스트 추가 * [#29] PostServiceTest의 피드 조회 성공, 피드 조회 실패 테스트 수정 * [#29] 피드백 반영 - 각 테스트마다 어떤 상황에 대한 테스트인지 자세한 설명 추가 - SessionLoginServiceTest에 새로운 테스트 케이스 추가 * [#29] SessionLoginServiceTest 일부 테스트 로직 수정 및 테스트 케이스 추가 * [#37] Firebase를 활용한 푸시 서비스 (#38) * [#37] Firebase를 활용한 푸시 서비스 - Firebase를 활용하여 푸시 서비스 구현 - 푸시 메시지 전송을 비동기로 처리 * [#37] Firebase 푸시 서비스 수정 - 각 클래스마다 사용한 어노테이션에 대한 주석 추가 - AccessToken을 Redis에 저장하여 관리하도록 로직 변경 * [#37] Firebase 푸시 서비스 수정 - 로그인 / 로그아웃에 AccessToken 등록 / 삭제 로직 추가 * [#37] Firebase 푸시 서비스 빠트린 주석 추가 및 테스트 코드 수정 * [#37] 피드백 반영 - MessageSource와 LocaleResolver를 이용하여 사용자의 위치에 따라 푸시 메시지의 언어가 다르게 나오도록 수정 - Redis에 저장되는 토큰 만료시간 설정 - 팩토리 메소드를 사용하여 스레드풀 생성 * [#37] 피드백 반영 - @async 대신 sendAsync() 메소드로 비동기 처리 * [#29] Jenkins Failed Test Code 수정 * [#29] Jenkins Failed Test Code 수정 * [#43] 추가된 단위 테스트에 따른 Service Layer 로직 수정 (#44) * [#43] 추가된 단위 테스트에 따른 Service Layer 로직 수정 - 각 DTO 별로 필드 유효성 제약 조건 추가 - PostController 사용자가 존재하지 않는 게시물이나 존재하지 않는 사용자의 피드 조회를 요청하면 Not Found 응답을 반환하도록 로직 수정 * [#43] 피드백 반영 - 일부 응답 객체 팩토리 메소드로 생성 - SessionLoginServiceTest의 추가된 테스트에 따른 SessionLoginService, UserController 로직 수정 * [#43] Jenkins Failed Test Code 수정 * [#29] UserControllerTest 코드 수정 * [#29] Jenkins Failed Test Code 수정
cyj199637
added a commit
that referenced
this pull request
Jan 7, 2021
* [#29] Service Layer 단위테스트 작성 - UserServiceTest의 일부 테스트 메소드 로직 수정 - PostServiceTest 작성 - SessionLoginServiceTest 작성 * [#31] 팔로우 / 언팔로우 기능 구현 - 팔로우 기능 구현 - 언팔로우 기능 구현 * [#31] 알림 추가 기능 구현 - 팔로잉된 사용자에게 알림이 가도록 FollowController의 followUser()에 알림 추가 기능을 추가 * [#31] Follow 테이블 컬럼 변경된 이름 반영 * [#31] 피드백 반영 - FollowController URL 수정 - FollowService에서 알림 기능을 처리하도록 수정 - FollowService 인터페이스 제거 * [#29] 피드백 반영 - 실패 시나리오에 대한 테스트 추가 * [#29] PostServiceTest의 피드 조회 성공, 피드 조회 실패 테스트 수정 * [#29] 피드백 반영 - 각 테스트마다 어떤 상황에 대한 테스트인지 자세한 설명 추가 - SessionLoginServiceTest에 새로운 테스트 케이스 추가 * [#29] SessionLoginServiceTest 일부 테스트 로직 수정 및 테스트 케이스 추가 * [#45] 원격지 script 파일 실행 테스트 * [#45] Jenkinsfile 수정 * [#45] Jenkinsfile 수정 * [#45] Jenkinsfile 수정 * [#37] Firebase를 활용한 푸시 서비스 (#38) * [#37] Firebase를 활용한 푸시 서비스 - Firebase를 활용하여 푸시 서비스 구현 - 푸시 메시지 전송을 비동기로 처리 * [#37] Firebase 푸시 서비스 수정 - 각 클래스마다 사용한 어노테이션에 대한 주석 추가 - AccessToken을 Redis에 저장하여 관리하도록 로직 변경 * [#37] Firebase 푸시 서비스 수정 - 로그인 / 로그아웃에 AccessToken 등록 / 삭제 로직 추가 * [#37] Firebase 푸시 서비스 빠트린 주석 추가 및 테스트 코드 수정 * [#37] 피드백 반영 - MessageSource와 LocaleResolver를 이용하여 사용자의 위치에 따라 푸시 메시지의 언어가 다르게 나오도록 수정 - Redis에 저장되는 토큰 만료시간 설정 - 팩토리 메소드를 사용하여 스레드풀 생성 * [#37] 피드백 반영 - @async 대신 sendAsync() 메소드로 비동기 처리 * [#45] 외부 설정 주입 테스트 * [#45] 외부 설정 주입 테스트 * [#49]Docker를 활용한 CD 테스트 * [#49] Docker를 활용한 CD 테스트 * [#29] Jenkins Failed Test Code 수정 * [#29] Jenkins Failed Test Code 수정 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#49] Docker 배포 테스트 * [#43] 추가된 단위 테스트에 따른 Service Layer 로직 수정 (#44) * [#43] 추가된 단위 테스트에 따른 Service Layer 로직 수정 - 각 DTO 별로 필드 유효성 제약 조건 추가 - PostController 사용자가 존재하지 않는 게시물이나 존재하지 않는 사용자의 피드 조회를 요청하면 Not Found 응답을 반환하도록 로직 수정 * [#43] 피드백 반영 - 일부 응답 객체 팩토리 메소드로 생성 - SessionLoginServiceTest의 추가된 테스트에 따른 SessionLoginService, UserController 로직 수정 * [#43] Jenkins Failed Test Code 수정 * [#49] Docker 배포 테스트를 위한 Dockerfile 수정 * [#49] Docker 배포 테스트를 위한 Dockerfile 수정 * [#29] UserControllerTest 코드 수정 * [#49] Jenkins Failed Test Code 수정 * [#49] Dockerfile 수정 * [#49] Dockerfile 수정 * [#49] Dockerfile 수정 * [#49] Dockerfile 수정 - JAVA_OPTS 환경변수 추가 * [#49] Dockerfile 수정 * [#49] Dockerfile 수정 * [#49] Dockerfile 수정 * [#49] Dockerfile 수정 * [#49] Dockerfile 수정 * [#49] Dockerfile 수정 * [#49] Dockerfile 수정 * [#49] Dockerfile 수정
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
No description provided.