実装+テスト修正 (不要と思われる改行削除)

moneyforward · Feb 21, 2023 · 580885c · 580885c
1 parent f2788dd
commit 580885c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/app/internal/domain/service/slack_response.go b/app/internal/domain/service/slack_response.go
@@ -76,9 +76,9 @@ func (s *slackResponseService) ReplyEmailList(ctx context.Context, event *slacke
 	if len(emails) <= lineSizeOfPostEmailList-1 {
 		var b strings.Builder
 		b.Grow(len(emails) + 1)
-		b.WriteString("参加者一覧\n")
+		b.WriteString("参加者一覧")
 		for _, email := range emails {
-			b.WriteString(email.Email)
+			b.WriteString("\n" + email.Email)
 		}
 		return s.slackRepository.PostMessage(ctx, event.Channel, b.String(), event.ThreadTimeStamp)
 	}

diff --git a/app/internal/domain/service/slack_response_test.go b/app/internal/domain/service/slack_response_test.go
@@ -157,7 +157,7 @@ func Test_slackErrorResponseService_ReplyEmailList(t *testing.T) {
 			},
 			prepare: func(msr *mock_repository.MockSlackRepository) {
 				msr.EXPECT().PostMessage(gomock.Any(), "sampleChannel",
-					"参加者一覧\n[email protected]\n[email protected]\n",
+					"参加者一覧\n[email protected]\n[email protected]",
 					"sampleThreadTimeStamp").Return(nil)
 			},
 		},
@@ -175,7 +175,7 @@ func Test_slackErrorResponseService_ReplyEmailList(t *testing.T) {
 			},
 			prepare: func(msr *mock_repository.MockSlackRepository) {
 				msr.EXPECT().PostMessage(gomock.Any(), "sampleChannel",
-					"参加者一覧\n[email protected]\n[email protected]\n",
+					"参加者一覧\n[email protected]\n[email protected]",
 					"sampleThreadTimeStamp").Return(errors.New("sample error"))
 			},
 			wantErr: true,