-
Notifications
You must be signed in to change notification settings - Fork 2
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
[FE] refactor: 리뷰 그룹 생성 API 요청 함수 및 MSW 핸들러, 테스트 추가 #420
Conversation
|
||
import QueryClientWrapper from '@/queryTestSetup/QueryClientWrapper'; | ||
|
||
import { CREATED_REVIEW_REQUEST_CODE } from '../../../mocks/mockData/group'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
경로에 절대 경로 사용하면 어떨까요?
import { CREATED_REVIEW_REQUEST_CODE } from '../../../mocks/mockData/group'; | |
import { CREATED_REVIEW_REQUEST_CODE } from '@/mocks/mockData/group'; |
describe('usePostDataForReviewRequestCode', () => { | ||
it('ReviewRequestCode를 발급받을 수 있다.', async () => { | ||
// given | ||
const dataForReviewRequestCode = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const dataForReviewRequestCode = { | |
const DATA_FOR_REVIEW_REQUEST_CODE= { |
상수로 바꿔도 좋을 것 같아요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 변수는 해당 테스트에서만 사용돼서 일단 별도로 상수화하지는 않았습니다!
…eview-me into fe/feat/413-new-create-review-group-api
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생했어요 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
올리 고생했어
🚀 어떤 기능을 구현했나요 ?
HomePage
에서 이름, 프로젝트 이름, 비밀번호를 입력했을 때ReviewDashboard
로 접근할 수 있는 URL을 얻기 위한 API 호출 함수와 관련된 작업을 했습니다.🔥 어떻게 해결했나요 ?
📝 어떤 부분에 집중해서 리뷰해야 할까요?
📚 참고 자료, 할 말
groupAccessCode
로 통일하자고 했는데, 예전에 컴포넌트를 만들 때 이미password
라는 이름을 너무 많이 사용하고 있어서 컴포넌트상의password
는 별도로 수정하지 않고 API 요청을 보낼 때만groupAccessCode
로 명시해서 보내고 있습니다.