-
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
Implement expiration notifications Allow and Deny API #143
Implement expiration notifications Allow and Deny API #143
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.
확인했습니다!
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.
확인했습니다.
* 만료 알림 수신 허용 테스트 | ||
*/ | ||
@Test | ||
@WithMockUser(username = "이진우", roles = "USER") |
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.
P4 : 다음 번에는 roles를 우리가 정의한 role로 설정해서 해보면 한층 더 정교한(role에 따라 접근 권한이 다른 api일 수 있으니) 테스트가 될 것 같습니다.
PR Type
Motivation
기존 알림동의하기 관리 진행할 시, pending 처리 후 새로고침이 안됨
Problem Solving
To Reviewer
요청, 응답 형식은 postman을 참고해주세요.