fix(UserService): 修复更新用户信息异常的问题

更新用户信息时使用 BeanCopy 复制 DTO，DTO 中的 null 字段也被
复制到当前已登录用户信息中，导致信息缺失

Author: juzi214032

src/main/java/io/github/talelin/latticy/service/impl/UserServiceImpl.java

@@ -101,10 +101,27 @@ public UserDO updateUserInfo(UpdateInfoDTO dto) {
             if (exist) {
                 throw new ForbiddenException(10071);
             }
+
+            boolean changeSuccess = userIdentityService.changeUsername(user.getId(), dto.getUsername());
+            if (changeSuccess) {
+                user.setUsername(dto.getUsername());
+            }
+        }
+
+        // todo 增加工具类实现忽略 null 的 BeanCopy,简化这段代码
+        if (dto.getUsername() != null) {
             user.setUsername(dto.getUsername());
-            userIdentityService.changeUsername(user.getId(), dto.getUsername());
         }
-        BeanUtils.copyProperties(dto, user);
+        if (dto.getAvatar() != null) {
+            user.setAvatar(dto.getAvatar());
+        }
+        if (dto.getEmail() != null) {
+            user.setEmail(dto.getEmail());
+        }
+        if (dto.getNickname() != null) {
+            user.setNickname(dto.getNickname());
+        }
+
         this.baseMapper.updateById(user);
         return user;
     }

src/test/java/io/github/talelin/latticy/service/impl/UserServiceImplTest.java

@@ -134,16 +134,13 @@ public void updateUserInfoNoUsername() {
 
     @Test
     public void changeUserPassword() {
-        // TODO 待解决 单独执行不会抛出空指针异常，批量执行则会抛出空指针异常。
-        assertThrows(NullPointerException.class, () -> {
-            ChangePasswordDTO dto = new ChangePasswordDTO();
-            dto.setNewPassword("147258");
-            dto.setConfirmPassword("147258");
-            dto.setOldPassword("123456");
-            UserDO user = userService.changeUserPassword(dto);
-            boolean b = userIdentityService.verifyUsernamePassword(user.getId(), "pedro", "147258");
-            assertTrue(b);
-        });
+        ChangePasswordDTO dto = new ChangePasswordDTO();
+        dto.setNewPassword("147258");
+        dto.setConfirmPassword("147258");
+        dto.setOldPassword("123456");
+        UserDO user = userService.changeUserPassword(dto);
+        boolean b = userIdentityService.verifyUsernamePassword(user.getId(), "pedro", "147258");
+        assertTrue(b);
     }
 
     @Test
@@ -204,4 +201,4 @@ public void checkCreateAndDeleteUser() {
         UserDO newUser = userService.createUser(dto);
         assertEquals(newUser.getUsername(), "pedro111");
     }
-}
+}