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

✨ 피드 메인 목록 테스트 코드 작성 #39

Merged
merged 3 commits into from
May 8, 2024

Conversation

BangDori
Copy link
Collaborator

@BangDori BangDori commented May 7, 2024

작업 이유

  • 피드 메인 목록 테스트 코드 작성

작업 사항

  • 피드 메인 페이지 에러 테스트
    image

  • 피드 메인 페이지 스켈레톤 UI 테스트 및 콘텐츠 테스트
    image


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

  • 테스트 코드의 타이틀이 직관적인가요?
  • 부족한 테스트가 있나요?

발견한 이슈

  • 원래 PR할 때 테스트 코드 작성까지 추가해야 했는데, 작업을 미루다보니 이제야 작성하게 되었습니다. 다음부터 기능 구현에 대한 PR을 작성할 때, 테스트 코드까지 함께 업로드하겠습니다.

@BangDori BangDori requested a review from Legitgoons May 7, 2024 16:13
@BangDori BangDori self-assigned this May 7, 2024
Copy link

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

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

Copy link
Member

@Legitgoons Legitgoons left a comment

Choose a reason for hiding this comment

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

타이틀 관련해서 조그만 의견 남겨봅니당 ㅋㅋㅋ 병합하셔도 좋습니다.

@BangDori BangDori merged commit 6bae5cf into main May 8, 2024
2 checks passed
@BangDori BangDori deleted the test/feed-main-list branch May 8, 2024 05:19
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