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

Feature/19: MemberRepository에 Spring Data Jpa 도입 #20

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

KanuBang
Copy link
Collaborator

@KanuBang KanuBang commented Jan 14, 2025

  • MemberRepository에 Spring Data Jpa 도입
  • Member 엔티티 관련 Service, Controller, Repository 테스트
  • Member, MemberResponse에 Builder 패턴 제거
  • DTO들 네이밍 변경 -> xxxRequest, xxxResponse
  • Feature/15: 회원 관리 기능 DB와 연동 #17 리뷰 내용 반영

- MemberRepository에 Spring Data Jpa 도입
- Member 엔티티 관련 Service, Controller, Repository 테스트
- Member, MemberResponse에 Builder 패턴 제거
- DTO들 네이밍 변경 -> xxxRequest, xxxResponse
@KanuBang KanuBang changed the title Refactor: MemberRepository에 Spring Data Jpa 도입 Feature/19: MemberRepository에 Spring Data Jpa 도입 Jan 14, 2025
KanuBang and others added 5 commits January 14, 2025 18:10
기존 세션 기반 인증을 JWT 기반 인증으로 바꾸었습니다.
Feature/18: JWT 기반 인증 구현
@KanuBang KanuBang closed this Jan 22, 2025
@KanuBang KanuBang reopened this Jan 22, 2025
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@Autowired
EntityManager em;
// 테스타가 실했다가 성공했다가 반복 -> 테스트에 사용하는 데이터가 같아서 중복된 데이터가 삽입될 가능성이 있음.
// 각 테스트가 독립적으로 실행되도록 해야 함 -> 테스트 데이터를 다르게 구성?
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

org.springframework.orm.objectoptimisticlockingfailureexception: row was updated or deleted by another transaction

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant