Skip to content

[Test] Store API 테스트 #207

Merged
isak-kang merged 4 commits into
developfrom
test/#186-store-test
Oct 28, 2025
Merged

[Test] Store API 테스트 #207
isak-kang merged 4 commits into
developfrom
test/#186-store-test

Conversation

@isak-kang

@isak-kang isak-kang commented Oct 26, 2025

Copy link
Copy Markdown
Contributor

🛠️ Issue Number

closes #186

📌 작업 내용 및 특이사항

  • 상점 생성 (공통 응답 포맷에 맞춰 수정.)

Store API Test

  • 상점 생성 API 테스트
    • 상점명 누락
    • 연락처 누락
    • 주소 누락
    • 사업자등록번호 누락
    • 통신판매업번호 누락
  • 상점 수정 API 테스트 (PATCH /stores/{storeId})
  • 상점 삭제 API 테스트 (DELETE /stores/{storeId})
  • 상점 목록 조회 API 테스트 (GET /stores)
  • 상점 상세 조회 API 테스트 (GET /stores/{storeId})
  • 내 상점 상품 목록 조회 API 테스트 (GET /stores/products)
  • 특정 상점 상품 목록 조회 API 테스트 (GET /stores/{storeId}/products)

수정사항 : 경로 파라미터 추가하기

@isak-kang isak-kang self-assigned this Oct 26, 2025
@isak-kang isak-kang added the ✅ test 테스트를 작성, 또는 수정합니다. label Oct 26, 2025
@isak-kang isak-kang added this to the Sprint 3 milestone Oct 26, 2025
@Bal1oon

Bal1oon commented Oct 26, 2025

Copy link
Copy Markdown
Contributor

제가 테스트 코드는 딱히 해본적이 없어서, 해당 테스트는 단순 CRUD만 가능한 테스트인가요 아니면 검증까지 전부 할 수 있는 테스트인가요?
@willjsw

다양한 시나리오 검증이 가능하다면 추가될 수 있는 시나리오는 아래와 같이 더 있을 거 같아요. 생각해보면 더 나올 거에요:

- 상점명 중복 불가
- 같은 유저의 다중 상점 소유 불가
- OWNER 권한만 상점 생성 가능 (Security 적용이 안되어 안 될 수 있음)
- 본인 상점만 수정/삭제 가능
- 상점 삭제 후 다시 조회 시 조회 불가되어야 함 (Soft Delete 확인)
- 사업자등록번호, 연락처 등 형식이 정해진 필드 안 지켰을 때의 검증 확인
- 존재하지 않는 상점 수정/삭제

@willjsw

willjsw commented Oct 26, 2025

Copy link
Copy Markdown
Collaborator

제가 작성한 mvc 테스트는 비즈니스 로직에 대한 테스트보다는 요청/응답에 대한 테스트입니다!
컨트롤러가 호출하는 서비스 컴포넌트가 저희가 원하는 반환값을 도출한다는 전제 하에,

의도한 입력값들에 대한 발리데이션(@valid)이 제대로 이루어지는지, 의도한 응답 필드들이 제대로 출력되는 지 검증하는 테스트입니다! 그 외 서비스 레어의 비즈니스 로직들은 Junit등 별도 테스트를 작성해야 합니다!

작성해주신 부분에 대한 테스트가 꼭 필요하다고 저도 생각합니다! 다만 테스트가 처음이신 분들이 있고, 계획했던 것보다 개발 일정이 지연되고 있어서 내일 함께 논의해보면 좋을 듯 하네요!

@willjsw willjsw left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

고생하셨습니다!

@isak-kang
isak-kang merged commit 7a167a0 into develop Oct 28, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✅ test 테스트를 작성, 또는 수정합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Test] Store Test

3 participants