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

feat(assertions): Mocking 클래스 추가 #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kang-hyungu
Copy link

해결하려는 문제가 무엇인가요?

  • Randoms 클래스의 정적 메서드를 1개 이상 stubbing 할 수 있도록 수정
  • 코드 정리

어떻게 해결했나요?

  • Mocking 클래스를 추가하여 verification, value, values를 하나로 묶음
  • assertRandomTest를 public으로 열어서 사용자가 Mocking를 활용하여 랜덤 메서드를 직접 지정해서 stubbing하도록 유연성 제공

어떤 부분에 집중하여 리뷰해야 할까요?

  • 라이브러리 사용자에게 Mocking 클래스를 제공하는게 좋을지, 적절한 설계인지 의견 부탁드려요.
  • Assertions 클래스에서 기존 로직과 큰 차이가 없다고 생각하여 @SafeVarargs를 붙인 메서드가 문제 없을지 크로스 체크 부탁드립니다.

RCA 룰

r: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
c: 웬만하면 반영해 주세요. (Comment)
a: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant