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

Swagger 제거 및 Spring Rest docs 적용 #57

Merged
merged 15 commits into from
Feb 21, 2024
Merged

Conversation

devFancy
Copy link
Owner

@devFancy devFancy commented Feb 21, 2024

  • 💯 테스트는 통과했나?
  • ✅ 빌드는 성공했나?
  • 🧹 불필요한 코드는 제거했나?
  • 💭 이슈는 등록했는가?
  • 🔖 라벨은 등록했나? ex. feature, refactor

작업 내용

  • 기존 Swagger와 관련된 어노테이션 제거, 의존성 제거
  • build.gradle에 Spring RestDocs 관련 의존성 추가
  • 기존 컨트롤러 테스트 코드에 RestDocs 적용
    • 회원 컨트롤러 테스트 코드에 RestDocs 적용
    • 인증 컨트롤러 테스트 코드에 RestDocs 적용
    • 프로필 컨트롤러 테스트 코드에 RestDocs 적용
    • 게시글 컨트롤러 테스트 코드에 RestDocs 적용
  • 기존 코드에서 에러 처리
    • fix: 본인의 프로필을 조회하는 API에 대한 테스트 코드에서 get 방식으로 수정

추가

  • build.gradle 에서 RestDocs에 대한 의존성 수정 및 순서에 대한 설명 추가 - 커밋(3a8433a)

주의사항

실행 및 결과

실행

  • 방법1. 오른쪽 Gradle - Tasks - documentation - asciidoctor 실행
  • 방법2. 터미널 창에 ./gradlew asciidoctor 입력
  • 방법3. build.gradle 파일에서 asciidoctor 왼쪽에 Run 버튼 클릭

결과

  • build > docs > asciidoc > index.html 파일 -> Chome 창을 열면 아래와 같이 API 문서가 나온다.
spring-rest-docs-result

- 각 테스트 실행 전 데이터베이스 초기화 적용
- aws s3 의존성 제거
- Restdocs 적용을 위해 ControllerTestSupport 클래스에 AutoConfigureRestDocs 어노테이션 추가
- 게시글 등록, 전체 조회, 상세 조회, 검색에 대한 RestDocs 적용
- 본인의 프로필을 조회하는 API에 대한 테스트 코드 수정(put -> get)
- ProfileController와 순서와 동일하도록 맞춤
- ProfileFixtures 에서 브루스 프로필에 대한 값 수정
@devFancy devFancy added 프로젝트 환경 리팩터링 코드 리팩터링 테스트 코드 테스트 코드 작성 labels Feb 21, 2024
@devFancy devFancy self-assigned this Feb 21, 2024
Copy link

Test Results

486 tests  ±0   486 ✅ ±0   3s ⏱️ ±0s
 29 suites ±0     0 💤 ±0 
 29 files   ±0     0 ❌ ±0 

Results for commit c46b4ac. ± Comparison against base commit e84424f.

This pull request removes 10 and adds 10 tests. Note that renamed tests count towards both.
com.hibitbackendrefactor.post.application.PostServiceTest ‑ [2] logs=10:2/3, expectedIncreasedViewCount=1
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [1] log=, postId=1, expectedUpdatedLog=10:1
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [2] log=9:1/2/3, postId=1, expectedUpdatedLog=10:1
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [2] logs=10:1/2/3, postId=4, expected=true
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [3] log=10:1/2/3, postId=4, expectedUpdatedLog=10:1/2/3/4
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [3] logs=10:1/2/3, postId=3, expected=false
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [4] log=9:1/2/3&10:1/2, postId=3, expectedUpdatedLog=10:1/2/3
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [4] logs=9:1/2/3, postId=3, expected=true
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [5] logs=9:1/2/3&10:1/2, postId=3, expected=true
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [6] logs=9:1/2/3&10:1/2/3, postId=3, expected=false
com.hibitbackendrefactor.post.application.PostServiceTest ‑ [2] logs=21:2/3, expectedIncreasedViewCount=1
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [1] log=, postId=1, expectedUpdatedLog=21:1
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [2] log=20:1/2/3, postId=1, expectedUpdatedLog=21:1
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [2] logs=21:1/2/3, postId=4, expected=true
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [3] log=21:1/2/3, postId=4, expectedUpdatedLog=21:1/2/3/4
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [3] logs=21:1/2/3, postId=3, expected=false
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [4] log=20:1/2/3&21:1/2, postId=3, expectedUpdatedLog=21:1/2/3
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [4] logs=20:1/2/3, postId=3, expected=true
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [5] logs=20:1/2/3&21:1/2, postId=3, expected=true
com.hibitbackendrefactor.post.domain.ViewCountManagerTest ‑ [6] logs=20:1/2/3&21:1/2/3, postId=3, expected=false

@devFancy devFancy merged commit c7c1770 into main Feb 21, 2024
4 checks passed
@devFancy devFancy changed the title [feat] Swagger 제거 및 Spring Rest docs 적용 Swagger 제거 및 Spring Rest docs 적용 Feb 23, 2024
@devFancy devFancy added the 문서 작성 Improvements or additions to documentation label Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
리팩터링 코드 리팩터링 문서 작성 Improvements or additions to documentation 테스트 코드 테스트 코드 작성 프로젝트 환경
Projects
None yet
Development

Successfully merging this pull request may close these issues.

기존 API 문서로 사용했던 Swagger 에서 Spring Rest Docs로 전환한다.
1 participant