Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: feature/#558-apply-calldapter 테스트용 PR #573

Closed
wants to merge 53 commits into from

Conversation

hxeyexn
Copy link
Contributor

@hxeyexn hxeyexn commented Dec 18, 2024

feature/#558-apply-calldapter 테스트용 PR입니다.

- ResponseResult Exception의 message 파라미터에 기본 인자 설정
- 이전: handleApiResponse2
- 이후: handleApiResponse
- 이전: ApiResponseHandler
- 이후: ApiResultHandler
- 이전: NetworkResultCall
- 이후: ApiResultCall
- 이전: NetworkResultCallAdapter
- 이후: ApiResultCallAdapter
- 이전: NetworkResultCallAdapterFactory
- 이후: ApiResultCallAdapterFactory
- 테스트 시나리오: 유효한 형식의 카테고리로 생성을 요청
- 테스트 시나리오: 유효하지 않은 형식의 카테고리로 생성을 요청
- 테스트 시나리오: 20MB를 초과하는 사진을 업로드 요청
- 테스트 시나리오: 서버의 응답이 없는 경우
- 테스트 시나리오: 서버 장애 발생
- 테스트 시나리오: 댓글 삭제를 요청한 사용자와 댓글 작성자의 인증 정보 불일치
- 테스트 시나리오: 인증되지 않은 사용자가 카테고리 생성을 요청
- api 요청 성공(200) 코드 작성 시 201을 200으로 잘못 작성함
- 따라서 200을 201로 수정함
- 테스트 시나리오: 존재하는 카테고리를 조회
- 이전: CallAdapterTest
- 이후: ApiResultCallAdapterTest
- Fixture를 함수로 분리하면 테스트 독립성을 더욱 보장할 수 있음
- 이전: makeFakeImageFile()
- 이후: createFakeImageFile()
- 이전: MockWebServerFixture
- 이후: MockWebServerProvider
- 이전: makeMockResponse()
- 이후: createMockResponse()
@hxeyexn hxeyexn self-assigned this Dec 18, 2024
@hxeyexn hxeyexn linked an issue Dec 18, 2024 that may be closed by this pull request
2 tasks
@hxeyexn hxeyexn changed the title feature/#558-apply-calldapter 테스트용 PR refactor: feature/#558-apply-calldapter 테스트용 PR Dec 18, 2024
@hxeyexn hxeyexn closed this Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

refactor: CallAdapter 적용
1 participant