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

[고도화] 댓글 기능 고도화 - 1차 대댓글: 쿼리 업데이트 #68

Closed
Tracked by #66
lmw7414 opened this issue Jan 8, 2023 · 0 comments
Closed
Tracked by #66

[고도화] 댓글 기능 고도화 - 1차 대댓글: 쿼리 업데이트 #68

lmw7414 opened this issue Jan 8, 2023 · 0 comments
Assignees
@lmw7414
Labels
enhancement New feature or request
Milestone
게시판 서비스 만들기

Comments

@lmw7414
Copy link
Owner

lmw7414 commented Jan 8, 2023 •
edited
Loading

1차 대댓글에 필요한 쿼리 변경이 있는지 살피고, 업데이트한다.
@lmw7414 lmw7414 mentioned this issue Jan 8, 2023
Closed
4 tasks
@lmw7414 lmw7414 self-assigned this Jan 8, 2023
@lmw7414 lmw7414 added the enhancement New feature or request label Jan 8, 2023
@lmw7414 lmw7414 added this to the 게시판 서비스 만들기 milestone Jan 8, 2023
@lmw7414 lmw7414 changed the title 쿼리 업데이트 [고도화] 댓글 기능 고도화 - 1차 대댓글: 쿼리 업데이트 Jan 8, 2023
lmw7414 added a commit that referenced this issue Jan 8, 2023
@lmw7414
#68 - 대댓글 테스트 데이터 업데이트 + 대댓글 쿼리 동작을 확인하기 위한 JPA 테스트 추가
a3e0da0 
새로 도메인이 바뀌었으므로 테스트 데이터 밀어넣는
쿼리문도 대응을 해주어야 한다.
또한 jpa 테스트를 통해 사용할 쿼리 기능들을 미리 테스트함
@lmw7414 lmw7414 mentioned this issue Jan 8, 2023
Merged
lmw7414 added a commit that referenced this issue Jan 8, 2023
@lmw7414
#68 - 대댓글 테스트 데이터 업데이트 + 대댓글 쿼리 동작을 확인하기 위한 JPA 테스트 추가
38b098e 
새로 도메인이 바뀌었으므로 테스트 데이터 밀어넣는
쿼리문도 대응을 해주어야 한다.
또한 jpa 테스트를 통해 사용할 쿼리 기능들을 미리 테스트함
@lmw7414 lmw7414 closed this as completed Jan 8, 2023
oyoungsun pushed a commit to oyoungsun/fastcampus-project-board-1 that referenced this issue Jan 9, 2023
lmw7414#68-update
7b47a03
oyoungsun added a commit to oyoungsun/fastcampus-project-board-1 that referenced this issue Jan 9, 2023
@oyoungsun
Merge pull request #2 from oyoungsun/feature/lmw7414#68-new-doamin
551d1b3 
Merge pull request #1 from oyoungsun/feature/lmw7414#67-usecase-update
oyoungsun pushed a commit to oyoungsun/fastcampus-project-board-1 that referenced this issue Jan 16, 2023
lmw7414#70 - 게시글 서비스의 해시태그 기능 변경점 반영
5d5752d 
테스트의 변화는 lmw7414#68 의 e9f36a4 에서 도메인 변경을 하면서
이미 구상하여 작업하였음

주로 게시글이 저장, 수정, 삭제될 때
해시태그가 함께 어떻게 움직여야 하는지를 묘사한다.

강의 진행하면서 도메인 변경이 얼마나 넓은 영역에
영향을 미치는지 보여주면서, 컴파일 에러를 해소하기 위해
일단 테스트 변경까지 모두 반영하게 되었다.

이런 과정을 거치면서
코드 작성 과정을 잘 구분하고 보여줄 시간이 부족하여
이런 커밋들이 발생했는데,
실전에서 이렇게 작업 내역을 나누면
커밋으로 이력을 추적하면서 볼 때 다소 불편할 것 같다.
또한 사이의 커밋이 테스트 실패하는
불완전한 상태로 진행된 것도 문제.

서비스의 변경은 테스트 변경과 함께 담길 수 있도록 하자.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lmw7414 lmw7414

Labels
enhancement New feature or request
Projects
None yet
Milestone
게시판 서비스 만들기
Development

No branches or pull requests
1 participant
@lmw7414