Merge pull request Team-TenTen#200 from Team-TenTen/fix/test

[Fix]: Integration 서비스 테스트 시 `@MockBean`으로 인한 테스트 속도 저하 문제 해결.

src/test/java/com/tenten/linkhub/IntegrationApplicationTest.java

@@ -0,0 +1,15 @@
+package com.tenten.linkhub;
+
+import com.tenten.linkhub.global.aws.s3.ImageFileUploader;
+import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.boot.test.mock.mockito.MockBean;
+import org.springframework.test.context.ActiveProfiles;
+
+@ActiveProfiles("test")
+@SpringBootTest
+public abstract class IntegrationApplicationTest {
+
+    @MockBean
+    protected ImageFileUploader mockImageFileUploader;
+
+}

src/test/java/com/tenten/linkhub/domain/member/service/MemberServiceTest.java

@@ -1,5 +1,6 @@
 package com.tenten.linkhub.domain.member.service;
 
+import com.tenten.linkhub.IntegrationApplicationTest;
 import com.tenten.linkhub.domain.auth.JwtProvider;
 import com.tenten.linkhub.domain.auth.MemberDetails;
 import com.tenten.linkhub.domain.member.model.Member;
@@ -21,7 +22,6 @@
 import com.tenten.linkhub.domain.member.service.dto.MemberUpdateResponse;
 import com.tenten.linkhub.domain.space.model.category.Category;
 import com.tenten.linkhub.global.aws.dto.ImageInfo;
-import com.tenten.linkhub.global.aws.s3.ImageFileUploader;
 import com.tenten.linkhub.global.exception.DataDuplicateException;
 import com.tenten.linkhub.global.exception.DataNotFoundException;
 import org.assertj.core.api.Assertions;
@@ -31,12 +31,9 @@
 import org.junit.jupiter.api.Test;
 import org.mockito.BDDMockito;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.boot.test.mock.mockito.MockBean;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.mock.web.MockMultipartFile;
 import org.springframework.security.core.Authentication;
-import org.springframework.test.context.ActiveProfiles;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.Optional;
@@ -45,20 +42,15 @@
 import static org.assertj.core.api.AssertionsForClassTypes.assertThatThrownBy;
 import static org.mockito.ArgumentMatchers.any;
 
-@SpringBootTest
 @Transactional
-@ActiveProfiles("test")
-class MemberServiceTest {
+class MemberServiceTest extends IntegrationApplicationTest {
 
     @Autowired
     private MemberService memberService;
 
     @Autowired
     private MemberEmailRedisRepository redisRepository;
 
-    @MockBean
-    private ImageFileUploader mockImageFileUploader;
-
     @Autowired
     private JwtProvider jwtProvider;
 

src/test/java/com/tenten/linkhub/domain/notification/service/NotificationServiceTest.java

@@ -1,5 +1,6 @@
 package com.tenten.linkhub.domain.notification.service;
 
+import com.tenten.linkhub.IntegrationApplicationTest;
 import com.tenten.linkhub.domain.member.model.FavoriteCategory;
 import com.tenten.linkhub.domain.member.model.Member;
 import com.tenten.linkhub.domain.member.model.ProfileImage;
@@ -20,20 +21,16 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.data.domain.PageRequest;
-import org.springframework.test.context.ActiveProfiles;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.ArrayList;
 import java.util.List;
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-@ActiveProfiles("test")
-@SpringBootTest
 @Transactional
-public class NotificationServiceTest {
+public class NotificationServiceTest extends IntegrationApplicationTest {
 
     @Autowired
     private NotificationService notificationService;

src/test/java/com/tenten/linkhub/domain/space/facade/CommentFacadeTest.java

@@ -1,5 +1,6 @@
 package com.tenten.linkhub.domain.space.facade;
 
+import com.tenten.linkhub.IntegrationApplicationTest;
 import com.tenten.linkhub.domain.member.model.FavoriteCategory;
 import com.tenten.linkhub.domain.member.model.Member;
 import com.tenten.linkhub.domain.member.model.ProfileImage;
@@ -22,20 +23,16 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.data.domain.PageRequest;
-import org.springframework.test.context.ActiveProfiles;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.List;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
-@ActiveProfiles("test")
 @Transactional
-@SpringBootTest
-class CommentFacadeTest {
+class CommentFacadeTest extends IntegrationApplicationTest {
 
     @Autowired
     private CommentFacade commentFacade;

src/test/java/com/tenten/linkhub/domain/space/facade/LinkFacadeTest.java

@@ -1,5 +1,6 @@
 package com.tenten.linkhub.domain.space.facade;
 
+import com.tenten.linkhub.IntegrationApplicationTest;
 import com.tenten.linkhub.domain.member.model.FavoriteCategory;
 import com.tenten.linkhub.domain.member.model.Member;
 import com.tenten.linkhub.domain.member.model.ProfileImage;
@@ -28,19 +29,15 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.data.domain.PageRequest;
-import org.springframework.test.context.ActiveProfiles;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.Optional;
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-@ActiveProfiles("test")
 @Transactional
-@SpringBootTest
-class LinkFacadeTest {
+class LinkFacadeTest extends IntegrationApplicationTest {
 
     @Autowired
     private LinkFacade linkFacade;

src/test/java/com/tenten/linkhub/domain/space/facade/SpaceFacadeTest.java

@@ -1,5 +1,6 @@
 package com.tenten.linkhub.domain.space.facade;
 
+import com.tenten.linkhub.IntegrationApplicationTest;
 import com.tenten.linkhub.domain.member.model.FavoriteCategory;
 import com.tenten.linkhub.domain.member.model.Member;
 import com.tenten.linkhub.domain.member.model.ProfileImage;
@@ -23,21 +24,16 @@
 import com.tenten.linkhub.domain.space.repository.scrap.ScrapRepository;
 import com.tenten.linkhub.domain.space.repository.space.SpaceJpaRepository;
 import com.tenten.linkhub.domain.space.service.LinkService;
-import com.tenten.linkhub.domain.space.service.SpaceService;
 import com.tenten.linkhub.domain.space.service.dto.link.LinkCreateRequest;
 import com.tenten.linkhub.global.aws.dto.ImageInfo;
-import com.tenten.linkhub.global.aws.s3.ImageFileUploader;
 import com.tenten.linkhub.global.exception.PolicyViolationException;
 import com.tenten.linkhub.global.exception.UnauthorizedAccessException;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 import org.mockito.BDDMockito;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.boot.test.mock.mockito.MockBean;
 import org.springframework.mock.web.MockMultipartFile;
-import org.springframework.test.context.ActiveProfiles;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.ArrayList;
@@ -48,10 +44,8 @@
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.mockito.ArgumentMatchers.any;
 
-@ActiveProfiles("test")
 @Transactional
-@SpringBootTest
-class SpaceFacadeTest {
+class SpaceFacadeTest extends IntegrationApplicationTest {
 
     @Autowired
     private SpaceFacade spaceFacade;
@@ -74,9 +68,6 @@ class SpaceFacadeTest {
     @Autowired
     private ScrapRepository scrapRepository;
 
-    @MockBean
-    private ImageFileUploader mockImageFileUploader;
-
     private Long myMemberId;
     private Long anotherMemberId;
 

src/test/java/com/tenten/linkhub/domain/space/facade/SpaceInvitationFacadeTest.java

@@ -1,5 +1,6 @@
 package com.tenten.linkhub.domain.space.facade;
 
+import com.tenten.linkhub.IntegrationApplicationTest;
 import com.tenten.linkhub.domain.member.model.FavoriteCategory;
 import com.tenten.linkhub.domain.member.model.Member;
 import com.tenten.linkhub.domain.member.model.ProfileImage;
@@ -24,17 +25,13 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.test.context.ActiveProfiles;
 import org.springframework.transaction.annotation.Transactional;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
-@ActiveProfiles("test")
 @Transactional
-@SpringBootTest
-class SpaceInvitationFacadeTest {
+class SpaceInvitationFacadeTest extends IntegrationApplicationTest {
 
     @Autowired
     private SpaceInvitationFacade spaceInvitationFacade;

src/test/java/com/tenten/linkhub/domain/space/service/CommentServiceTest.java

@@ -1,5 +1,6 @@
 package com.tenten.linkhub.domain.space.service;
 
+import com.tenten.linkhub.IntegrationApplicationTest;
 import com.tenten.linkhub.domain.member.model.FavoriteCategory;
 import com.tenten.linkhub.domain.member.model.Member;
 import com.tenten.linkhub.domain.member.model.ProfileImage;
@@ -22,19 +23,15 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.test.context.ActiveProfiles;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.Optional;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
-@ActiveProfiles("test")
 @Transactional
-@SpringBootTest
-class CommentServiceTest {
+class CommentServiceTest extends IntegrationApplicationTest {
 
     @Autowired
     private CommentService commentService;
@@ -251,7 +248,7 @@ void createReply_DataNotFoundException() {
                 .isInstanceOf(DataNotFoundException.class);
     }
 
-    private void setUpTestData(){
+    private void setUpTestData() {
         Member member1 = new Member(
                 "testSocialId",
                 Provider.kakao,
@@ -297,7 +294,7 @@ private void setUpTestData(){
                 "두번째 스페이스",
                 "두번째 스페이스 소개글",
                 Category.ETC,
-                new SpaceImage( "https://testimage2", "테스트 이미지2"),
+                new SpaceImage("https://testimage2", "테스트 이미지2"),
                 new SpaceMember(setUpMemberId1, Role.OWNER),
                 true,
                 false,

src/test/java/com/tenten/linkhub/domain/space/service/DefaultLinkServiceTest.java

@@ -1,5 +1,6 @@
 package com.tenten.linkhub.domain.space.service;
 
+import com.tenten.linkhub.IntegrationApplicationTest;
 import com.tenten.linkhub.domain.member.model.FavoriteCategory;
 import com.tenten.linkhub.domain.member.model.Member;
 import com.tenten.linkhub.domain.member.model.ProfileImage;
@@ -25,20 +26,16 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Sort;
-import org.springframework.test.context.ActiveProfiles;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.List;
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-@ActiveProfiles("test")
-@SpringBootTest
 @Transactional
-class DefaultLinkServiceTest {
+class DefaultLinkServiceTest extends IntegrationApplicationTest {
 
     @Autowired
     private LinkService linkService;

src/test/java/com/tenten/linkhub/domain/space/service/DefaultSpaceServiceTest.java

@@ -1,5 +1,6 @@
 package com.tenten.linkhub.domain.space.service;
 
+import com.tenten.linkhub.IntegrationApplicationTest;
 import com.tenten.linkhub.domain.member.model.FavoriteCategory;
 import com.tenten.linkhub.domain.member.model.Member;
 import com.tenten.linkhub.domain.member.model.ProfileImage;
@@ -26,21 +27,17 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Sort;
-import org.springframework.test.context.ActiveProfiles;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.util.List;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
-@ActiveProfiles("test")
 @Transactional
-@SpringBootTest
-class DefaultSpaceServiceTest {
+class DefaultSpaceServiceTest extends IntegrationApplicationTest {
 
     @Autowired
     private SpaceService spaceService;

src/test/java/com/tenten/linkhub/domain/space/service/FavoriteServiceTest.java

@@ -1,5 +1,6 @@
 package com.tenten.linkhub.domain.space.service;
 
+import com.tenten.linkhub.IntegrationApplicationTest;
 import com.tenten.linkhub.domain.member.model.FavoriteCategory;
 import com.tenten.linkhub.domain.member.model.Member;
 import com.tenten.linkhub.domain.member.model.ProfileImage;
@@ -23,9 +24,7 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.data.domain.PageRequest;
-import org.springframework.test.context.ActiveProfiles;
 
 import java.util.List;
 import java.util.concurrent.CountDownLatch;
@@ -35,9 +34,7 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
-@ActiveProfiles("test")
-@SpringBootTest
-class FavoriteServiceTest {
+class FavoriteServiceTest extends IntegrationApplicationTest {
 
     @Autowired
     private FavoriteService favoriteService;

src/test/java/com/tenten/linkhub/domain/space/service/LikeServiceTest.java

@@ -1,5 +1,6 @@
 package com.tenten.linkhub.domain.space.service;
 
+import com.tenten.linkhub.IntegrationApplicationTest;
 import com.tenten.linkhub.domain.member.model.FavoriteCategory;
 import com.tenten.linkhub.domain.member.model.Member;
 import com.tenten.linkhub.domain.member.model.ProfileImage;
@@ -20,8 +21,6 @@
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.test.context.ActiveProfiles;
 import org.springframework.test.context.jdbc.Sql;
 
 import java.util.List;
@@ -31,11 +30,9 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-@SpringBootTest
-@ActiveProfiles("test")
 @Slf4j
 @Sql(scripts = "/sql/clean_up.sql", executionPhase = Sql.ExecutionPhase.AFTER_TEST_METHOD)
-public class LikeServiceTest {
+public class LikeServiceTest extends IntegrationApplicationTest {
     private static int THREAD_COUNT = 100;
 
     @Autowired