[FEAT] 로컬에서도 cors 허용 요청에 따른 수정

[LimdaeIl]

📝 Pull Request

📌 PR 종류

해당하는 항목에 체크해주세요.

• 기능 추가 (Feature)
• 버그 수정 (Fix)
• 문서 수정 (Docs)
• 코드 리팩터링 (Refactor)
• 테스트 추가 (Test)
• 기타 변경 (Chore)

---

✨ 변경 내용

로컬에서도 cors 허용 요청에 따른 수정

🔍 관련 이슈

• Closes [FEAT] 로컬에서도 cors 허용 요청에 따른 수정 #96

🧪 테스트

변경된 기능에 대한 테스트 범위 또는 테스트 결과를 작성해주세요.

• 유닛 테스트 추가 / 수정
• 통합 테스트 검증
• 수동 테스트 완료

---

🚨 확인해야 할 사항 (Checklist)

PR을 제출하기 전에 아래 항목들을 확인해주세요.

• 코드 포매팅 완료
• 불필요한 파일/코드 제거
• 로직 검증 완료
• 프로젝트 빌드 성공
• 린트/정적 분석 통과 (해당 시)

---

🙋 기타 참고 사항

리뷰어가 참고하면 좋을 만한 추가 설명이 있다면 적어주세요.

Summary by CodeRabbit

변경사항

• 새로운 기능

  • 로그인하지 않은 상태에서도 모임 목록을 조회할 수 있습니다.

• 변경사항

  • 회원가입 요청에서 전화번호 필드를 제거했습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

HTTP 테스트 파일에서 회원가입 요청 페이로드의 phoneNumber 필드를 제거하고, 모임 목록 조회 테스트에 인증 없는 요청 시나리오를 추가하였습니다.

Changes

Cohort / File(s)	변경 요약
회원가입 요청 필드 제거 src/test/http/group/group-create.http	phoneNumber 필드 제거; nickName은 유지
인증 없는 모임 목록 조회 테스트 추가 src/test/http/group/group-get-cursor.http	섹션 헤더 추가 및 인증되지 않은 GET 요청 시나리오 추가 (localhost:8080/api/v1/groups?size=5)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• HTTP 테스트 파일의 단순한 필드 제거 및 시나리오 추가
• 기존 로직이나 엔드포인트 동작에 영향 없음

Possibly related PRs

• [FEAT] 모임에서 공개할 API 설정  #79: /api/v1/groups 엔드포인트를 JWT 필터 및 보안 설정에서 공개 엔드포인트로 표시하여, 이 PR의 인증 없는 모임 목록 조회 테스트를 가능하게 함
• [FEAT] 모임 상세 조회와 커서 기반 검색 구현 #68: GET /api/v1/groups 커서 기반 목록 조회 및 페이지네이션 로직 구현으로, 현재 PR의 테스트 시나리오와 직접 연관

Poem

🐰 전화번호를 빼내고
비로그인도 환영하니
테스트는 더욱 열려있고
모임 목록도 자유로워
마음껏 살펴보세요!

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/cors-local-add

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 12e21bd and f780fa5.

📒 Files selected for processing (2)

• src/test/http/group/group-create.http (1 hunks)
• src/test/http/group/group-get-cursor.http (1 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

This PR modifies HTTP test files to support testing unauthenticated access to group endpoints. The changes include adding a new test case for retrieving group lists without authentication and updating signup request body in test data.

• Added test case for unauthenticated group list retrieval
• Modified signup test data structure by removing phoneNumber field

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/test/http/group/group-get-cursor.http	Added new test case to verify unauthenticated access to group list endpoint
src/test/http/group/group-create.http	Removed phoneNumber field from signup request body

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The phoneNumber field has been removed from this signup request, but it's still present in the same signup request in group-get-cursor.http (line 9). This creates inconsistency in the test data. Consider removing phoneNumber from all test files or keeping it in all of them for consistency.

Suggested change

	"nickName": "Beemo"
	"nickName": "Beemo",
	"phoneNumber": "010-1234-5678"