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

test: 테스트 환경에서 FCM 서버 대체하던 클래스명 fake -> stub 변경 #686

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

test: 테스트 환경에서 FCM 서버 대체하던 클래스명 fake -> stub 변경 #686

wants to merge 1 commit into from

Conversation

helenason
Copy link
Contributor

📌 관련 이슈

close #685

✨ 작업 내용

테스트 환경에서 FCM 서버 대체하던 클래스명 fake -> stub 변경

📚 기타

@helenason helenason self-assigned this Jan 7, 2025
fromitive
fromitive approved these changes Jan 8, 2025
View reviewed changes
Copy link
Contributor

@fromitive fromitive left a comment

Choose a reason for hiding this comment

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

에버 확인 완료했습니다~

approve 드릴게요

한가지 궁금한 점이 있습니다!
제가 이해하기론 fake는 테스트 편의상 만드는 객체이고 stub는 호출 정보를 기록한다는 측면에서 차이가 있다고 알고 있는데 맞나요?

masonkimseoul
masonkimseoul approved these changes Jan 8, 2025
View reviewed changes
Copy link
Member

@masonkimseoul masonkimseoul left a comment

Choose a reason for hiding this comment

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

확인 완료했습니다.

저도 stub으로 바꾼 이유가 궁금해요!

@helenason
Copy link
Contributor Author

@fromitive @masonkimseoul

fake와 stub 모두 테스트 더블의 방식 중 한 종류인데요. fake는 기존 클래스의 로직과 비슷하게 동작하도록 구현하는 방법이고, stub는 하드 코딩 방식으로 특정 값만 리턴하도록 하는 방식으로 알고 있습니다. 제가 구현한 클래스는 후자에 해당되었기 때문에 stub로 이름을 변경하였어요!

포케가 언급해준 아래 문장은 어떤 의미인가요 ??

fake는 테스트 편의상 만드는 객체이고 stub는 호출 정보를 기록한다는 측면에서 차이가 있다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@masonkimseoul masonkimseoul masonkimseoul approved these changes

@fromitive fromitive fromitive approved these changes

Assignees

@helenason helenason

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

✅ 테스트 환경에 사용되던 테스트 더블 클래스명 변경
3 participants
@helenason @fromitive @masonkimseoul