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

테스트 만들기 #10

Open
3 of 5 tasks
EATSTEAK opened this issue Sep 21, 2023 · 4 comments
Open
3 of 5 tasks

테스트 만들기 #10

EATSTEAK opened this issue Sep 21, 2023 · 4 comments
Assignees
Labels
refactoring refactoring code
Milestone

Comments

@EATSTEAK
Copy link
Owner

EATSTEAK commented Sep 21, 2023

현상

  • 단위 테스트 부족
  • 미발견된 다수의 버그 존재

(예상되는) 원인

  • WebDynpro의 동작 재현의 어려움
  • u-saint 데이터의 엣지 케이스
  • 어떤 테스트가 필요한지 알기 어려움

(예상되는) 해결책

@EATSTEAK
Copy link
Owner Author

EATSTEAK commented Apr 28, 2024

  • Clientreqwest::Client 사이의 의존성을 최소화하여 가상의 클라이언트 역할을 하는 더미 클라이언트 필요(항상 최신 버전의 WebDynpro에서 작동한다는 보장이 필요하므로 항상 Web request를 하는 것이 필요함)
  • 특정 HTML 문서 케이스를 이용하여 테스트하는 것도 방법

@EATSTEAK EATSTEAK added this to the 1.0 milestone Apr 28, 2024
@EATSTEAK
Copy link
Owner Author

  • 애플리케이션 API의 경우 대부분 통합 테스트가 되어있어 추가 작성은 불필요할 것으로 예상
  • 각각의 기능에 대한 테스트는 어떻게 해야 할지?(모든 기능들에 웹 리퀘스트가 조금씩 필요하다 보니 테스트가 쉽지 않음)

@EATSTEAK EATSTEAK self-assigned this Oct 23, 2024
@EATSTEAK EATSTEAK added the refactoring refactoring code label Oct 23, 2024
@EATSTEAK
Copy link
Owner Author

EATSTEAK commented Nov 6, 2024

  • 파싱 등 로직이 복잡한 경우 테스트 추가 작성
  • 웹 리퀘스트가 없다면 의미 없는 테스트는 진행하지 않아야 함(최신 u-saint에서 동작이 가능함을 보장해야 하기 때문)

@EATSTEAK EATSTEAK pinned this issue Dec 21, 2024
@EATSTEAK
Copy link
Owner Author

webdynpro 모듈에 대한 테스트 커버리지 80% 달성 목표

@EATSTEAK EATSTEAK modified the milestones: 1.0, next Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring refactoring code
Projects
None yet
Development

No branches or pull requests

1 participant