forked from lmw7414/fastcampus-project-board
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
lmw7414#70 - 게시글 서비스의 해시태그 기능 변경점 반영
테스트의 변화는 lmw7414#68 의 e9f36a4 에서 도메인 변경을 하면서 이미 구상하여 작업하였음 주로 게시글이 저장, 수정, 삭제될 때 해시태그가 함께 어떻게 움직여야 하는지를 묘사한다. 강의 진행하면서 도메인 변경이 얼마나 넓은 영역에 영향을 미치는지 보여주면서, 컴파일 에러를 해소하기 위해 일단 테스트 변경까지 모두 반영하게 되었다. 이런 과정을 거치면서 코드 작성 과정을 잘 구분하고 보여줄 시간이 부족하여 이런 커밋들이 발생했는데, 실전에서 이렇게 작업 내역을 나누면 커밋으로 이력을 추적하면서 볼 때 다소 불편할 것 같다. 또한 사이의 커밋이 테스트 실패하는 불완전한 상태로 진행된 것도 문제. 서비스의 변경은 테스트 변경과 함께 담길 수 있도록 하자.
- Loading branch information
oyoungsun
committed
Jan 16, 2023
1 parent
a08221e
commit 5d5752d
Showing
1 changed file
with
51 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters