Skip to content

Commit 10839e5

Browse files
committed
refactor: 테스트 코드 간결화 및 가독성 증진
#1397
1 parent 0137acb commit 10839e5

File tree

1 file changed

+13
-19
lines changed

1 file changed

+13
-19
lines changed

backend/src/test/java/wooteco/prolog/roadmap/application/RecommendedPostServiceTest.java

+13-19
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,6 @@
1616
import wooteco.prolog.session.domain.Session;
1717
import wooteco.prolog.session.domain.repository.SessionRepository;
1818

19-
import java.util.Optional;
20-
2119
import static org.assertj.core.api.Assertions.assertThat;
2220
import static org.assertj.core.api.SoftAssertions.assertSoftly;
2321

@@ -53,44 +51,40 @@ public void removeAll() {
5351
@DisplayName("추천 포스트 생성 테스트")
5452
void create() {
5553
//given
56-
final RecommendedRequest request = new RecommendedRequest("https//:example.com");
54+
final RecommendedRequest request = new RecommendedRequest("https://example.com");
5755

5856
//when
59-
Long recommendedPostId = recommendedPostService.create(keyword.getId(), request);
60-
61-
final Keyword persistedKeyword = keywordRepository.findById(keyword.getId()).get();
62-
final RecommendedPost persistedPost = recommendedPostRepository.findById(recommendedPostId).get();
57+
recommendedPostService.create(keyword.getId(), request);
6358

6459
//then
65-
assertSoftly(softAssertions -> {
66-
assertThat(persistedPost.getUrl()).isEqualTo(request.getUrl());
67-
assertThat(persistedKeyword.getRecommendedPosts()).containsExactly(persistedPost);
68-
});
60+
assertThat(recommendedPostRepository.findAll()).hasSize(1);
6961
}
7062

7163
@Test
7264
@DisplayName("추천 포스트 수정 테스트")
7365
void update() {
7466
//given
75-
final RecommendedRequest request = new RecommendedRequest("https//:example.com");
76-
Long recommendedPostId = recommendedPostService.create(keyword.getId(), request);
77-
String newUrl = "https//:example222.com";
78-
final RecommendedUpdateRequest updateRrequest = new RecommendedUpdateRequest(newUrl);
67+
final Long recommendedPostId = recommendedPostService.create(
68+
keyword.getId(),
69+
new RecommendedRequest("https://example.com"));
70+
71+
final String newUrl = "https://new.com";
72+
final RecommendedUpdateRequest updateRequest = new RecommendedUpdateRequest(newUrl);
7973

8074
//when
81-
recommendedPostService.update(recommendedPostId, updateRrequest);
82-
Optional<RecommendedPost> actual = recommendedPostRepository.findById(recommendedPostId);
75+
recommendedPostService.update(recommendedPostId, updateRequest);
8376

8477
//then
85-
assertThat(actual.get().getUrl()).isEqualTo(newUrl);
78+
final RecommendedPost post = recommendedPostRepository.findById(recommendedPostId).get();
79+
assertThat(post.getUrl()).isEqualTo(newUrl);
8680
}
8781

8882
@Test
8983
@DisplayName("추천 포스트 삭제 테스트")
9084
void delete() {
9185
//given
9286
final RecommendedRequest request = new RecommendedRequest("https://example.com");
93-
Long recommendedPostId = recommendedPostService.create(keyword.getId(), request);
87+
final Long recommendedPostId = recommendedPostService.create(keyword.getId(), request);
9488

9589
//when
9690
recommendedPostService.delete(recommendedPostId);

0 commit comments

Comments
 (0)