Merge pull request #32 from classic-daramg/feature/28-post-update

Feature/28 post update

Author: hyobin-yang

build.gradle

@@ -79,7 +79,7 @@ test {
 
 		afterTest { desc, result ->
 			if (result.resultType == TestResult.ResultType.FAILURE) {
-				println "\n ❌===== FAILED: ${desc.getDisplayName}"
+				println "\n ❌===== FAILED: ${desc.displayName}"
 
 				def exception = result.exception
 				if (exception?.getClass()?.name == 'org.opentest4j.AssertionFailedError') {
@@ -109,10 +109,13 @@ openapi3{
 	description = '클래식 듣는 다람쥐의 API 문서입니다.'
 	version = '0.1.0'
 	format = 'yaml'
-	outputDirectory = file('build/api-spec')
+	outputDirectory = file('src/main/resources/static/swagger-ui')
 }
 
 tasks.named('bootJar') {
+	duplicatesStrategy = DuplicatesStrategy.INCLUDE
+	dependsOn tasks.named('openapi3')
+
 	layered {}
 	from(tasks.named('openapi3').map { it.outputDirectory }) {
 		into 'static/swagger-ui'

src/main/java/com/daramg/server/common/application/EntityUtils.java

@@ -1,6 +1,5 @@
 package com.daramg.server.common.application;
 
-import com.daramg.server.common.exception.CommonErrorStatus;
 import com.daramg.server.common.exception.NotFoundException;
 import jakarta.persistence.EntityManager;
 import lombok.RequiredArgsConstructor;
@@ -16,6 +15,8 @@ public class EntityUtils {
 
     public <T> T getEntity(Long id, Class<T> entityType) {
         return Optional.ofNullable(entityManager.find(entityType, id))
-                .orElseThrow(() -> new NotFoundException(CommonErrorStatus.NOT_FOUND));
+                .orElseThrow(() -> new NotFoundException(
+                        String.format("존재하지 않는 %s입니다. (target ID: %d)", entityType.getSimpleName(), id)
+                ));
     }
 }

src/main/java/com/daramg/server/common/exception/BusinessException.java

@@ -9,10 +9,22 @@ public class BusinessException extends RuntimeException{
     private final BaseErrorCode errorCode;
 
     public BusinessException() {
-        this(defaultErrorCode);
+        super(defaultErrorCode.getMessage());
+        this.errorCode = defaultErrorCode;
     }
 
     public BusinessException(BaseErrorCode errorCode) {
+        super(errorCode.getMessage());
+        this.errorCode = errorCode;
+    }
+
+    public BusinessException(String message) {
+        super(message);
+        this.errorCode = defaultErrorCode;
+    }
+
+    public BusinessException(String message, BaseErrorCode errorCode) {
+        super(message);
         this.errorCode = errorCode;
     }
 }

src/main/java/com/daramg/server/common/exception/NotFoundException.java

@@ -1,7 +1,18 @@
 package com.daramg.server.common.exception;
 
 public class NotFoundException extends BusinessException {
+
+    private final static BaseErrorCode defaultErrorCode = CommonErrorStatus.NOT_FOUND;
+
+    public NotFoundException() {
+        super(defaultErrorCode);
+    }
+
     public NotFoundException(BaseErrorCode errorCode) {
         super(errorCode);
     }
+
+    public NotFoundException(String message) {
+        super(message, defaultErrorCode);
+    }
 }

…amg/server/domain/composer/Composer.java …ver/domain/composer/domain/Composer.javasrc/main/java/com/daramg/server/domain/composer/Composer.java renamed to src/main/java/com/daramg/server/domain/composer/domain/Composer.java src/main/java/com/daramg/server/domain/composer/Composer.java renamed to src/main/java/com/daramg/server/domain/composer/domain/Composer.java

@@ -1,4 +1,4 @@
-package com.daramg.server.domain.composer;
+package com.daramg.server.domain.composer.domain;
 
 import com.daramg.server.common.domain.BaseEntity;
 import jakarta.persistence.*;
@@ -8,9 +8,6 @@
 import lombok.NoArgsConstructor;
 import org.springframework.lang.NonNull;
 
-import java.util.ArrayList;
-import java.util.List;
-
 @Entity
 @Getter
 @Table(name = "composers")
@@ -48,7 +45,7 @@ public class Composer extends BaseEntity<Composer> {
     private Continent continent;
 
     @Builder
-    public Composer(@NonNull String koreanName, @NonNull String englishName, 
+    private Composer(@NonNull String koreanName, @NonNull String englishName,
                    String nativeName, @NonNull Gender gender, String nationality,
                    Short birthYear, Short deathYear, Era era, Continent continent) {
         this.koreanName = koreanName;

…server/domain/composer/ComposerPost.java …domain/composer/domain/ComposerPost.javasrc/main/java/com/daramg/server/domain/composer/ComposerPost.java renamed to src/main/java/com/daramg/server/domain/composer/domain/ComposerPost.java src/main/java/com/daramg/server/domain/composer/ComposerPost.java renamed to src/main/java/com/daramg/server/domain/composer/domain/ComposerPost.java

@@ -1,4 +1,4 @@
-package com.daramg.server.domain.composer;
+package com.daramg.server.domain.composer.domain;
 
 import com.daramg.server.common.domain.BaseEntity;
 import com.daramg.server.domain.post.domain.Post;

…mg/server/domain/composer/Continent.java …er/domain/composer/domain/Continent.javasrc/main/java/com/daramg/server/domain/composer/Continent.java renamed to src/main/java/com/daramg/server/domain/composer/domain/Continent.java src/main/java/com/daramg/server/domain/composer/Continent.java renamed to src/main/java/com/daramg/server/domain/composer/domain/Continent.java

@@ -1,4 +1,4 @@
-package com.daramg.server.domain.composer;
+package com.daramg.server.domain.composer.domain;
 
 public enum Continent {
     ASIA,

…m/daramg/server/domain/composer/Era.java …g/server/domain/composer/domain/Era.javasrc/main/java/com/daramg/server/domain/composer/Era.java renamed to src/main/java/com/daramg/server/domain/composer/domain/Era.java src/main/java/com/daramg/server/domain/composer/Era.java renamed to src/main/java/com/daramg/server/domain/composer/domain/Era.java

@@ -1,4 +1,4 @@
-package com.daramg.server.domain.composer;
+package com.daramg.server.domain.composer.domain;
 
 public enum Era {
     MEDIEVAL_RENAISSANCE,

…aramg/server/domain/composer/Gender.java …erver/domain/composer/domain/Gender.javasrc/main/java/com/daramg/server/domain/composer/Gender.java renamed to src/main/java/com/daramg/server/domain/composer/domain/Gender.java src/main/java/com/daramg/server/domain/composer/Gender.java renamed to src/main/java/com/daramg/server/domain/composer/domain/Gender.java

@@ -1,4 +1,4 @@
-package com.daramg.server.domain.composer;
+package com.daramg.server.domain.composer.domain;
 
 public enum Gender {
     MALE,

src/main/java/com/daramg/server/domain/composer/repository/ComposerRepository.java

@@ -0,0 +1,9 @@
+package com.daramg.server.domain.composer.repository;
+
+import com.daramg.server.domain.composer.domain.Composer;
+import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.stereotype.Repository;
+
+@Repository
+public interface ComposerRepository extends JpaRepository<Composer, Long> {
+}