Skip to content

Conversation

@CodeVac513
Copy link
Collaborator

🔨 테스크

ACL 문제

부끄러운 말이지만, 그 동안 로컬 환경에서 Nest API 서버의 테스트를 돌려본 적이 없었다.
그래서 환경변수가 있으면 ACL 인증을 시도하려다가 test container에는 .env가 없어서 터지는 문제를 이제야 확인할 수 있었다.
이 부분은 성윤님 도움을 받아서, 문제를 구체화할 수 있었다.
integration-test-global-setup.ts에서 username과 password를 빈 문자열로 설정하여 ACL 인증을 건너뛰도록 설정했다.

📋 작업 내용

  • 게시글 상세 모달에 필요한 데이터 조회 API 구현
  • 페이지네이션 API에서 tag 정보를 반환하도록 수정
  • 게시글 상세 모달 데이터 조회 API 테스트 작성
  • 각각 API의 스웨거 문서 작성 및 수정
  • 환경변수로 인해 ACL 인증을 시도하는 부분 수정

📷 스크린 샷

  • 상세 모달 데이터 조회 API 응답 형식 확인
    image

  • 페이지네이션 API 응답 형식 확인
    image

  • 테스트 동작
    image

@CodeVac513 CodeVac513 added ✨ Feature 기능 구현 ✅ Test 테스트 관련 (storybook, vitest, jest 등) labels Feb 13, 2025
@CodeVac513 CodeVac513 self-assigned this Feb 13, 2025
Copy link
Member

@Jo-Minseok Jo-Minseok left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 오류 하나 발견해서 고쳐주시면 감사하겠습니다.

Copy link
Member

@asn6878 asn6878 left a comment

Choose a reason for hiding this comment

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

확인했습니다!
고생하셨어요~

Copy link
Member

@Jo-Minseok Jo-Minseok left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@CodeVac513 CodeVac513 merged commit a01fb56 into main Feb 13, 2025
2 checks passed
@CodeVac513 CodeVac513 deleted the feature/feed-detail-modal-api branch February 13, 2025 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feature 기능 구현 ✅ Test 테스트 관련 (storybook, vitest, jest 등)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants