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

Dollar/Franc 중복 #26

Closed
BBBOMi opened this issue Jul 15, 2020 · 0 comments · Fixed by #27
Closed

Dollar/Franc 중복 #26

BBBOMi opened this issue Jul 15, 2020 · 0 comments · Fixed by #27
Labels
step

Comments

@BBBOMi
Copy link
Owner

BBBOMi commented Jul 15, 2020 •
edited
Loading

  1. 해결하고자 하는 목표
    • Dollar와 Franc의 중복 제거

  1. 해당 이슈에서 한 작업
    • 하위 클래스의 속을 들어내는 걸 완료, 하위 클래스를 삭제
    • 기존의 소스 구조에서는 필요했지만 새로운 구조에서는 필요 없게 된 테스트를 제거
@BBBOMi BBBOMi added the step label Jul 15, 2020
BBBOMi added a commit that referenced this issue Aug 20, 2020
@BBBOMi
Refactor : Dollar 클래스 제거 #26
4db26b4 
- 하위 클래스에 대한 참조를 상위 클래스에 대한 참조로 변경
- 사용되지 않는 Dollar class 제거
BBBOMi added a commit that referenced this issue Aug 20, 2020
@BBBOMi
Refactor : FrancTest class 제거 #26
6d7f01a 
- FrancTest class의 메서드를 DollarTest class로 이동
- 중복되거나 불필요한 테스트 메서드 제거
- 테스트 검증 완료
BBBOMi added a commit that referenced this issue Aug 20, 2020
@BBBOMi
Refactor : Franc class 제거 #26
72eed59 
- 테스트 클래스에서 직접 참조하고 있던 코드 수정
- 테스트 검증 완료
BBBOMi added a commit that referenced this issue Aug 20, 2020
@BBBOMi
Refactor : Test Class 이름 변경 #26
cdaadcd 
- DollarTest 에서 수정된 코드에 더 적합한 MoneyTest 로 변경
BBBOMi added a commit that referenced this issue Aug 20, 2020
@BBBOMi
Refactor : Franc의 곱하기 테스트 메서드 제거 #26
d8f94a0 
- 화폐의 로직상의 차이가 없으므로 testDollarMultiplication()만 남기고 제거
@BBBOMi BBBOMi linked a pull request Aug 20, 2020 that will close this issue
Refactor : Dollar 와 Franc 제거 #27
Merged
@BBBOMi BBBOMi mentioned this issue Aug 20, 2020
Open
23 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
step
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactor : Dollar 와 Franc 제거 BBBOMi/TDD-practice
1 participant
@BBBOMi