[refactor] #32 코드 통일 및 보수 (#37)

* refactor: 필드 주입 대신 생성자 주입으로 변경

* refactor: 개행 및 EOL 추가

* refactor: 서류, 최종 메일 분리 및 비동기 처리

* refactor: 빌더 생성자로 변경

* test: displayName 추가

* fix: 스레드 작업 대기 시간 연장

* feat: 메일 예외 추가

* feat: 메일 병렬처리 추가

* fix: 대기시간 연장

* refactor: 메일 예약, 메일 전송 로직 분리

* fix: 메일 전송 테스트 비활성화

src/main/java/land/leets/domain/admin/domain/Admin.java

@@ -1,16 +1,22 @@
 package land.leets.domain.admin.domain;
 
-import jakarta.persistence.*;
+import jakarta.persistence.Column;
+import jakarta.persistence.Entity;
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.Id;
 import land.leets.domain.shared.BaseTimeEntity;
-import lombok.*;
+import lombok.AccessLevel;
+import lombok.AllArgsConstructor;
+import lombok.Builder;
+import lombok.Getter;
+import lombok.NoArgsConstructor;
 import org.hibernate.annotations.GenericGenerator;
 
 import java.util.UUID;
 
 @Entity(name = "admins")
 @Builder
 @Getter
-@Setter
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 @AllArgsConstructor
 public class Admin extends BaseTimeEntity {

src/main/java/land/leets/domain/admin/presentation/mapper/AdminMapper.java

@@ -2,10 +2,14 @@
 
 import land.leets.domain.admin.domain.Admin;
 import land.leets.domain.admin.presentation.dto.AdminDetailsResponse;
-import org.mapstruct.*;
+import org.mapstruct.BeanMapping;
+import org.mapstruct.Mapper;
+import org.mapstruct.NullValuePropertyMappingStrategy;
+import org.mapstruct.ReportingPolicy;
 
 @Mapper(componentModel = "spring", unmappedTargetPolicy = ReportingPolicy.IGNORE)
 public interface AdminMapper {
+
     @BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.IGNORE)
     AdminDetailsResponse mappingAdminToDto(Admin admin);
-}
+}

src/main/java/land/leets/domain/admin/usercase/AdminLoginImpl.java

@@ -11,9 +11,10 @@
 import org.springframework.security.crypto.password.PasswordEncoder;
 import org.springframework.stereotype.Service;
 
-@RequiredArgsConstructor
 @Service
+@RequiredArgsConstructor
 public class AdminLoginImpl implements AdminLogin {
+
     private final JwtProvider jwtProvider;
     private final AdminRepository adminRepository;
     private final PasswordEncoder passwordEncoder;

src/main/java/land/leets/domain/admin/usercase/AdminRefreshTokenImpl.java

@@ -9,9 +9,10 @@
 
 import java.util.UUID;
 
-@RequiredArgsConstructor
 @Service
+@RequiredArgsConstructor
 public class AdminRefreshTokenImpl implements AdminRefreshToken {
+
     private final JwtProvider jwtProvider;
 
     @Override

src/main/java/land/leets/domain/application/domain/Application.java

@@ -1,20 +1,30 @@
 package land.leets.domain.application.domain;
 
+import java.time.LocalDateTime;
 
-import jakarta.persistence.*;
+import jakarta.persistence.Column;
+import jakarta.persistence.Entity;
+import jakarta.persistence.EnumType;
+import jakarta.persistence.Enumerated;
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.GenerationType;
+import jakarta.persistence.Id;
+import jakarta.persistence.JoinColumn;
+import jakarta.persistence.OneToOne;
 import land.leets.domain.application.type.ApplicationStatus;
 import land.leets.domain.application.type.Position;
 import land.leets.domain.application.type.SubmitStatus;
 import land.leets.domain.shared.BaseTimeEntity;
 import land.leets.domain.user.domain.User;
-import lombok.*;
-
-import java.time.LocalDateTime;
+import lombok.AccessLevel;
+import lombok.AllArgsConstructor;
+import lombok.Builder;
+import lombok.Getter;
+import lombok.NoArgsConstructor;
 
 @Entity(name = "applications")
 @Builder
 @Getter
-@Setter
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 @AllArgsConstructor
 public class Application extends BaseTimeEntity {

src/main/java/land/leets/domain/application/exception/ApplicationAlreadyExistsException.java

@@ -7,4 +7,4 @@ public class ApplicationAlreadyExistsException extends ServiceException {
     public ApplicationAlreadyExistsException() {
         super(ErrorCode.APPLICATION_ALREADY_EXISTS);
     }
-}
+}

src/main/java/land/leets/domain/application/exception/ApplicationNotFoundException.java

@@ -7,4 +7,4 @@ public class ApplicationNotFoundException extends ServiceException {
     public ApplicationNotFoundException() {
         super(ErrorCode.APPLICATION_NOT_FOUND);
     }
-}
+}

src/main/java/land/leets/domain/application/presentation/mapper/ApplicationMapper.java

@@ -7,7 +7,12 @@
 import land.leets.domain.application.presentation.dto.StatusRequest;
 import land.leets.domain.interview.presentation.dto.res.InterviewDetailsResponse;
 import land.leets.domain.interview.presentation.dto.res.InterviewResponse;
-import org.mapstruct.*;
+import org.mapstruct.BeanMapping;
+import org.mapstruct.Mapper;
+import org.mapstruct.Mapping;
+import org.mapstruct.MappingTarget;
+import org.mapstruct.NullValuePropertyMappingStrategy;
+import org.mapstruct.ReportingPolicy;
 
 @Mapper(componentModel = "spring", unmappedTargetPolicy = ReportingPolicy.IGNORE)
 public interface ApplicationMapper {
@@ -23,4 +28,4 @@ public interface ApplicationMapper {
     @BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.IGNORE)
     @Mapping(target = "id", source = "application.id")
     ApplicationDetailsResponse mappingDetailsToDto(Application application, InterviewDetailsResponse interview, String phone);
-}
+}

src/main/java/land/leets/domain/application/type/ApplicationStatus.java

@@ -1,16 +1,26 @@
 package land.leets.domain.application.type;
 
+import java.util.List;
+
 import lombok.AllArgsConstructor;
 import lombok.Getter;
 
 @Getter
 @AllArgsConstructor
 public enum ApplicationStatus {
-    PASS_PAPER("서류 합격"),
-    FAIL_PAPER("서류 불합격"),
-    PASS("최종 합격"),
-    FAIL("최종 불합격"),
-    PENDING("보류");
+	PASS_PAPER("서류 합격"),
+	FAIL_PAPER("서류 불합격"),
+	PASS("최종 합격"),
+	FAIL("최종 불합격"),
+	PENDING("보류");
+
+	private String statusKo;
+
+	public static List<ApplicationStatus> finals() {
+		return List.of(PASS, FAIL);
+	}
 
-    private String statusKo;
-}
+	public static List<ApplicationStatus> papers() {
+		return List.of(PASS_PAPER, FAIL_PAPER);
+	}
+}

src/main/java/land/leets/domain/application/type/Position.java

@@ -12,4 +12,4 @@ public enum Position {
     UX_UI("UX/UI");
 
     private final String statusKo;
-}
+}

src/main/java/land/leets/domain/application/type/SubmitStatus.java

@@ -8,8 +8,7 @@
 public enum SubmitStatus {
     NONE("미제출"),
     SAVE("임시저장"),
-    SUBMIT("제출")
-    ;
+    SUBMIT("제출");
 
     private String statusKo;
-}
+}

src/main/java/land/leets/domain/application/usecase/CreateApplicationImpl.java

@@ -18,6 +18,7 @@
 @Service
 @RequiredArgsConstructor
 public class CreateApplicationImpl implements CreateApplication {
+
     private final ApplicationRepository applicationRepository;
     private final UserRepository userRepository;
 

src/main/java/land/leets/domain/application/usecase/GetAllApplicationImpl.java

@@ -15,6 +15,7 @@
 @Service
 @RequiredArgsConstructor
 public class GetAllApplicationImpl implements GetAllApplication {
+
     private final ApplicationRepository applicationRepository;
     private final ApplicationMapper applicationMapper;
     private final GetInterview getInterview;

src/main/java/land/leets/domain/auth/AdminAuthDetailsService.java

@@ -12,6 +12,7 @@
 @RequiredArgsConstructor
 @Service
 public class AdminAuthDetailsService implements UserDetailsService {
+
     private final AdminRepository adminRepository;
 
     @Override

src/main/java/land/leets/domain/auth/AuthDetails.java

@@ -14,6 +14,7 @@
 @Getter
 @AllArgsConstructor
 public class AuthDetails implements UserDetails {
+
     private final UUID uid;
     private final String email;
     private final AuthRole role;

src/main/java/land/leets/domain/auth/AuthService.java

@@ -19,10 +19,14 @@
 
 import java.io.IOException;
 import java.security.GeneralSecurityException;
-import java.util.*;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.Map;
+import java.util.Optional;
 
 @Service
 public class AuthService {
+
     private final String googleAuthUrl;
     private final String googleRedirectUrl;
     private final String googleClientId;
@@ -91,4 +95,4 @@ public User getUser(String idToken) throws GeneralSecurityException, IOException
 
         return userRepository.save(user);
     }
-}
+}

src/main/java/land/leets/domain/auth/UserAuthDetailsService.java

@@ -9,8 +9,8 @@
 import org.springframework.security.core.userdetails.UsernameNotFoundException;
 import org.springframework.stereotype.Service;
 
-@RequiredArgsConstructor
 @Service
+@RequiredArgsConstructor
 public class UserAuthDetailsService implements UserDetailsService {
     private final UserRepository userRepository;
 

src/main/java/land/leets/domain/auth/presentation/AuthController.java

@@ -24,6 +24,7 @@
 @RestController
 @RequiredArgsConstructor
 public class AuthController {
+
     private final AuthService authService;
     private final JwtProvider jwtProvider;
 

src/main/java/land/leets/domain/comment/domain/Comment.java

@@ -1,14 +1,22 @@
 package land.leets.domain.comment.domain;
 
-import jakarta.persistence.*;
+import jakarta.persistence.Column;
+import jakarta.persistence.Entity;
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.GenerationType;
+import jakarta.persistence.Id;
+import jakarta.persistence.OneToOne;
 import land.leets.domain.admin.domain.Admin;
 import land.leets.domain.shared.BaseTimeEntity;
-import lombok.*;
+import lombok.AccessLevel;
+import lombok.AllArgsConstructor;
+import lombok.Builder;
+import lombok.Getter;
+import lombok.NoArgsConstructor;
 
 @Entity(name = "comments")
 @Builder
 @Getter
-@Setter
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 @AllArgsConstructor
 public class Comment extends BaseTimeEntity {

src/main/java/land/leets/domain/comment/presentation/dto/CommentRequest.java

@@ -7,6 +7,7 @@
 @Getter
 @NoArgsConstructor
 public class CommentRequest {
+
     @NotBlank
     private Long applicationId;
 

src/main/java/land/leets/domain/comment/presentation/dto/CommentResponse.java

@@ -11,6 +11,7 @@
 @Getter
 @AllArgsConstructor
 public class CommentResponse {
+
     @NotBlank
     private String content;
 

src/main/java/land/leets/domain/comment/presentation/mapper/CommentMapper.java

@@ -11,4 +11,4 @@
 public interface CommentMapper {
     @BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.IGNORE)
     CommentResponse mappingCommentToDto(Comment comment);
-}
+}

src/main/java/land/leets/domain/comment/usecase/CreateCommentImpl.java

@@ -15,6 +15,7 @@
 @Service
 @RequiredArgsConstructor
 public class CreateCommentImpl implements CreateComment {
+
     private final CommentRepository commentRepository;
     private final AdminRepository adminRepository;
     private final CommentMapper commentMapper;

src/main/java/land/leets/domain/comment/usecase/GetCommentsImpl.java

@@ -12,6 +12,7 @@
 @Service
 @RequiredArgsConstructor
 public class GetCommentsImpl implements GetComments {
+
     private final CommentRepository commentRepository;
     private final CommentMapper commentMapper;
 

src/main/java/land/leets/domain/contributor/domain/Contributor.java

@@ -1,9 +1,21 @@
 package land.leets.domain.contributor.domain;
 
 import com.fasterxml.jackson.annotation.JsonIgnore;
-import jakarta.persistence.*;
+import jakarta.persistence.Column;
+import jakarta.persistence.Entity;
+import jakarta.persistence.FetchType;
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.GenerationType;
+import jakarta.persistence.Id;
+import jakarta.persistence.JoinColumn;
+import jakarta.persistence.ManyToOne;
 import land.leets.domain.portfolio.domain.Portfolio;
-import lombok.*;
+import lombok.AccessLevel;
+import lombok.AllArgsConstructor;
+import lombok.Builder;
+import lombok.Getter;
+import lombok.NoArgsConstructor;
+import lombok.Setter;
 
 @Entity(name = "contributors")
 @Builder
@@ -38,4 +50,3 @@ public class Contributor {
     @Column
     private String profile;
 }
-

src/main/java/land/leets/domain/contributor/domain/Position.java

@@ -6,10 +6,11 @@
 @Getter
 @AllArgsConstructor
 public enum Position {
+
     BACK_END("BackEnd"),
     FRONT_END("FrontEnd"),
     BX_BI("BX/BI"),
     UX_UI("UX/UI");
 
     private final String position;
-}
+}

src/main/java/land/leets/domain/contributor/domain/repository/ContributorRepository.java

@@ -1,9 +1,7 @@
 package land.leets.domain.contributor.domain.repository;
 
-import land.leets.domain.comment.domain.Comment;
 import land.leets.domain.contributor.domain.Contributor;
 import org.springframework.data.jpa.repository.JpaRepository;
 
-import java.util.List;
 public interface ContributorRepository extends JpaRepository<Contributor, Long> {
-}
+}

src/main/java/land/leets/domain/contributor/usecase/CreateContributorImpl.java

@@ -10,8 +10,10 @@
 
 @Service
 @RequiredArgsConstructor
-public class CreateContributorImpl implements CreateContributor{
+public class CreateContributorImpl implements CreateContributor {
+
     private final ContributorRepository contributorRepository;
+
     @Override
     public void execute(List<Contributor> request, Portfolio portfolio) {
         for (Contributor contributor : request) {