글자수 Size 검증 (#818)

* fix(/dto): 글자수 Size 검증 * fix(request): 빠진 , 추가
woowacourse-teams · Oct 18, 2024 · 44c64d0 · 44c64d0
1 parent a1c386c
commit 44c64d0
Show file tree

Hide file tree

Showing 8 changed files with 12 additions and 9 deletions.
diff --git a/backend/src/main/java/codezap/category/dto/request/UpdateCategoryRequest.java b/backend/src/main/java/codezap/category/dto/request/UpdateCategoryRequest.java
@@ -10,7 +10,7 @@
 public record UpdateCategoryRequest(
         @Schema(description = "카테고리 이름", example = "Spring")
         @NotBlank(message = "카테고리 이름이 null 입니다.", groups = NotNullGroup.class)
-        @Size(max = 255, message = "카테고리 이름은 최대 255자까지 입력 가능합니다.", groups = SizeCheckGroup.class)
+        @Size(max = 15, message = "카테고리 이름은 최대 15자까지 입력 가능합니다.", groups = SizeCheckGroup.class)
         String name
 ) {
 }
diff --git a/backend/src/main/java/codezap/member/dto/request/SignupRequest.java b/backend/src/main/java/codezap/member/dto/request/SignupRequest.java
@@ -15,7 +15,7 @@ public record SignupRequest(
         @Schema(description = "비밀번호. 영어와 숫자를 반드시 포함해야 합니다.", example = "password1234")
         @Pattern(regexp = "^(?=.*[a-zA-Z])(?=.*\\d).*$", message = "영어와 숫자를 포함해야합니다.")
         @NotBlank(message = "비밀번호가 입력되지 않았습니다.")
-        @Size(min = 8, max = 255, message = "비밀번호는 8~16자 사이로 입력해주세요.")
+        @Size(min = 8, max = 16, message = "비밀번호는 8~16자 사이로 입력해주세요.")
         String password
 ) {
 }
diff --git a/backend/src/main/java/codezap/template/dto/request/CreateSourceCodeRequest.java b/backend/src/main/java/codezap/template/dto/request/CreateSourceCodeRequest.java
@@ -1,5 +1,6 @@
 package codezap.template.dto.request;
 
+import jakarta.validation.constraints.Min;
 import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotNull;
 import jakarta.validation.constraints.Size;
@@ -22,6 +23,7 @@ public record CreateSourceCodeRequest(
 
         @Schema(description = "소스 코드 순서", example = "1")
         @NotNull(message = "소스 코드 순서가 null 입니다.", groups = NotNullGroup.class)
-        int ordinal
+        @Min(value = 1, message = "소스 코드 순서는 1 이상이어야 합니다.")
+        Integer ordinal
 ) {
 }
diff --git a/backend/src/main/java/codezap/template/dto/request/CreateTemplateRequest.java b/backend/src/main/java/codezap/template/dto/request/CreateTemplateRequest.java
@@ -41,9 +41,8 @@ public record CreateTemplateRequest(
 
         @Schema(description = "태그 목록")
         @NotNull(message = "태그 목록이 null 입니다.", groups = NotNullGroup.class)
-        @ByteLength(max = 30, message = "태그 명은 최대 30자까지 입력 가능합니다.", groups = SizeCheckGroup.class)
         @Valid
-        List<String> tags,
+        List<@Size(max = 30, message = "태그 명은 최대 30자까지 입력 가능합니다.", groups = SizeCheckGroup.class) String> tags,
 
         @Schema(description = "템플릿 공개 여부", example = "PUBLIC")
         @NotNull(message = "템플릿 공개 여부가 null 입니다.", groups = NotNullGroup.class)

diff --git a/backend/src/main/java/codezap/template/dto/request/UpdateSourceCodeRequest.java b/backend/src/main/java/codezap/template/dto/request/UpdateSourceCodeRequest.java
@@ -1,5 +1,6 @@
 package codezap.template.dto.request;
 
+import jakarta.validation.constraints.Min;
 import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotNull;
 import jakarta.validation.constraints.Size;
@@ -26,6 +27,7 @@ public record UpdateSourceCodeRequest(
 
         @Schema(description = "소스 코드 순서", example = "1")
         @NotNull(message = "소스 코드 순서가 null 입니다.", groups = NotNullGroup.class)
-        int ordinal
+        @Min(value = 1, message = "소스 코드 순서는 1 이상이어야 합니다.")
+        Integer ordinal
 ) {
 }
diff --git a/backend/src/main/java/codezap/template/dto/request/UpdateTemplateRequest.java b/backend/src/main/java/codezap/template/dto/request/UpdateTemplateRequest.java
@@ -47,7 +47,7 @@ public record UpdateTemplateRequest(
 
         @Schema(description = "태그 목록")
         @NotNull(message = "태그 목록이 null 입니다.", groups = NotNullGroup.class)
-        List<String> tags,
+        List<@Size(max = 30, message = "태그 명은 최대 30자까지 입력 가능합니다.", groups = SizeCheckGroup.class) String> tags,
 
         @Schema(description = "템플릿 공개 여부", example = "PUBLIC")
         @NotNull(message = "템플릿 공개 여부가 null 입니다.", groups = NotNullGroup.class)

diff --git a/backend/src/test/java/codezap/category/controller/CategoryControllerTest.java b/backend/src/test/java/codezap/category/controller/CategoryControllerTest.java
@@ -167,7 +167,7 @@ void updateCategoryFailWithlongName() throws Exception {
                             .contentType(MediaType.APPLICATION_JSON)
                             .content(objectMapper.writeValueAsString(updateCategoryRequest)))
                     .andExpect(status().isBadRequest())
-                    .andExpect(jsonPath("$.detail").value("카테고리 이름은 최대 255자까지 입력 가능합니다."))
+                    .andExpect(jsonPath("$.detail").value("카테고리 이름은 최대 15자까지 입력 가능합니다."))
                     .andExpect(jsonPath("$.errorCode").value(1101));
         }
     }

diff --git a/backend/src/test/java/codezap/template/controller/TemplateControllerTest.java b/backend/src/test/java/codezap/template/controller/TemplateControllerTest.java
@@ -213,7 +213,7 @@ private static CreateTemplateRequest createRequestWithLongTag() {
                     validRequest.sourceCodes(),
                     validRequest.thumbnailOrdinal(),
                     validRequest.categoryId(),
-                    List.of("a".repeat(31)),
+                    List.of("ㄱ".repeat(31)),
                     validRequest.visibility());
         }