부산대_3조_번개볼링_7주차

[sososo0]

Summary

안녕하세요! 손민철 멘토님 부산대 3조 번개볼링 BE입니다. 7주차 코드리뷰 잘 부탁드립니다~ 😄

7주차 작업내역입니다.

• Custom Exception 수정
• 회원가입/로그인 등 인증 관련 api들에 대한 통합 테스트 코드 작성
• 기존 API 버그 사항 수정
• intellij를 통한 저장 시 자동 포맷팅 설정 및 기존 코드들에 대한 일괄 reformat 적용

Description

7주차 작업내역 상세입니다.

Custom Exception 수정

• Exception들의 정보는 ErrorCode Enum class에서 관리하게 됩니다.
• 에러 응답에서 response에 ErrorCode의 이름도 추가되어 전달됩니다.

  {
  "status": 404,
  "response": "POST_NOT_FOUND",
  "errorMessage": "존재하지 않는 모집글입니다."
  }

회원가입/로그인 등 인증 관련 api들에 대한 통합 테스트 코드 작성

• 로컬 개발에 사용되는 data.sql과 분리된 teardown.sql의 더미데이터를 사용하여 테스트를 진행

teardown.sql

• 동일하게 지역 데이터를 전부 넣기에는 리소스 낭비일 것 같아서 서울과 부산의 정보만을 넣었습니다.
• data.sql을 적용하지 않고 teardown.sql을 적용하기 위해 별도의 application-test.yml을 만들어 적용해야했습니다. yml 설정으로 data.sql 적용을 취소하고, Test Class에 Sql annotation으로 teardown.sql을 적용합니다.

CustomUserDetailsService

• WithUsersDetails로 간편하게 인증을 뚫기 위해 작성하였습니다.
• 우리 서비스는 userName이 unique하기 때문에, 이름으로 가져오도록 하였습니다.
• test에서만 사용될 것이기 때문에 user 이름이 변경되거나 그런 상황까지의 고려는 하지 않았습니다.

MockBean

• 테스트를 하며 별개의 프로그램과 상호 작용하게 되어, redis의 경우 MockBean으로 하였습니다.

기존 API 버그 사항 수정

• 참여 기록 조회, 신청자 목록의 버그를 수정
  • 참여 확정인원(currentNumber)를 가져올 때 중복 조회되는 문제
  • 기존 OneToMany를 사용할때 중복 조회 발생 위험 존재, 따라서 distinct()로 중복 제거
  • 신청자 수 표시에 대한 간단한 버그 수정
• 점수 등록 API에서 점수 등록이 제대로 되지 않는 버그를 수정
  • 점수에 대한 데이터 유효성 검사 추가
  • image의 null 처리 로직 개선
• post내의 applicants를 제거하여 연관관계를 단방향으로 만들기

intellij를 통한 저장 시 자동 포맷팅 설정 및 기존 코드들에 대한 일괄 reformat 적용

• intelliJ 설정을 통해 저장 시 자동 포맷팅을 설정하였고, 기존 코드들에 대해 일괄 reformat을 적용시켰습니다.

Related Issue

연관된 이슈는 다음과 같습니다.

51 : Custom Exception 수정

65 : 참여 기록 조회 end 기준 수정

59 : post 엔티티, 참여 기록, 신청자 목록 버그 수정

69 : 점수 등록 트러블 슈팅

52 : 신청 상태 조회 api에 신청 id 추가하기

63 : 통합 테스트(성공) 작성

질문 사항

7주차 질문 사항입니다! 🤔

• 비대면 멘토 피드백 때 같이 이야기 나누었던 jpa specification 관련 코드 부분을 표시하였습니다!