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

사용하지 않는 초기데이터 세팅 제거 #714

Closed
wants to merge 37 commits into from

Conversation

kwonyj1022
Copy link
Collaborator

📄 작업 내용 요약

사용하지 않는 초기데이터 세팅 제거

🙋🏻 리뷰 시 주의 깊게 확인해야 하는 코드

  • 별 거 없습니다. 저번에 한다고 해놓고 빼먹었네요

📎 Issue 번호

kwonyj1022 and others added 30 commits October 11, 2023 06:55
* feat: 경매 상세 조회 시 경매글을 찾을 수 없을 때만 404를 보내도록 수정

* refactor: 빈 유저와 빈 경매 상수 추가 및 적용

* refactor: 지정한 경매 아이디와 관련된 채팅방을 조회할 때 경매를 찾을 수 없을 때 예외를 던지도록 다시 변경

* test: 테스트 메서드 이름 명확하게 변경
* refactor: 경매 목록 조회 시 마감된 경매의 정렬 순서를 후순위로 변경

* test: 정렬 조건 변경으로 인한 테스트 케이스 변경

* refactor: 복잡하지 않은 쿼리를 querydsl로 표현하지 않고 JPQL로 표현

* refactor: left outer join이 필요하지 않은 경우를 inner join으로 변경

* refactor: 모든 관계를 fetch join하는 Auction 조회 메서드 네이밍 변경

* feat: Auction만을 조회하는 메서드 추가

* refactor: Auction의 정보만이 필요한 비즈니스 로직에서 fetch join을 하지 않도록 변경

* chore: 배포 스크립트 의미 없는 break 키워드 삭제
* refactor: 마감 시간을 미래만 적용할 수 있도록 제약조건 변경

* feat: 마감 시간 Custom Validator 추가

* refactor: 경매 등록 요청 시 마감 시간 검증을 Custom Validator으로 하도록 변경
* refactor: 소셜 로그인 시 사용자가 최초 로그인을 하는지 여부를 의미하는 필드 추가

* docs: 문서 최신화

* refactor: 신규 회원가입을 했는지에 대한 네이밍을 명확하게 변경

* test: 실패하는 테스트 케이스 수정

* style: 개행 및 누락된 final 키워드 추가
* feat: 신뢰도 점수 타입을 Double -> double로 변경

* feat: 평가 저장 시 신뢰도를 갱신하지 않고 저장만 하도록 변경

* feat: 신뢰도 업데이트에 필요한 비즈니스 로직 추가

* feat: 신뢰도 업데이트 서비스 구현에 필요한 비즈니스 로직 보완 및 레포지토리 추가

* feat: 신뢰도 업데이트 서비스 추가

* feat: 스케줄링 설정 추가

* feat: flyway 스크립트 추가

* style: 개행 수정

* feat: 신뢰도 업데이트 이력을 관리하는 엔티티의 toString 대상 항목 추가

* refactor: 평가 관련 일급컬렉션에서 EqualsAndHashCode 재정의 제거

* test: 테스트코드에서 선언과 assertion 분리

* comment: 주석 제거
* chore: flyway 버전 17 추가

* feat: User 도메인에 사용한 소셜 로그인 방식 필드 추가

* refactor: 회원 탈퇴 시 소셜 로그인 방식을 DB에서 조회하도록 변경

* refactor: Oauth2와 관련된 필드를 별도의 클래스로 분리

* chore: flyway 스크립트 버전 변경

* test: 의미 없는 테스트 mocking 삭제

* refactor: access token 재발급 시 refresh token도 재발급되도록 변경
* feat: 답변 서비스에서 알림 이벤트 발행

* feat: 질문 서비스에서 알림 이벤트 발행

* feat: 질문, 답변 이벤트 리스너 추가

* chore: 수정 필요한 부분 주석 추가

* refactor: 이미지 절대 경로 컨트롤러에서 전송하도록 수정

* refactor: 이벤트 발행 시 별도의 DTO가 아닌 도메인 전달하도록 수정

* fix: 잘못 복제한 커밋 내역 복구

* refactor: 레포지토리 저장 엔티티를 이벤트 dto에 전달하도록 수정
* refactor: Querydsl 기반의 Auction 인터페이스 제거 및 구현체에서 일반 클래스로 변환

* refactor: Querydsl 기반의 Auction 조회 인터페이스를 domain 패키지로 이동

* refactor: 경매 목록 조회 메서드 파라미터 순서 변경

* feat: 도메인 영역의 Repository 추가

* feat: 도메인 영역의 Repository 구현체 추가

* test: 누락된 테스트 케이스 추가

* feat: 경매 id가 존재하는지 여부를 반환하는 기능 추가

* refactor: Service에서 도메인 영역의 Repository를 사용하도록 변경

* refactor: Service에서 도메인 영역의 Repository를 사용하도록 변경

* rename: 경매 목록 조회 관련 테스트 패키지 분리

* remove: 사용하지 않는 경매 더미 데이터 초기화 로직 삭제

* test: fixture에서 구체적인 DB 구현 기술에 의존적이지 않도록 변경
* feat: 도메인 영역의 Repository 추가

* feat: 도메인 영역의 Repository 구현체 추가

* refactor: Service에서 도메인 영역의 Repository를 사용하도록 변경

* refactor: Service에서 도메인 영역의 Repository를 사용하도록 변경
* test: 누락된 테스트 케이스 추가

* feat: 도메인 영역의 Repository 추가

* feat: 도메인 영역의 Repository 구현체 추가

* refactor: Service에서 도메인 영역의 Repository를 사용하도록 변경

* test: when & then 주석 추가

* test: BlackListToken의 도메인 인터페이스 Fixture에서 구체적인 인프라 구현 방식을 알지 못하도록 변경

* test: fixture 메서드 네이밍 변경
* feat: 도메인 영역의 사용자 Repository 추가

* feat: 도메인 영역의 사용자 Repository 구현체 추가

* test: 누락된 테스트 케이스 추가

* refactor: Service에서 도메인 영역의 사용자 Repository를 사용하도록 변경

* refactor: id로 사용자 조회 시 삭제된 값은 조회하지 않도록 수정

* feat: 도메인 영역의 사용자 신뢰도 Repository 추가

* feat: 도메인 영역의 사용자 신뢰도 Repository 구현체 추가

* test: 누락된 테스트 케이스 추가

* refactor: Service에서 도메인 영역의 사용자 신뢰도 Repository를 사용하도록 변경

* refactor: 사용자 신뢰도 조회 시 사용자도 함께 조회되도록 수정

* fix: 사용자 레포지토리 구현체에서 잘못된 메서드를 호출해서 테스트를 실패하는 현상 해결

* test: 테스트 실패 해결

* refactor: 사용자 신뢰도 엔티티에서 cascade 옵션 수정

* feat: 도메인 영역의 신뢰도 업데이트 기록 Repository 추가

* feat: 도메인 영역의 신뢰도 업데이트 기록 Repository 구현체 추가

* test: 누락된 테스트 케이스 추가

* refactor: Service에서 도메인 영역의 신뢰도 업데이트 기록 Repository를 사용하도록 변경

* refactor: 픽스처에서 Jpa 사용자 Repository 의존 제거

* refactor: 픽스처에서 Jpa 사용자 신뢰도 Repository 의존 제거

* refactor: 픽스처에서 Jpa 신뢰도 업데이트 기록 Repository 의존 제거

* style: 개행 추가

* refactor: 누락된 final 추가

* test: jpa 저장 테스트 시 불필요한 엔티티 매니저 호출 제거

* refactor: 누락된 final 추가

* refactor: RepositoryImpl 테스트에서 Jpa 의존 제거
* feat: 도메인 영역의 채팅방 Repository 추가

* refactor: 불필요하게 Querydsl을 사용한 코드를 JPQL을 사용하도록 변경

* feat: 도메인 영역의 채팅방 Repository 구현체 추가

* refactor: Service에서 도메인 영역의 채팅방 Repository를 사용하도록 변경

* refactor: 픽스처에서 Jpa 채팅방 Repository 의존 제거

* feat: 도메인 영역의 채팅방 정보(채팅방, 이미지) dto를 반환하는 Repository 추가

* feat: 도메인 영역의 채팅방 정보(채팅방, 이미지) dto를 반환하는 Repository 구현체 추가

* refactor: Service에서 도메인 영역의 채팅방 정보(채팅방, 이미지) dto를 반환하는 Repository를 사용하도록 변경

* feat: 도메인 영역의 채팅방 정보(채팅방, 메시지, 이미지) dto를 반환하는 Repository 추가

* feat: 도메인 영역의 채팅방 정보(채팅방, 메시지, 이미지) dto를 반환하는 Repository 구현체 추가

* feat: Service에서 도메인 영역의 채팅방 정보(채팅방, 메시지, 이미지) dto를 반환하는 Repository를 사용하도록 변경

* style: 개행 추가

* refactor: 누락된 final 추가

* ci: 브랜치 최신화 과정에서 누락된 사항 추가 및 충돌 해결
* feat: 도메인 영역의 지역 Repository 추가

* feat: 도메인 영역의 지역 Repository 구현체 추가

* test: 누락된 테스트 케이스 추가

* refactor: Service에서 도메인 영역의 지역 Repository를 사용하도록 변경

* test: 테스트에서 Jpa 의존 제거

* test: 테스트에서 Jpa 의존 제거

* style: 공백 추가

* refactor: 누락된 final 추가
* feat: 도메인 영역의 평가 Repository 추가

* feat: 도메인 영역의 평가 Repository 구현체 추가

* refactor: Service에서 도메인 영역의 평가 Repository를 사용하도록 변경

* test: 픽스처에서 Jpa 의존 제거

* test: 테스트 실패 케이스 해결

* refactor: 누락된 final 추가

* ci: 브랜치 최신화 과정에서 누락된 사항 추가 및 충돌 해결
* refactor: 입찰 목록 조회 시 배열이 그대로 반환되도록 수정

* docs: 문서화 최신화

* refactor: 체이닝 개행 수정
* refactor: 컨벤션에 따른 개행 수정

* feat: 질문과 답변 조회 시 질문 작성자인지 판단하는 값 추가

* refactor: 컨벤션에 따른 개행 수정

* refactor: 메서드 순서 변경

* fix: 충돌로 발생한 문제 해결

* refactor: import 와일드카드 제거
* fix: 신규 회원 가입 시 null이 될 수 없는 필드를 초기화하지 않아 레코드 저장에 실패하던 문제 수정

* test: 테스트 케이스 검증 부분 추가 및 잘못된 테스트 케이스 수정

* test: Fixture 네이밍과 테스트 메서드 네이밍을 명확하게 변경

* refactor: 초기 신뢰도를 상수로 초기화

* refactor: 초기 신뢰도 상수 변경
* refactor: 변수명을 명확하게 수정

* feat: 도메인 영역에 경매 이미지 repository 추가

* feat: 도메인 영역에 경매 이미지 repository의 구현체 추가

* refactor: service에서 도메인 영역의 repository를 사용하도록 변경

* feat: 도메인 영역에 프로필 이미지 repository 추가

* feat: 도메인 영역에 프로필 이미지 repository의 구현체 추가

* refactor: service에서 도메인 영역의 프로필 이미지 repository를 사용하도록 변경

* refactor: 변수명을 의도에 맞게 수정

* refactor: 레파지토리 어노테이션 누락 추가

* test: final 추가

* test: final 추가

* refactor: import 와일드카드 제거

* refactor: 쿼리문 명령어를 대문자로 수정

* refactor: 세팅 메서드명 수정

* refactor: 잘못된 클래스명 수정
* feat: 도메인 영역에 입찰 repository 추가

* refactor: 메서드 네이밍과 파라미터 변수명을 명확하게 수정

* feat: 도메인 영역에 입찰 repository의 구현체 추가

* refactor: service에서 도메인 영역의 입찰 repository를 사용하도록 변경

* refactor: 레파지토리 어노테이션 누락 추가

* test: 테스트 영역에서 도메인 영역의 입찰 repository를 사용하도록 변경

* refactor: 쿼리문 명령어를 대문자로 수정

* refactor: 레파지토리에서 null이 아닌 Optional을 통해 반환되도록 수정

* test: 도메인 영역에 입찰 repository의 구현체 테스트 추가

* test: 테스트 영역에서 도메인 영역의 repository를 사용하도록 변경
* feat: 도메인 영역에 경매 신고 repository 추가

* refactor: EntityGraph를 jpql로 변경

* refactor: EntityGraph를 jpql로 변경

* feat: 도메인 영역에 경매 신고 repository 구현체 추가

* refactor: service에서 도메인 영역의 경매 신고 repository를 사용하도록 변경

* test: 테스트에서 도메인 영역의 경매 신고 repository를 사용하도록 변경

* feat: 도메인 영역에 채팅방 신고 repository 추가

* refactor: EntityGraph를 jpql로 변경

* feat: 도메인 영역에 쪽지방 신고 repository 구현체 추가

* test: 테스트 실패 해결

* test: 픽스처 클래스 네이밍 수정

* feat: 도메인 영역에 질문 신고 repository 추가

* refactor: EntityGraph를 jpql로 변경

* test: 테스트에서 도메인 영역의 질문 신고 repository를 사용하도록 변경

* refactor: service에서 도메인 영역의 쪽지방 신고 repository를 사용하도록 변경

* refactor: service에서 도메인 영역의 질문 신고 repository를 사용하도록 변경

* refactor: import 와일드카드 제거

* refactor: EntityGraph를 jpql로 변경

* feat: 도메인 영역에 답변 신고 repository 추가

* feat: 도메인 영역에 답변 신고 repository 구현체 추가

* refactor: service에서 도메인 영역의 답변 신고 repository를 사용하도록 변경

* test: 테스트에서 도메인 영역의 신고관련 repository를 사용하도록 변경

* refactor: import 와일드카드 제거

* test: 테스트 실패 문제 해결

* test: final 추가

* refactor: 쿼리문 명령어를 대문자로 수정

* refactor: 클래스명 수정

* refactor: 필드 순서 변경 및 불필효한 필드 제거

* test: 테스트 영역에서 도메인 영역의 repository를 사용하도록 변경
* feat: 도메인 영역에 질문 repository 추가

* refactor: EntityGraph를 jpql로 변경

* refactor: 사용하지 않는 클래스 제거

* refactor: 잘못된 쿼리문 수정

* feat: 도메인 영역에 질문 repository 구현체 추가

* refactor: service에서 도메인 영역의 질문 repository를 사용하도록 변경

* test: 테스트에서 도메인 영역의 질문 repository를 사용하도록 변경

* refactor: 사용하지 않는 메서드 제거

* refactor: 쿼리문 명령는 대문자로 수정

* feat: 도메인 영역에 답변 repository 추가

* feat: 도메인 영역에 답변 repository 구현체 추가

* test: 테스트에서 도메인 영역의 답변 repository를 사용하도록 변경

* test: 테스트 실패 문제 해결

* fix: 잘못된 쿼리 조건 수정

* test: 불필요한 로직 제거

* refactor: final 누락 추가
* feat: 이름 수정시 이미 존재하는 이름에 대한 예외처리 추가

* feat: 예외 처리 추가
* feat: 도메인 영역에 메시지 레포지토리 추가

* refactor: 메서드명에서 레포지토리명과 중복되는 부분 제거

* fix: 불필요한 메서드 제거

* refactor: 도메인 메시지 레포지토의 구현체 추가

* refactor: 서비스에서 도메인 레포지토리 사용하도록 변경

* refactor: 클래스명 변경에 따른 테스트 네이밍 변경

* test: 도메인 메시지 레포지토리 구현체 테스트 추가

* refactor: 변수명 명확하게 변경

* refactor: 누락된 then절 추가

* test: 테스트에서 도메인 영역의 메시지 레포지토리 사용하도록 수정

* refactor: 테스트 픽스처에서 도메인 영역의 메시지 레포지토리를 사용하도록 변경

* refactor: 불필요한 주석 삭제

* refactor: 누락된 SuppressWarnings 어노테이션 추가

* refactor: 불필요한 import문 삭제 및 사용되지 않는 필드 지역변수로 변경

* style: 불필요한 개행 제거
#695)

* refactor: 경매 상세 조회 api 요청 시 로그인한 사용자가 해당 경매의 최종 입찰자인지 여부를 반환하도록 변경

* docs: 문서 최신화

* style: 개행 변경
* feat: 도메인 영역 기기토큰 레포지토리 추가

* feat: 도메인 기기토큰 레포지토리의 구현체 추가

* feat: 기기토큰 레포지토리 메서드 중 누락된 메서드 추가

* refactor: 서비스 레이어에서 도메인 영역 기기토큰 레포지토리 사용하도록 변경

* refactor: 누락된 final 추가

* test: 테스트에서 도메인 영역의 기기토큰 레포지토리 사용하도록 변경

* refactor: 레포지토리명에 맞춰 변수명 변경

* refactor: 테스트 픽스처에서 도메인 영역의 사용자 레포지토리 사용하도록 변경

* test: 기기토큰 레포지토리 구현체 테스트 추가

* chore: 질문에 대한 todo 주석 추가

* style: 개행 추가

* refactor: jpaQueryFactory 파라미터명 통일

* refactor: 필드 순서 변경

* refactor: given절 추가

* refactor: jpa 레포지토리 저장 테스트 메서드에서 엔티티 매니저 사용하는 코드 삭제

* refactor: 불필요한 필드 삭제

* refactor: 테스트에서 도메인 레포지토리 사용하도록 변경

* refactor: 불필요한 todo 삭제
JJ503 and others added 7 commits October 15, 2023 16:57
* test: 질문이 삭제된 경우 조회되지 않음을 나타내는 테스트 추가

* refactor: 답변이 삭제된 경우 null이 반환되도록 수정
* feat: 탈퇴 시 닉네임을 랜덤 UUID로 변경하는 로직 추가

* feat: 탈퇴 시 프로필 이미지 삭제 기능 추가

* refactor: 가입 시 이미지를 기본 이미지가 아닌 null로 전달

* feat: 프로필 이미지 아이디가 null이라면 기본 이미지 url을 반환 기능 추가

* refactor: import 와일드카드 제거

* feat: 이미지가 null인 경우 기본 이미지를 반환하는 기능 추가

* test: 픽스처 네이밍을 명확하게 수정

* fix: 충돌 문제 해결
* feat: 답변에 작성자 필드 추가

* feat: 필드 추가에 대한 flyway 스크립트 추가

* feat: 답변 삭제 시 질문과의 연관관계 끊기 기능 추가

* refactor: 문자열 상수 처리

* test: import 와일드카드 제거

* fix: 문제가 발생할 flyway 스크립트 수정
* refactor: 경매 상세 조회 시 탈퇴한 사용자라면 알 수 없음으로 출력되도록 수정

* refactor: ToString과 EqualsAndHashCode 설정 수정

* feat: 사용자가 등록한 경매, 마지막인 경매 중 현재 진행 중인 경매가 있는지 확인하는 기능 레파지토리에 추가

* feat: 사용자가 등록한 경매, 마지막인 경매 중 현재 진행 중인 경매가 있는지 확인하는 기능 서비스에 추가

* refactor: jpql에서 querydsl로 변경

* test: 컨트롤러 테스트 추가

* test: import 와일드카드 제거

* refactor: querydsl을 jpa 레파지토리로 이동
@kwonyj1022 kwonyj1022 added refactor 기존 기능에 변경이 없는 구현 변경 시 backend 백엔드와 관련된 이슈나 PR에 사용 labels Oct 18, 2023
@kwonyj1022 kwonyj1022 added this to the 최종 데모데이 milestone Oct 18, 2023
@kwonyj1022 kwonyj1022 requested review from apptie, JJ503 and swonny October 18, 2023 14:49
@kwonyj1022 kwonyj1022 self-assigned this Oct 18, 2023
Copy link
Collaborator

@apptie apptie left a comment

Choose a reason for hiding this comment

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

main은 소중합니다

@kwonyj1022 kwonyj1022 closed this Oct 18, 2023
@kwonyj1022 kwonyj1022 deleted the refactor/713 branch November 3, 2023 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend 백엔드와 관련된 이슈나 PR에 사용 refactor 기존 기능에 변경이 없는 구현 변경 시
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

사용하지 않는 초기데이터 세팅 제거
4 participants