Skip to content

Bug/#66 #68

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Hgyeol merged 4 commits into from
Sep 4, 2025
Merged

Bug/#66 #68

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
7506df6
bug(#66): test 코드 UUID 오류 수정
Sep 3, 2025
6d30617
Merge branch 'bug/#66' of https://github.com/EscapeFrame/clueBackend …
Sep 3, 2025
cefb43a
bug(#66): dep 값 변경
Sep 4, 2025
c326ca5
bug(#66): dep 값 변경
Sep 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/main/resources/application-prod.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
spring:
jpa:
hibernate:
ddl-auto: update
ddl-auto: none
Copy link

@coderabbitai coderabbitai bot Sep 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Verification agent

🧩 Analysis chain

prod 환경에서 hbm2ddl 비활성화는 바람직. 다만 마이그레이션 경로(예: Flyway/Liquibase) 확보 필요.

  • ddl-auto: none은 운영에서 자동 스키마 변경을 막아 안전합니다. 단, 스키마 드리프트를 막으려면 Flyway/Liquibase 기반 배포 마이그레이션이 필수입니다.
  • 운영 실수 방지를 위해 none 대신 validate로 두면 스키마 불일치 시 애플리케이션이 즉시 실패하여 문제를 조기에 드러낼 수 있습니다. 선택지로 검토해주세요.

다음 스크립트로 마이그레이션 도구 사용 여부와 다른 프로필의 ddl-auto 설정을 점검해 주세요.

원하시면 validate로 바꾸는 간단 패치입니다:

-      ddl-auto: none
+      ddl-auto: validate

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== Flyway/Liquibase 의존성 및 설정 탐지 =="
rg -n -C2 -g '!**/build/**' -g '!**/target/**' -e 'org\.flywaydb|flyway|liquibase' build.gradle* pom.xml || true
rg -n -C2 -g '!**/build/**' -g '!**/target/**' -e 'spring\.flyway|spring\.liquibase' -S src/main/resources || true

echo
echo "== 프로필별 ddl-auto 설정 확인 =="
rg -n -C1 -S 'ddl-auto' src/main/resources || true

Length of output: 980

prod 프로파일에서 마이그레이션 도구 추가 및 ddl-auto 설정 검토

  • application-prod.yml(src/main/resources)에서 ddl-auto: none은 운영에서 자동 스키마 변경을 막아 적절하지만, build.gradle 또는 pom.xml에 Flyway/Liquibase 의존성이 전혀 없습니다. 운영 스키마 관리를 위해 마이그레이션 도구를 도입해 주세요.
  • 참고: 다른 프로파일 설정
    • application-dev.yml: ddl-auto: validate
    • application-dep.yml: ddl-auto: none
  • 운영 중 스키마 불일치 탐지를 위해 none 대신 validate 설정 검토 가능(마이그레이션 적용 보장 필요).
-      ddl-auto: none
+      ddl-auto: validate
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
ddl-auto: none
ddl-auto: validate


config:
activate:
Expand Down
9 changes: 6 additions & 3 deletions src/test/java/hello/cluebackend/domain/classroom/presentation/ClassRoomControllerTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,22 +35,25 @@ class ClassRoomControllerTest {
void testGetAllClassRooms() {
// given
String token = "fake-token";
UUID userId = 1L;
UUID userId = UUID.randomUUID();

UUID classRoomId1 = UUID.randomUUID();
UUID classRoomId2 = UUID.randomUUID();

HttpServletRequest mockRequest = mock(HttpServletRequest.class);
when(jwtUtil.getToken(mockRequest)).thenReturn(token);
when(jwtUtil.getUserId(token)).thenReturn(userId);

List<ClassRoomCardDto> mockList = List.of(
ClassRoomCardDto.builder()
.classRoomId(1L)
.classRoomId(classRoomId1)
.name("자바를 자바라")
.sort("JAVA")
.target("2-2")
.studentCount(2)
.build(),
ClassRoomCardDto.builder()
.classRoomId(2L)
.classRoomId(classRoomId2)
.name("자바를 자바라")
.sort("JAVA")
.target("2-1")
Expand Down
Loading