wjrmffldrhrl / goldenblock

블록체인교육 프로젝트
1 stars 1 forks source link

MemberController #31

Closed wjrmffldrhrl closed 4 years ago

wjrmffldrhrl commented 4 years ago

MemberController.java

https://github.com/wjrmffldrhrl/goldenblock/blob/8f1e5700df752fcf991c130f985ea7b859b0f554/src/main/java/com/blockchain/goldenblock/controller/MemberController.java#L15-L28

해당 코드에서 몇가지 테스트와 수정해야 할 사항이 있습니다.

  1. 해당 StudentDto는 id값이 존재하는데 post 요청으로 body에 담기는 값이 맵핑될 때 비어있는 값은 null처리 되는가?
  2. idcheck는 controller에서 사용되지 않을 것 같습니다. 또한 service에 구현한 idCheck는 jpa서 기본으로 제공해주는 기능이 있으니 그것을 사용하면 될 것입니다.
  3. StudentDto에서 @Builder 어노테이션을 잘 사용하면 아레에 작성한 코드들도 생략 가능 할 것입니다.
wz0405 commented 4 years ago

네 확인했습니다.

  1. 회원이 등록될때마다 id값이 자동으로 생성되는것으로 알고 있습니다. 확실하지 않으니 틀린 부분이면 코멘트 부탁드립니다.
  2. 409 에러로 처리하도록 추가했습니다. 테스트는 하지 못했으니 테스트 부탁드립니다.
  3. 코드 생략은 완료했습니다.
wz0405 commented 4 years ago

{ "name":"hhh", "school":"han", "email":"gks0405@naver.com", "password":"1234asdfsadfasgasdf" }

  1. 다음과 같이 입력받으면 DB에 저장되는것까지 확인했습니다.
  1. 중복된 이메일이 가입을 시도하면 409에러가 나도록 유도했습니다.