Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ADD] 유저 정보 UPDATE #6

Closed
wants to merge 1 commit into from
Closed

[ADD] 유저 정보 UPDATE #6

wants to merge 1 commit into from

Conversation

Gyelanjjim
Copy link
Contributor

@Gyelanjjim Gyelanjjim commented Dec 2, 2022

:: 최근 작업 주제 (하나 이상의 주제를 선택해주세요.)

  • 기능 추가
  • 데이터베이스 작업
  • 리팩토링
  • 버그 수정
  • 컨벤션 수정

:: 구현 목표 - 해당 브랜치(PR)에서 구현하고자 하는 하나의 목표 작성

  • 유저 정보 업데이트 PATCH

:: 구현 사항 설명 - 해당 브랜치(PR)에서 작업한 내용 작성

  • 이미지 업로드 테스트를 위한 PATCH, /test/upload 를 테스트

  • app.js와 server.js의 분리

  • 토큰검증 미들웨어 추가 및 postman 테스트 완료.

  • s3 이미지 업로드 기능 구현(검증x)

  • API 호출 시 필요한 값들에 대한 유효성 검사

    • 유저정보를 일괄 수정하는 방식이 아니라, 한 종류의 데이터마다 API를 호출 : 어떤 데이터도 들어오지 않았을 때만 KEY_ERROR 처리
    • 유저 소개글(description)은 빈문자열("")로 설정할 수 있어야 하므로, undefined일 때 KEY_ERROR 처리
  • service 에서

    • 유저 닉네임 중복 시 중복 에러 반환
    • 수정할 유저 정보를 Bulk update하는 함수 선언
      • 유저 정보가 string type일 때 따옴표('')로 wrapping .... ex) SET address = "주소", latitude = 34.111, longitude = 127.111
  • 유닛테스트 및 AWS S3 미구현


:: 테스트 결과 이미지

  1. Server가 잘 동작하는지 확인할 수 있는 Terminal 캡쳐 이미지

스크린샷 2022-12-02 오후 7 25 14

  1. Postman(Client Tool)을 이용한 API 테스트 결과 이미지

스크린샷 2022-12-02 오후 6 25 59

스크린샷 2022-12-02 오후 6 26 22

스크린샷 2022-12-02 오후 6 27 33

스크린샷 2022-12-02 오후 6 27 45

스크린샷 2022-12-02 오후 6 28 37

스크린샷 2022-12-02 오후 6 28 48

스크린샷 2022-12-02 오후 6 29 42

스크린샷 2022-12-02 오후 6 29 53

  1. 작성한 Test Code가 잘 통과했는지 확인할 수 있는 이미지
  2. DB 작업 PR인 경우 Table 생성 및 수정 결과에 대해 확인할 수 있는 이미지

:: 기타 질문 및 특이 사항

  • 포스트맨으로 테스트해봤는데 access denied 됩니다 이유가 뭘까요.. 👉 해결됐습니다! s3 버킷정책을 덜 설정했기 때문.

스크린샷 2022-12-07 오후 10 23 33

스크린샷 2022-12-07 오후 10 29 21

스크린샷 2022-12-07 오후 10 31 35

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.

None yet

2 participants