AS
@Builder
public record RecordUpdateCommand(
Long editorId,
String recordPublicId,
String title,
LocalDateTime startedAt,
LocalDateTime endAt,
Long totalDistanceInMeters,
Long averagePaceInMilliSeconds
) {
}
서비스에서 사용하는 Command 객체는 Public Id를 요하는데 api에서는 PK값을 활용하고 있음.
또한 API응답은 PK값을 제공하는데 UPDATE 로직에서는 publicId 를 사용함.
- Record Update에 대한 검증을 하지 않았네요 ㅡㅡ 죄송합니당
@PatchMapping("/{recordId}")
public ResponseEntity<Void> updateRecord(
@RequestBody RecordUpdateRequest request,
@UserId Long userId
) {
recordUpdateUsecase.updateRecord(RecordUpdateRequest.toCommand(userId, request));
return ResponseEntity.ok().build();
}
- path variable은 사용하지도 않는군요... 고치고 테스트도 짜겠습니다 😢
TOBE
- 기존의 논리 오류를 수정합니다.
- 테스트코드를 작성합니다.
- 모든 Client - Server 통신에 public ID를 사용합니다.
AS
서비스에서 사용하는 Command 객체는 Public Id를 요하는데 api에서는 PK값을 활용하고 있음.
또한 API응답은 PK값을 제공하는데 UPDATE 로직에서는 publicId 를 사용함.
TOBE