search

f-lab-edu / awesome-wiki

[연봉 5000만원 이상 이직] Spring framework 를 이용한 다중 접속 Wiki
15 stars 4 forks source link

문서 편집이력 조회 기능 #45

Closed kmmin78 closed 3 years ago

kmmin78 commented 3 years ago

주요 내용

  1. 문서 편집이력 조회 기능을 usecase 인터페이스로 정의함으로써, 해당 프로젝트에 어떤 usecase가 있는지 보다 쉽게 파악할 수 있도록 함. 참고할만한 내용 - https://blog.cleancoder.com/uncle-bob/2011/09/30/Screaming-Architecture.html

  2. 위 사항에 따른 인터페이스, 테스트코드 분리 및 생성

  3. DocumentHistory 도메인 추가 (property = readonly)

  4. Document의 property는 변경 가능하도록 수정

Closes #44

FrancescoJo commented 3 years ago

테스트 충실하게 잘 만들어 주셔서 뿌듯하네요. 코멘트 한번 확인해 보시고, 좀더 고칠 부분 있나 고민해 보세요. 딱히 크게 바꿀 내용은 없으므로 이대로도 괜찮다면 저는 OK 입니다.

PR 에 적어두신 내용은 추후 README 에도 반드시 업데이트 해주세요!!

kmmin78 commented 3 years ago

@neropsys 저도 초기에 그렇게 생각했었는데, DocumentHistory 개념이 들어가면서 좀 혼동이 왔었네요... 문서의 최초 생성부터 title 변경도 이력으로 남겨야 된다고 생각했었는데, 말씀하신대로 전혀 다른 Document로 생각한다면 val이 맞을 것 같습니다. 수정해놓을게요 ㅎㅎ

FrancescoJo commented 3 years ago

@neropsys 님은 이 브랜치에서 작성하셨던 스타일로 테스트 코드를 작성해 주세요. #47 는 닫고, 테스트 올려주시면 병합하시죠.