Skip to content

FIX : 아이 정보 응답에 빠져 있던 특이사항 - #96

Merged
RosieOh merged 1 commit into
mainfrom
fix/child-special-needs-response
Aug 24, 2026
Merged

FIX : 아이 정보 응답에 빠져 있던 특이사항#96
RosieOh merged 1 commit into
mainfrom
fix/child-special-needs-response

Conversation

@RosieOh

Copy link
Copy Markdown
Contributor

🏫 관련 이슈

Closes#95

🏫 PR 타입

  • 기능 추가
  • 기능 수정
  • 기능 삭제
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

🏫 반영 브랜치

fix/child-special-needs-response -> main

🏫 변경 사항

ChildInfoResponsespecialNeeds 가 없었다.
등록·수정 요청인 ChildCreateRequest 는 이 값을 받는데 응답에만 빠져 있어서,
수정 화면이 현재 값을 읽어올 방법이 없었다.

ChildService.updateChild 가 전체 교체라는 점이 겹쳐서 데이터가 유실된다.

child.setSpecialNeeds(request.getSpecialNeeds()); // 조건 없이 덮어쓴다

클라이언트가 이름만 고쳐 보내면 특이사항이 null 로 지워진다.
알레르기·기저질환처럼 지워지면 곤란한 값이라 조용한 유실이 특히 나쁘다.

응답에 필드를 실어 클라이언트가 현재 값을 되돌려줄 수 있게 했다.

파일변경
ChildInfoResponsespecialNeeds 필드 추가
ChildMapper매핑 추가
ChildMapperTest회귀 테스트 (신규)

근본적으로는 부분 수정(PATCH)이 없어서 생기는 문제다.
지금은 응답에 값을 실어 왕복시키는 것으로 막았고,
수정 요청을 부분 갱신으로 바꾸는 것은 별도로 다루는 편이 낫다고 봤다.

🏫 테스트 결과

380 tests, 0 failures
BUILD SUCCESSFUL (jacocoTestCoverageVerification 포함)

ChildMapperTest 4건 추가.

케이스확인
특이사항이 응답에 포함된다값이 없으면 수정 요청이 되돌려줄 수 없어 유실된다
특이사항이 없으면 null
나머지 필드도 그대로 옮긴다id, userId, name, gender, birthDate, createdAt
updatedAtnull 이어도 터지지 않는다수정된 적 없는 아이

🏫 To Reviewer

  • 응답에 필드가 늘어나므로 프런트 타입(CareCode_FE)에도 specialNeeds 를 반영해야 한다.
  • 같은 형태의 "요청에는 있는데 응답에 없는 필드" 가 다른 도메인에도 있는지는 확인하지 않았다.
    전체 교체 방식의 수정 API 가 여럿이라 같은 유실이 반복될 수 있다.

ChildInfoResponse 에 specialNeeds 가 없었다. 등록·수정 요청(ChildCreateRequest)은
이 값을 받는데 응답에는 없어서, 수정 화면이 현재 값을 읽어올 방법이 없었다.
updateChild 는 전체 교체다.
child.setSpecialNeeds(request.getSpecialNeeds()); // 조건 없이 덮어쓴다
그래서 클라이언트가 이름만 바꿔 보내면 특이사항이 null 로 지워졌다.
알레르기·기저질환처럼 지워지면 곤란한 값이라 응답에 실어 되돌려줄 수 있게 한다.
회귀 테스트를 붙였다.
380 tests, 0 failures.
Sign up for freeto 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.

[ Child ] 아이 정보 응답에 특이사항이 빠져 수정 시 값이 지워진다

1 participant

@RosieOh