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

⚙️ MSW Follow, Search, User Test 코드 #51

Merged
merged 19 commits into from
May 11, 2024

Conversation

Legitgoons
Copy link
Member

작업 이유

Follow, Search, User API에 대한 Test 코드 등록


작업 사항

  • 기존의 API 명세에서는 User API 하나로 통일되어 있었습니다.
  • 하지만 test 코드를 작성하며 분리할 필요성을 느껴, Follow, Search, User API로 분리하였습니다.
  • wiki를 수정하고, 이에 따라 test코드를 작성하였습니다.

1️⃣ Follow

  • 동작 test
    image
  • 에러 test
    image

2️⃣ Search

  • 동작 test
    • 좋아요를 누른 유저 검색을 위해 likeUser mock data를 추가했습니다.
      image
  • Search는 잘못된 값을 입력해도 빈 배열만 return받기 때문에 error test가 없습니다.

3️⃣ User

  • 동작 test
    image
  • 에러 test
    image

리뷰어가 중점적으로 확인해야 하는 부분

  • User API가 분리된 것을 인지하였나요?
  • Follow, Search, User test 코드 중 추가해야할 부분은 없나요?
  • 이슈를 확인하셨나요?

발견한 이슈

  • 사용자 프로필 피드 조회와 사용자 조회를 같은 URI + query string로 가져오려고 했습니다.
  • 하지만 URI와 method가 동일해 사용자 조회만 이뤄지고 있었습니다.
  • 따라서 아래와 같이 URI를 수정했습니다.
// 기존 uri
/users/{user_id} // 사용자 조회
/users/{user_id}?page= // 사용자 프로필 피드 조회
// 변경된 uri
/users/{user_id} // 사용자 조회
/profile/{user_id}?page= // 사용자 프로필 피드 조회

@Legitgoons Legitgoons self-assigned this May 11, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-51.d37mn03xh3qyyz.amplifyapp.com

Copy link
Collaborator

@BangDori BangDori left a comment

Choose a reason for hiding this comment

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

테스트 코드를 간결하고 가독성 높게 잘 작성하신 것 같아서 편하게 코드를 읽을 수 있었던 것 같습니다!!

테스트 코드 작성하시느라 고생하셨어요 ㅎㅎ

src/app/mocks/handler/follow.ts Show resolved Hide resolved
src/app/mocks/test/user/user.test.ts Show resolved Hide resolved
src/app/mocks/test/user/user.test.ts Show resolved Hide resolved
@Legitgoons Legitgoons merged commit 86deaf8 into main May 11, 2024
2 checks passed
@BangDori BangDori deleted the feature/PW-296-user-test branch May 12, 2024 16:48
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.

2 participants