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

feat: 에픽 생성, 수정, 삭제 API 구현 #303

Merged
merged 5 commits into from
Jul 3, 2024
Merged

Conversation

choyoungwoo9
Copy link
Collaborator

🎟️ 태스크

에픽 생성, 수정, 삭제 API 구현(백엔드)

✅ 작업 내용

  • 테스트 ws-landing-page 디렉토리 추가
  • 에픽 생성, 삭제, 업데이트 엔티티, 레포지토리, 서비스 작성
  • 에픽 생성, 삭제, 업데이트 게이트웨이, 컨트롤러, DTO작성
  • 에픽 생성, 삭제, 업데이트 E2E 테스트 작성
  • E2E테스트에서 외래키 제약조건으로 인한 테이블 삭제 실패를 방지하는 로직 추가

🖊️ 구체적인 작업

테스트 ws-landing-page 디렉토리 추가

  • 웹소켓 테스트가 많아져 페이지 별로 테스트 분리

에픽 생성, 삭제, 업데이트 엔티티, 레포지토리, 서비스 작성

  • 에픽 엔티티 작성
  • 에픽을 모듈에 추가
  • 에픽 create, delete, update 레포지토리, 서비스 메서드 작성

에픽 생성, 삭제, 업데이트 게이트웨이, 컨트롤러, DTO작성

  • 에픽 이벤트를 처리하는 게이트웨이 로직 추가
  • 에픽 생성, 삭제, 업데이트를 처리하는 컨트롤러 추가
  • 에픽 생성, 삭제, 업데이트에 대해 요청/응답을 확인/생성하는 request, notify DTO 추가

에픽 생성, 삭제, 업데이트 E2E 테스트 작성

E2E테스트에서 외래키 제약조건으로 인한 테이블 삭제 실패를 방지하는 로직 추가

  • 테이블을 삭제할때 해당 테이블의 행을 참조하는 다른 테이블의 행이 있어도 테이블 삭제가 가능하도록 FOREIGN_KEY_CHECKS 변수를 0으로 설정
  • 테이블 삭제 후 모두 FOREIGN_KEY_CHECKS변수를 1로 설정

🤔 고민 및 의논할 거리

  • E2E 테스트 시 외래키 제약조건으로 테이블 삭제가 실패하지 않도록 FOREIGN_KEY_CHECKS를 설정해 외래키 제약조건을 해제해주는 로직을 추가했습니다~

- 에픽 엔티티 작성
- 에픽을 모듈에 추가
- 에픽 create, delete, update 레포지토리, 서비스 메서드 작성
@choyoungwoo9 choyoungwoo9 requested a review from kimsj-git July 3, 2024 01:53
- 에픽 이벤트를 처리하는 게이트웨이 로직 추가
- 에픽 생성, 삭제, 업데이트를 처리하는 컨트롤러 추가
- 에픽 생성, 삭제, 업데이트에 대해 요청/응답을 확인/생성하는 request, notify DTO 추가
- 테이블을 삭제할때 해당 테이블의 행을 참조하는 다른 테이블의 행이 있어도 테이블 삭제가 가능하도록 FOREIGN_KEY_CHECKS 변수를 0으로 설정
- 테이블 삭제 후 모두 FOREIGN_KEY_CHECKS변수를 1로 설정
@choyoungwoo9 choyoungwoo9 merged commit ba92c9e into dev Jul 3, 2024
1 check passed
@choyoungwoo9 choyoungwoo9 deleted the feature/epic-crud branch July 3, 2024 02:02
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