Skip to content
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.

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

[Fix] testcontainers로 대체 & 트랜잭션 오류 해결 #55

Merged
merged 6 commits into from
Dec 25, 2024

Conversation

DongminL
Copy link
Collaborator

@DongminL DongminL commented Dec 24, 2024

#️⃣ 연관된 이슈

#53

✅ 체크리스트

  • 🔀 PR 제목의 형식 맞추기 ex) [feat] ㅇㅇ 기능 추가
  • 💡 이슈 등록
  • 🏷️ 라벨 등록
  • 🧹 불필요한 코드 제거
  • 🧪 로컬 테스트 완료
  • 🏗️ 빌드 성공
  • 💯 테스트 통과

📝 작업 내용

H2 DB를 MariaDB Test Container로 변경
TestContainers 적용 시, 트랜잭션이 적용되지 않아 기존 테스트들이 실패하는 오류 해결

📸 스크린샷 (UI 변경 시 필수)

👀 리뷰어 가이드라인

시간은 조금 더 걸리지만, 현재 로컬에 있는 MariaDB와 완전히 동일한 환경에서 테스트할 수 있습니다.
테스트할 때는 무조건 도커가 실행되어 있어야 합니다!
이전 PR한 브랜치는 커밋이 쓸데없이 너무 많아져서 새롭게 브랜치를 만들었어요

@DongminL DongminL self-assigned this Dec 24, 2024
@DongminL DongminL changed the title [Fix] 많은 오류 해결 [Fix] testcontainers로 대체& 많은 오류 해결 Dec 24, 2024
@DongminL DongminL changed the title [Fix] testcontainers로 대체& 많은 오류 해결 [Fix] testcontainers로 대체 & 많은 오류 해결 Dec 24, 2024
@DongminL DongminL changed the title [Fix] testcontainers로 대체 & 많은 오류 해결 [Fix] testcontainers로 대체 & 트랜잭션 오류 해결 Dec 24, 2024
Copy link

Test Results

30 tests   30 ✅  5s ⏱️
11 suites   0 💤
11 files     0 ❌

Results for commit e51bbdf.

@DongminL DongminL linked an issue Dec 24, 2024 that may be closed by this pull request
@DongminL DongminL removed the Bug label Dec 25, 2024
@limseohyeon limseohyeon merged commit 0f6bd1f into main Dec 25, 2024
3 checks passed
@DongminL DongminL deleted the fix/#52-많은-오류-해결 branch December 26, 2024 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TestContainers로 대체
2 participants