Skip to content

Conversation

@X1n9fU
Copy link
Contributor

@X1n9fU X1n9fU commented Jul 1, 2025

#️⃣ 연관된 이슈

ex) #이슈번호, #이슈번호
#215

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

제휴업체를 반환하는 기능과 추후에 관리자가 제휴업체를 관리할 가능성이 있어, 생성 삭제 로직까지만 제작하였습니다.
이에 대한 테스트까지 진행하였습니다.

아직 실제 제휴업체의 정보를 받지 못하여서 더미 데이터를 넣어 놓고 진행하였습니다.

'Department` 에서 직렬화하는 과정에서 오류가 나서 수정하였습니다.

스크린샷 (선택)

image

image

ROLE이 "USER" 인 유저가 제휴업체를 생성하거나 삭제할 경우 -> Access Denied 오류
image

image

더미 데이터 내용

[{
  "_id": {
    "$oid": "686388cc0f24350acbee1b8d"
  },
  "name": "홍콩반점 송도점",
  "tags": [
    "COMPUTER_SCI",
    "EMBEDDED"
  ],
  "benefits": [
    "탕수육 주문 시, 탕수육 공짜!",
    "평일 언제나 80% 대박 할인!"
  ],
  "startDate": {
    "$date": "2025-02-28T15:00:00.000Z"
  },
  "endDate": {
    "$date": "2026-02-28T15:00:00.000Z"
  },
  "img": {
    "main": "https://codin-s3-bucket.s3.ap-northeast-2.amazonaws.com/d1bf2a2d-efac-4c6c-9239-cd0876705d47.png",
    "sub": [
      "https://codin-s3-bucket.s3.ap-northeast-2.amazonaws.com/44db3472-8981-4ff6-8e90-4669eca3abd8.jpg",
      "https://codin-s3-bucket.s3.ap-northeast-2.amazonaws.com/99390def-3eb7-456e-9712-23bf8793b021.jpg"
    ]
  },
  "_class": "inu.codin.codin.domain.info.entity.Partner",
  "location": "인천 연수구 송도동 3-2"
},
{
  "_id": {
    "$oid": "68638efc8f1ee454fe6c9225"
  },
  "name": "이미지 없는 제휴업체",
  "tags": [
    "COMPUTER_SCI",
    "INFO_COMM",
    "EMBEDDED"
  ],
  "benefits": [
    "탕수육 주문 시, 탕수육 공짜!",
    "평일 언제나 80% 대박 할인!"
  ],
  "startDate": {
    "$date": "2025-02-28T15:00:00.000Z"
  },
  "endDate": {
    "$date": "2026-02-28T15:00:00.000Z"
  },
  "location": "인천 연수구 송도동 3-2",
  "img": {
    "sub": []
  },
  "_class": "inu.codin.codin.domain.info.entity.Partner"
},
{
  "_id": {
    "$oid": "686390968f1ee454fe6c9226"
  },
  "name": "메가커피 송도 센트럴파크점",
  "tags": [
    "IT_COLLEGE",
    "INFO_COMM"
  ],
  "benefits": [
    "매주 수요일 아메리카노 50% 할인!",
    "도장 10장 적립 시, 어떤 음료라도 무료 제공"
  ],
  "startDate": {
    "$date": "2025-02-28T15:00:00.000Z"
  },
  "endDate": {
    "$date": "2026-02-28T15:00:00.000Z"
  },
  "location": "인천광역시 연수구 하모니로 138번길 11 1층",
  "img": {
    "main": "https://codin-s3-bucket.s3.ap-northeast-2.amazonaws.com/b888c55d-b964-4e0f-9706-aa428ec425f8.png",
    "sub": [
      "https://codin-s3-bucket.s3.ap-northeast-2.amazonaws.com/f08343d3-7b84-42fb-bd45-4ee0206df7c4.jpg"
    ]
  },
  "_class": "inu.codin.codin.domain.info.entity.Partner"
}]

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

X1n9fU and others added 29 commits April 1, 2025 02:20
fix : Google Access_Token 세션 저장 후 로그아웃
fix : 강의 후기 Participants 반환 오류 수정 완료
fix : 게시글 반환 시 자신의 게시글인지 확인하는 isMine 변수 추가
fix : 베스트 게시글 Scheduler 및 조회 시에 삭제된 게시글에 대해 확인 및 데이터 삭제
chore : Swagger ADMIN만 접근 권한 부여
- EmailAuthService와 EmailSendService 계층 분리
- 과거 EmailAuthService을 JoinEmailAuthService와 PasswordResetEmailService로 책임 분리
- EmailSendService에서 EmailTemplateService로 변경을 통해 이메일 전송 역할 명확화
아래와 같은 세가지에 대해서 테스트를 작성했습니다.
- EmailTemplateServiceTest
- JoinEmailAuthServiceTest
- PasswordResetEmailServiceTest

테스트시 객체 생성 메서드가 없기에 두개의 DTO 클래스에 Builder를 추가했습니다.
- JoinEmailCheckRequestDto
- JoinEmailSendRequestDto
- EmailController : 불필요한 공백 제거
- EmailTemplateService : 예외처리 추가
- EmailTemplateServiceTest : 가독성 개선
- JoinEmailAuthService : 잘못된 주석 제거와 가독성 향상
- PasswordResetEmailService : 잘못된 주석 제거
- EmailTemplateFailException 으로 예외처리 추가
비밀번호 재인증 로직 중 기존 인증정보가 존재하지 않을 때에는 예외처리
- EmailPasswordResetFailException 으로 예외처리 추가
수정된 기능 상황에 맞게 테스트 수정
- EmailTemplateFailException : import 최적화
- EmailTemplateFailException : 테스트 예외처리 수정
@X1n9fU X1n9fU requested a review from doma17 July 1, 2025 08:12
@X1n9fU X1n9fU requested a review from gisu1102 July 1, 2025 08:12
@X1n9fU X1n9fU self-assigned this Jul 1, 2025
@X1n9fU X1n9fU added ✨ Feature 기능 개발 Priority : High 우선순위 상 Status: Reviewing 처리하고 리뷰 중인 이슈 labels Jul 1, 2025
Copy link
Contributor

@doma17 doma17 left a comment

Choose a reason for hiding this comment

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

추가적으로 기능이 더 추가된다면 PartnerService를 Read, Write(Delete 포함)으로 나누어도 좋을 것 같아요.

Copy link
Contributor

@gisu1102 gisu1102 left a comment

Choose a reason for hiding this comment

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

확인완료했습니다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 기능 개발 Priority : High 우선순위 상 Status: Reviewing 처리하고 리뷰 중인 이슈

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants