Skip to content

3주차 멘토링

김서진 edited this page Nov 11, 2021 · 1 revision

질문 1

Q. 처음 로그인을 할 때 SWR이 undefined를 첨에 받는다 ㅠ 렌더링되기 전에 초기값을 설정할 수 있나

멘토님은 redux랑 useSWR을 쓰고 계신다.

SWR! 데이터가 넘어오기 전에는 undefined, 받아왔는데 데이터가 없을때는 빈 배열이 온다.

로딩 컴포넌트를 띄워주기. GraphQL에서도 로딩상태가 있어서 분기를 탄다.

라우팅할 때 새로고침하면 userData가 undefined라서 로그인창으로 튕겨저 나온다ㅠ (전역 데이터로 넣어서 로그인된 사용자인지 판별하고 싶었는데! 그래서 다른 방법을 사용 - 페이지를 이동할때마다 userData 요청을 한다 그래서 너무 요청이 많다고 생각)

→ 멘토님께서 코드를 더 읽어보시고 답변해주실 예정!

질문 2

Q. 테스트 어떻게 어디까지 할까요??

소켓까지 단위테스트를 해가는건 비추! 시간이 빡빡하니까. 기본적인 렌더링이나 유틸성 테스트를 해보는 것이 좋을 것 같다.

e2e 테스트 나쁘지않다. 진입장벽이 있어서 테스트를 작성하는것 자체가 일이다. 테스트짜는 습관을 들이는 것은 좋다. 일정이나 리소스를 확보해둔 상태에서 하는 것이 좋겠다.

유틸성 단위테스트랑 e2e 테스트를 진행하고 있다.

질문 3

Q. PR을 확인할 시간을 따로 갖나요?

pr을 리뷰하는 시간도 리소스를 갖는다.

시간이 엄청 길어질 것 같으면 merge하고 다음 pr에 리뷰를 반영한다.

여유가 없으면 오프라인 코드리뷰를 하거나 리팩터링하는 시간을 따로 마련한다

웬만하면 바로 리뷰를 상세하게 하는 것이 좋다 불필요한 시간을 줄일 수 있게

pr을 올리는 사람도 다른사람들이 이해하기 좋게 올려야 한다

유연함이 필요하다. 미리 approve를 한다든지 모아서 pr을 확인한다든지

하지만 모든 커밋이 history로 남기 때문에 잘 작성하는 것이 좋다

pr 단위, 커밋 단위도 중요하다

에러가 안나는 기준까지 pr을 날린다

Clone this wiki locally