From 3198ef5976d6d0f7c6b80d3b4e78b1993089dc07 Mon Sep 17 00:00:00 2001
From: eojinsarang joo <wndjwlstkfkd@naver.com>
Date: Fri, 30 Sep 2022 02:10:55 +0900
Subject: [PATCH 1/2] =?UTF-8?q?refactor:=20=EA=B0=80=EC=A1=B1=20=EC=A1=B0?=
 =?UTF-8?q?=ED=9A=8C=20=EC=8B=9C=20=EA=B0=80=EC=A1=B1=20=EC=97=86=EC=96=B4?=
 =?UTF-8?q?=EB=8F=84=20=EB=B9=88=20=EA=B0=92=20=EB=A6=AC=ED=84=B4=20#245?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../com/ceos/bankids/mapper/FamilyMapper.java | 20 ++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/src/main/java/com/ceos/bankids/mapper/FamilyMapper.java b/src/main/java/com/ceos/bankids/mapper/FamilyMapper.java
index d344648e..09ac55ca 100644
--- a/src/main/java/com/ceos/bankids/mapper/FamilyMapper.java
+++ b/src/main/java/com/ceos/bankids/mapper/FamilyMapper.java
@@ -23,6 +23,7 @@
 import java.util.Collections;
 import java.util.Comparator;
 import java.util.List;
+import java.util.Optional;
 import java.util.stream.Collectors;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
@@ -54,14 +55,19 @@ public FamilyDTO createFamily(User user) {
 
     @Transactional(readOnly = true)
     public FamilyDTO readFamily(User user) {
-        FamilyUser familyUser = familyUserService.findByUser(user);
-        Family family = familyUser.getFamily();
-        List<FamilyUser> familyUserList = familyUserService.getFamilyUserListExclude(family,
-            user);
+        Optional<FamilyUser> familyUser = familyUserService.findByUserNullable(user);
 
-        return new FamilyDTO(family, familyUserList.stream()
-            .map(FamilyUserDTO::new)
-            .collect(Collectors.toList()));
+        if (familyUser.isEmpty()) {
+            return new FamilyDTO(new Family(), List.of());
+        } else {
+            Family family = familyUser.get().getFamily();
+            List<FamilyUser> familyUserList = familyUserService.getFamilyUserListExclude(family,
+                user);
+
+            return new FamilyDTO(family, familyUserList.stream()
+                .map(FamilyUserDTO::new)
+                .collect(Collectors.toList()));
+        }
     }
 
     @Transactional(readOnly = true)

From ba8c0796fef2ac3f631d7242645e6f65616ed062 Mon Sep 17 00:00:00 2001
From: eojinsarang joo <wndjwlstkfkd@naver.com>
Date: Fri, 30 Sep 2022 02:24:32 +0900
Subject: [PATCH 2/2] =?UTF-8?q?refactor:=20=EA=B0=80=EC=A1=B1=20=EC=97=86?=
 =?UTF-8?q?=EC=9D=84=20=EB=95=8C=20test=20=EB=B3=80=EA=B2=BD=20#245?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../unit/controller/ChallengeControllerTest2.java   |  5 +----
 .../unit/controller/FamilyControllerTest.java       | 13 ++++++++-----
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/test/java/com/ceos/bankids/unit/controller/ChallengeControllerTest2.java b/src/test/java/com/ceos/bankids/unit/controller/ChallengeControllerTest2.java
index 16e91606..88d15056 100644
--- a/src/test/java/com/ceos/bankids/unit/controller/ChallengeControllerTest2.java
+++ b/src/test/java/com/ceos/bankids/unit/controller/ChallengeControllerTest2.java
@@ -14,7 +14,6 @@
 import com.ceos.bankids.repository.ProgressRepository;
 import com.ceos.bankids.repository.TargetItemRepository;
 import com.ceos.bankids.service.ChallengeServiceImpl;
-import com.ceos.bankids.service.FamilyServiceImpl;
 import java.util.Optional;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -112,9 +111,7 @@ public void testIfPostChallengeReqSuccessResponse() {
             mockProgressRepository,
             mockCommentRepository
         );
-        FamilyServiceImpl familyService = new FamilyServiceImpl()
-
-
+//        FamilyServiceImpl familyService = new FamilyServiceImpl()
 
     }
 }
diff --git a/src/test/java/com/ceos/bankids/unit/controller/FamilyControllerTest.java b/src/test/java/com/ceos/bankids/unit/controller/FamilyControllerTest.java
index adf42d74..2e76b015 100644
--- a/src/test/java/com/ceos/bankids/unit/controller/FamilyControllerTest.java
+++ b/src/test/java/com/ceos/bankids/unit/controller/FamilyControllerTest.java
@@ -334,8 +334,8 @@ public void testIfFamilyExistThenReturnGetResult() {
     }
 
     @Test
-    @DisplayName("조회 시 기존 가족 있으나, 삭제되었을 때 에러 처리 하는지 확인")
-    public void testIfFamilyExistedButDeletedWhenGetThenThrowBadRequestException() {
+    @DisplayName("조회 시 기존 가족 없을 때, 빈 가족 정보 리턴 하는지 확인")
+    public void testIfFamilyNotExistThenReturnResult() {
         // given
         User user1 = User.builder()
             .id(1L)
@@ -393,10 +393,13 @@ public void testIfFamilyExistedButDeletedWhenGetThenThrowBadRequestException() {
         );
         FamilyController familyController = new FamilyController(familyMapper);
 
+        CommonResponse<FamilyDTO> result = familyController.getFamily(user1);
+
         // then
-        Assertions.assertThrows(BadRequestException.class, () -> {
-            familyController.getFamily(user1);
-        });
+        FamilyDTO familyDTO = FamilyDTO.builder()
+            .family(new Family())
+            .familyUserList(List.of()).build();
+        Assertions.assertEquals(CommonResponse.onSuccess(familyDTO), result);
     }
 
     @Test