Skip to content

Commit

Permalink
[Jt-78] docs: 문서 수정
Browse files Browse the repository at this point in the history
  • Loading branch information
Shin-Jae-Yoon committed Sep 21, 2023
1 parent 336904c commit b7d27ad
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 15 deletions.
34 changes: 34 additions & 0 deletions docs/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
## 📌 개발 내용

수정 or 추가, 개발된 내용에 대해서 명확하게 작성바랍니다.

## 📑 PR 포인트

좀 더 상세하게 리뷰 받고 싶은 부분을 기재해주세요.

<br/>

### 👥 협업을 위한 코드리뷰

1. 세상에 바보같은 질문은 없다.
2. 실수를 예방하는 팀이 아니라, 실수를 잘 다루는 팀이 되자.
3. 분업이 아닌 협업을 하자. 우린 모두 같은 문제를 이겨내기 위해 모였다.
4. 영원한 것은 없으니 대화를 하자.
5. 서로 존중하자.
6. 많이 부딪치고 깨닫고 배우자.
7. 부드럽게 설득하고 열린 마음으로 설득당해보자.

### ✅ 리마인더

- 본인의 로컬에서 정상 동작하는지 확인해주세요.
- 최신 브랜치를 Pull 받고 PR을 요청했는지 확인해주세요.
- API가 추가되었을 경우 테스트를 하고 PR을 올려주세요.
- Conflict가 났을 때, UI상에서 해결하지 말고, 본인 local에서 해결해주세요.
- Commit 메시지 제대로 작성해주세요.

### ⚙️ 코드리뷰 룰

- R(Request Change): 해당 블럭은 꼭 변경해주셨으면 좋겠습니다.
- C(Comment): 웬만하면 고려해주시면 좋겠습니다.
- Q(Question) : 해당 라인이 궁금합니다.
- A(Approve): 반영해도 좋고 넘어가도 좋습니다. 혹은 사소한 의견입니다.
Original file line number Diff line number Diff line change
Expand Up @@ -20,21 +20,6 @@ class SmtpServiceTest {
@Mock
JavaMailSenderImpl javaMailSender;

// @Test
// @DisplayName("SMTP 연결 성공")
// void smtp_connection_success() {
// // when, then
// assertThatCode(() -> javaMailSender.testConnection()).doesNotThrowAnyException();
// }
//
// @Test
// @DisplayName("SMTP 연결 실패")
// void smtp_connection_fail() throws MessagingException {
// // when, then
// JavaMailSenderImpl emptyJavaMailSender = new JavaMailSenderImpl();
// assertThatThrownBy(emptyJavaMailSender::testConnection).isInstanceOf(MessagingException.class);
// }

@Test
@DisplayName("인증 Mail 생성 성공")
void create_authentication_mail_success() {
Expand Down

0 comments on commit b7d27ad

Please sign in to comment.