[moin-review] 2021-10-21

김_리트리버·2021년 11월 15일
0

Facts

송금 flow routing 작업 진행

모인의 경우 송금단계가 4단계에 걸쳐 이루어짐

  1. 송금방식 선택
  2. 과거 송금보낸 목록에서 받는 사람 선택 ( 송금 보낸적 있는 유저만 )
  3. 받는사람 정보 입력 ( 2번에서 받는사람 선택 안할 경우 또는 처음 송금을 보낼 경우 )
  4. 송금목적, 받는사람과 보내는 사람과의 관계, 통장메모

근데 1 단계의 경우 특정국가 ( 일본, 중국, 영국 ) 로 송금을 보낼 경우에만 화면을 보여주어야 했음

이 문제를 해결하기 위해 국가코드를 기준으로 조건에 부합하면 송금방식 화면을 보여주고

부합하지 않으면 다른 화면으로 넘기는 방식을 사용했음

단순히 redirect 시킬 까 생각해 보았지만 현재 앱에서는 사용할 수 없었음

왜냐면 현재 모인 웹의 경우 송금 보낼금액, 국가를 선택한 후 송금신청하기 버튼을 클릭해서 송금 과정으로 넘어오는데, 기존에 송금 보낼금액, 국가의 상태를 유지하고 있어야 하기 때문에 사실상 resource 를 초기화 하는

redirect 는 사용할 수 없었음

대신에 history api 의 replace 를 사용하여 url 만 history stack 에서 교체하는 방식을 사용했음

이렇게 하면 redux 상태를 유지하면서 화면만 변경할 수 있었음

2단계도 마찬가지로 특정 조건에 따라 화면을 그대로 보여주거나 replace 하는 방식을 사용할 예정임

Findings

mocking 을 사용하지 않으려면 최대한 함수의 관심사를 철저하게 분리해야 함

한 함수에서 여러가지 일을 하게 되면 굳이 상관없는 것 까지 mocking 해줘야 함

외부 라이브러리의 경우는 제대로 동작하는 지 test 할 필요가 굳이 없음

최대한 의존성을 분리해서 mocking 하는 부분을 줄여야 함

아니면 mocking 하느라 시간 다날림

Feelings

전반적으로 괜찮았지만 test code 쓸 때 자잘한 오타실수 때문에 시간을 날린점이 아쉬웠다.

profile
web-developer

0개의 댓글