🛠️ 현실적인 환경 구축을 위해 모든 API에 대해 delay 설정

[BangDori]

작업 이유

#45 이슈 해결

작업 사항

1️⃣ 모든 API에 대한 delay 적용

/** 0 <= 응답 시간 <= 4초 */
const delayTime = Math.random() * 4001;

export async function createHttpSuccessResponse<T>(data: T) {
  await delay(delayTime);

  // ...
}
export async function createHttpErrorResponse(code: keyof typeof errorMessage) {
  await delay(delayTime);
  
  // ...
}

모든 응답은 위 함수를 사용하고 있는 것으로 판단되어, 위 함수들에 대해 delay를 적용하였습니다. 아래는 테스트 결과에 대한 이미지입니다.

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

• 모든 API에 대한 delay가 정상적으로 적용되고 있나요?

발견한 이슈

1️⃣ 테스트 Timeout 에러

MSW에 등록된 모든 API에 delay 적용했더니 테스트 환경에서도 동일하게 delay가 적용되어, Timeout 에러가 발생하였습니다. Timeout이 발생하지 않더라도, delay 만큼의 시간이 소요되어 CI 파이프라인의 시간을 소모할 수 있을 것 같습니다.

• 해당 문제 해결을 위해 delay를 주석 처리하고 실제로 테스트가 필요한 상황에서 주석을 풀어서 테스트하는 방법으로 결정하였습니다!

2️⃣ Amplify 에러

주석 처리하면서, 사용하지 않는 변수 때문에 Amplify 에러가 발생하였습니다.

그래서 MSW에서 제공해주는 delay의 디폴트 파라메터를 이용하는 방식으로 설정하였습니다!

• 참고: https://mswjs.io/docs/api/delay/#implicit-delay

[aws-amplify-ap-northeast-2]

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

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

[Legitgoons]

확인했습니다. 좀 더 현실적인 환경에서 개발할 수 있겠네요. 병합하셔도 좋습니다!