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

[NB-131] 게시글 작성 api - 인증기반으로 수정 #25

Merged
merged 5 commits into from
Jul 24, 2024

Conversation

liljoon
Copy link
Contributor

@liljoon liljoon commented Jul 22, 2024

NB-131:

  • 게시글 생성 시 유저 기반 인증
  • 게시글 삭제 시 유저 기반 인증 및 인가 처리
  • 관련 테스트 코드 수정 및 추가

개요

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 테스트 추가, 테스트 리팩토링

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고 (Ctrl + 클릭하세요.)
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

liljoon added 4 commits July 22, 2024 15:12
- 로그인된 인증 유저를 기반으로 게시글 업로드
- 게시글 제목, 본문 글자 수 validation 설정
- 인증기반으로 테스트하도록 수정
- 게시글 제목, 본문 최대 길이 경계값 테스트 코드 추가
- 유저 소유의 게시글 검증 기능
- Forbidden 예외처리 클래스 생성
- 다른 사용자의 게시글 삭제 시 에러 테스트 구현
- public 접근제어지시자 제거
@liljoon liljoon requested a review from Aram-su July 22, 2024 08:38
- mockMvc를 통해 인증 토큰 생성 로직 이용
- 중복된 모킹 삭제
Copy link
Contributor

@Aram-su Aram-su left a comment

Choose a reason for hiding this comment

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

코드 확인하였습니다. 문제 없는 것 같습니다.

@liljoon liljoon merged commit a7da841 into dev Jul 24, 2024
1 check passed
@liljoon liljoon deleted the feat/post-auth branch July 24, 2024 01:13
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.

2 participants