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.

Sign up for GitHub

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

Jump to bottom

DELETE [#111] 네트워크 모듈 테스트코드 중 필요없는 코드 삭제 #112

Merged
merged 27 commits into from
Nov 18, 2024
Merged

DELETE [#111] 네트워크 모듈 테스트코드 중 필요없는 코드 삭제 #112

merged 27 commits into from
Nov 18, 2024

Conversation

HELLOHIDI
Copy link
Member

👾 작업 내용

네트워크 모듈 테스트코드 중 필요없는 코드 삭제

🚀 PR Point

네트워크 모듈 테스트코드 중 필요없는 코드 삭제

📸 스크린샷

구현 내용 스크린샷
화면종류 파일첨부바람

✅ CheckList

  • 오류 없이 빌드되는지 확인
  • 로그용 print문 제거
  • 불필요한 주석 제거
  • 코드 컨벤션 확인

🔗 Issue

Resolved #111

@HELLOHIDI
[Refactor/#107] ParameterEncoding 리펙토링
df6a657
@HELLOHIDI
[Test/#107] RequesetValidator URLEncoding 테스트코드 작성
155b676
@HELLOHIDI
[Test/#107] RequestDataValidatorTests Encodable 객체 테스트코드 작성
f3330ca
@HELLOHIDI
[Feat/#107] URLEncoding 테스트 코드 작성
5411c96
@HELLOHIDI
[Text/#107] JSONEncodingTest 테스트 코드 작성
edd600c
@HELLOHIDI
[Test/#107] JSONEncoding 정상적인 파라미터 테스트
22bbce1
@HELLOHIDI
[Chore/#107] 폴더링 변경
e76d1bd
@HELLOHIDI
[Test/#107] Task Test 테스트코드 작성
25107ca
@HELLOHIDI
[Test/#107] url 관련 테스트 코드 작성 (URLValidator 테스트코드로 이전 예정)
9c1ba5f
@HELLOHIDI
[Test/#107] URLValidatorTest 테스트코드 작성
5b4a9e1
@HELLOHIDI
[Test/#107] URLRequestTargetType 테스트코드 작성
a34906a
@HELLOHIDI
[Refactor/#107] RequestDataValidatorTests 리펙토링
5008476
@HELLOHIDI
[Refactor/#107] validateWithEncodable 리펙토링
cf86d67
@HELLOHIDI
[Chore/#107] MockData 수정
b8d4bdb
@HELLOHIDI
[Refactor/#107] JSONEncodingTest 리펙토링
1703b74
@HELLOHIDI
[Refactor/#107] URLEncoding 테스트 코드 리펙토링
62e3aea
@HELLOHIDI
[Test/#107] TaskTest 리펙토링 + Task 코드 Testable하게 리펙토링
d707830
@HELLOHIDI
[Refactor/#107] EncodingValidationHandler 도입해서 중복되는 코드 제거
6572472
@HELLOHIDI
[Test/#107] URLReqeustTargetType Test 코드 수정
f3807ca
@HELLOHIDI
[Refactor/#107] URLReqeustTargetType Test 코드 리펙토링
cf4c1c4
@HELLOHIDI
[Test/#107] RequestHandler 테스트코드 작성
2b97f00
@HELLOHIDI
[Delete/#107] validateURL 메소드 삭제 -> 인코딩 단계에서 검증할 문제 x
5c57edb
@HELLOHIDI
[Refactor/#107] JSONEncoding, URLEncoding 테스터블하게 리펙토링
bc6d900
@HELLOHIDI
[Refactor/#107] 각 객체의 책임에 맞게 테스트 코드 리펙토링
699ec86
@HELLOHIDI
[Refactor/#107] Error타입 테스터블하게 리펙토링
7857272
@HELLOHIDI
[Delete/#107] 현재 테스트 어려운 메소드 주석 처리
d4c4a9c
@HELLOHIDI
[Delete/#111] 필요없은 코드 삭제
1cf4843
@HELLOHIDI HELLOHIDI added 🔧FIX 오류 고치는 것 / 수정용 🍀희재 labels Nov 16, 2024
@HELLOHIDI HELLOHIDI requested a review from Zoe0929 November 16, 2024 03:39
@HELLOHIDI HELLOHIDI self-assigned this Nov 16, 2024
kim-seonwoo
kim-seonwoo approved these changes Nov 16, 2024
View reviewed changes
Copy link
Member

@kim-seonwoo kim-seonwoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다 !!

Zoe0929
Zoe0929 approved these changes Nov 17, 2024
View reviewed changes
Copy link
Member

@Zoe0929 Zoe0929 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다~

@HELLOHIDI HELLOHIDI merged commit 00449d1 into develop Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kim-seonwoo kim-seonwoo kim-seonwoo approved these changes

@Zoe0929 Zoe0929 Zoe0929 approved these changes

Assignees

@HELLOHIDI HELLOHIDI

Labels
🍀희재 🔧FIX 오류 고치는 것 / 수정용
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[DELETE] 네트워크 모듈 테스트코드 중 필요없는 코드 삭제
3 participants
@HELLOHIDI @Zoe0929 @kim-seonwoo