Skip to content

[#10] 요청 본문 처리와 실행 API를 구성한다#18

Merged
opficdev merged 4 commits into
developfrom
feat/#10-body&api
Apr 13, 2026
Merged

[#10] 요청 본문 처리와 실행 API를 구성한다#18
opficdev merged 4 commits into
developfrom
feat/#10-body&api

Conversation

@opficdev
Copy link
Copy Markdown
Owner

🔗 연관된 이슈

📝 작업 내용

📌 요약

  • 요청 본문 처리 API 구현
  • 요청 실행 API 구성 구현
  • 요청 본문 처리와 실행 API 테스트 구현

🔍 상세

  • 요청 본문 처리 API 추가
    • NXRequestBuilder.json(_:encoder:)
    • NXRequestBuilder.body(_:contentType:)
    • JSON 인코딩 시 Content-Type: application/json; charset=utf-8 설정
  • 요청 실행 API 구성
    • NXRequestBuilder.raw()
    • NXRequestBuilder.send(as:)
    • NXRequestBuilder.sendVoid()
    • 요청 빌드 로직이 아직 연결되지 않은 단계이므로 NXError.invalidRequest를 반환하도록 구성
  • 테스트 추가
    • NXRequestBodyExecutionAPITests
    • 설정 인코더 적용/오버라이드 인코더 우선 적용 검증
    • raw body 데이터 및 Content-Type 헤더 저장 검증
    • 실행 API 호출 시 NXError.invalidRequest 발생 검증

테스트

  • swift test 통과

📸 영상 / 이미지 (Optional)

image

@opficdev opficdev self-assigned this Apr 13, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

이번 PR은 NXRequestBuilder에 JSON 및 Raw 데이터 본문 설정을 위한 메서드와 요청 실행 API의 스텁 구현을 추가하고, 이에 대한 단위 테스트를 포함하고 있습니다. 리뷰 결과, requestSpec.body와 headers에 Content-Type을 중복으로 설정하는 부분이 지적되었습니다. 이는 데이터 불일치를 유발할 수 있으므로 body에만 정보를 담도록 수정이 권장됩니다.

Comment thread Nexa/Sources/Nexa/Public/NXRequestBuilder.swift Outdated
Comment thread Nexa/Sources/Nexa/Public/NXRequestBuilder.swift Outdated
@opficdev opficdev merged commit 6fdf19e into develop Apr 13, 2026
4 checks passed
@opficdev opficdev deleted the feat/#10-body&api branch April 13, 2026 03:07
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.

요청 본문 처리와 실행 API를 구성한다

1 participant