Revert "[feat] : 회원가입 api 구현 & use 엔티티 추가 구현 (#41)" (#49)

This reverts commit d438902.

core/build.gradle

@@ -7,8 +7,6 @@ dependencies {
     testFixturesImplementation 'org.springframework.boot:spring-boot-starter-test'
     testFixturesImplementation "org.testcontainers:testcontainers:1.19.3"
     testFixturesImplementation "org.testcontainers:postgresql:1.19.3"
-    // 패스워드 암호화
-    implementation group: 'org.mindrot', name: 'jbcrypt', version: '0.3m'
 }
 
 bootJar { enabled = false }

core/src/main/java/dev/hooon/user/application/UserService.java

@@ -3,47 +3,20 @@
 import static dev.hooon.user.exception.UserErrorCode.*;
 
 import org.springframework.stereotype.Service;
-import org.springframework.transaction.annotation.Transactional;
 
-import dev.hooon.auth.entity.EncryptHelper;
 import dev.hooon.common.exception.NotFoundException;
 import dev.hooon.user.domain.entity.User;
 import dev.hooon.user.domain.repository.UserRepository;
-import dev.hooon.user.dto.request.UserJoinRequest;
-import dev.hooon.common.exception.ValidationException;
 import lombok.RequiredArgsConstructor;
-import lombok.extern.slf4j.Slf4j;
 
 @Service
 @RequiredArgsConstructor
 public class UserService {
 
 	private final UserRepository userRepository;
-	private final EncryptHelper encryptHelper;
-
-	private void validateDuplicateEmail(String email) {
-		if (userRepository.findByEmail(email).isPresent()) {
-			throw new ValidationException(DUPLICATED_EMAIL);
-		}
-	}
 
 	public User getUserById(Long userId) {
 		return userRepository.findById(userId)
 			.orElseThrow(() -> new NotFoundException(NOT_FOUND_BY_ID));
 	}
-
-	@Transactional
-	public Long join(UserJoinRequest userJoinRequest) {
-		validateDuplicateEmail(userJoinRequest.email());
-		String password = userJoinRequest.password();
-		String encryptedPassword = encryptHelper.encrypt(password);
-
-		User user = User.ofBuyer(
-			userJoinRequest.email(),
-			userJoinRequest.name(),
-			encryptedPassword
-		);
-		return userRepository.save(user);
-	}
-
 }

core/src/main/java/dev/hooon/user/domain/entity/User.java

@@ -1,12 +1,8 @@
 package dev.hooon.user.domain.entity;
 
-import static dev.hooon.common.exception.CommonValidationError.*;
-import static dev.hooon.user.domain.entity.UserRole.*;
 import static jakarta.persistence.EnumType.*;
 import static jakarta.persistence.GenerationType.*;
 
-import org.springframework.util.Assert;
-
 import dev.hooon.common.entity.TimeBaseEntity;
 import jakarta.persistence.Column;
 import jakarta.persistence.Entity;
@@ -19,8 +15,8 @@
 
 @Entity
 @Getter
-@NoArgsConstructor
 @Table(name = "user_table")
+@NoArgsConstructor
 public class User extends TimeBaseEntity {
 
 	@Id
@@ -34,40 +30,14 @@ public class User extends TimeBaseEntity {
 	@Column(name = "user_name", nullable = false)
 	private String name;
 
-	@Column(name = "user_password", nullable = false)
-	private String password;
-
 	@Enumerated(STRING)
 	@Column(name = "user_role", nullable = false)
 	private UserRole userRole;
 
-	private User(
-		String email,
-		String name,
-		String password,
-		UserRole userRole
-	) {
-		validateUser(email, name, password, userRole);
+	// 테스트용 AllArgsConstructor
+	public User(String email, String name, UserRole userRole) {
 		this.email = email;
 		this.name = name;
-		this.password = password;
 		this.userRole = userRole;
 	}
-
-	private void validateUser(String email, String name, String password, UserRole userRole) {
-		Assert.hasText(email, getNotEmptyPostfix("User", "email"));
-		Assert.hasText(name, getNotEmptyPostfix("User", "name"));
-		Assert.hasText(password, getNotEmptyPostfix("User", "password"));
-		Assert.hasText(String.valueOf(userRole), getNotEmptyPostfix("User", "userRole"));
-		Assert.notNull(userRole, getNotNullMessage("User", "userRole"));
-	}
-
-	public static User ofBuyer(
-		String email,
-		String name,
-		String password
-	) {
-		return new User(email, name, password, BUYER);
-	}
-
 }

core/src/main/java/dev/hooon/user/domain/repository/UserRepository.java

@@ -1,6 +1,5 @@
 package dev.hooon.user.domain.repository;
 
-import java.util.List;
 import java.util.Optional;
 
 import dev.hooon.user.domain.entity.User;
@@ -10,9 +9,4 @@ public interface UserRepository {
 	User save(User user);
 
 	Optional<User> findById(Long id);
-
-	Optional<User> findByName(String name);
-	Long save(User user);
-
-	Optional<User> findByEmail(String email);
 }

core/src/main/java/dev/hooon/user/exception/UserErrorCode.java

@@ -8,8 +8,7 @@
 @RequiredArgsConstructor
 public enum UserErrorCode implements ErrorCode {
 
-	NOT_FOUND_BY_ID("ID 에 해당하는 사용자가 존재하지 않습니다", "U_001"),
-	DUPLICATED_EMAIL("이미 존재하는 이메일입니다. 회원가입을 다시 진행해주세요.", "U_002");
+	NOT_FOUND_BY_ID("ID 에 해당하는 사용자가 존재하지 않습니다", "U_001");
 
 	private final String message;
 	private final String code;

core/src/main/java/dev/hooon/user/infrastructure/adaptor/UserRepositoryAdaptor.java

@@ -1,6 +1,5 @@
 package dev.hooon.user.infrastructure.adaptor;
 
-import java.util.List;
 import java.util.Optional;
 
 import org.springframework.stereotype.Repository;
@@ -25,19 +24,4 @@ public User save(User user) {
 	public Optional<User> findById(Long id) {
 		return userJpaRepository.findById(id);
 	}
-
-	@Override
-	public Optional<User> findByName(String name) {
-		return userJpaRepository.findByName(name);
-	}
-
-	@Override
-	public Long save(User user) {
-		return userJpaRepository.save(user).getId();
-	}
-
-	@Override
-	public Optional<User> findByEmail(String email) {
-		return userJpaRepository.findByEmail(email);
-	}
 }

core/src/main/java/dev/hooon/user/infrastructure/repository/UserJpaRepository.java

@@ -1,13 +1,8 @@
 package dev.hooon.user.infrastructure.repository;
 
-import java.util.Optional;
-
 import org.springframework.data.jpa.repository.JpaRepository;
 
 import dev.hooon.user.domain.entity.User;
 
 public interface UserJpaRepository extends JpaRepository<User, Long> {
-	Optional<User> findByName(String name);
-
-	Optional<User> findByEmail(String email);
 }