refactor: null 체크

JECT-Study · Mar 30, 2024 · 386e882 · 386e882
1 parent 57d68f7
commit 386e882
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/main/java/balancetalk/module/member/application/MemberService.java b/src/main/java/balancetalk/module/member/application/MemberService.java
@@ -127,8 +127,10 @@ public void delete(final LoginRequest loginRequest, HttpServletRequest request)
         }
 
         List<Post> posts = member.getPosts();
-        for (Post post : posts) {
-            post.removeMember();
+        if (posts != null) {
+            for (Post post : posts) {
+                post.removeMember();
+            }
         }
         memberRepository.deleteByEmail(member.getEmail());
     }

diff --git a/src/test/java/balancetalk/module/member/application/MemberServiceTest.java b/src/test/java/balancetalk/module/member/application/MemberServiceTest.java
@@ -295,6 +295,7 @@ void deleteMemberSuccess() {
         when(passwordEncoder.matches(loginRequest.getPassword(), member.getPassword())).thenReturn(true);
 
         // when
+
         memberService.delete(loginRequest, request);
 
         // then
-Original file line number
+Diff line change
@@ Expand Up / @@ -295,6 +295,7 @@ void deleteMemberSuccess() { @@
             when(passwordEncoder.matches(loginRequest.getPassword(), member.getPassword())).thenReturn(true);
             // when
             memberService.delete(loginRequest, request);
             // then
@@ Expand Down @@