-
Notifications
You must be signed in to change notification settings - Fork 2
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.
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
Feature/#141-알림 목록 조회 API 구현 #162
The head ref may contain hidden characters: "Feature/#141-\uC54C\uB9BC_\uBAA9\uB85D_\uC870\uD68C_API_\uAD6C\uD604"
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
전반적으로 고칠부분은 없는 것 같아요 수고하셨습니다.
|
||
@Sql("/data-test.sql") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ServiceTest에서는 ServiceIntegerationTest를 상속받고 있어서, 굳이 @Sql(data-test.sql)을 명시할 필요는 없을거에요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ㅇㅈ
receiver); | ||
|
||
//then | ||
assertThat(notifications).extracting("message", String.class) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요기는 Notification::getMessage를 사용했어도 좋았을 것 같아요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다!
크게 수정할 부분은 보이지 않네요~~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다 ~~ 딱히 수정할 부분이 없어보입니다!
|
||
@Sql("/data-test.sql") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ㅇㅈ
#️⃣연관된 이슈
close #141
📝작업 내용
알림 목록 조회 api 구현
스크린샷 (선택)
예상 소요 시간 및 실제 소요 시간
1시간/1시간