FE/bugfix/#432 채팅방개설 취소시 튕기는버그

[Doosies]

변경 사항

• 해당 이슈 해결
• vi undefined 오류 해결

고민과 해결 과정

이슈 해결

원인은 최소 버튼을 누를시 window.close() 메서드가 호출되기 때문이었음.
즉, 함수의 파라미터로 close를 받았지만 정작 함수 안에서 구조분해 할당할 때 이를 사용하지 않아서 생기는 문제...

function Component({value}: {close: () => void; value: string}) {
  const handler = () => {
    // ⚠️ window.close() 호출됨!
    close();
  };
  return ...
}

그래서 close라고 적혀있는 메소드명을 전부 closeXXX open이라고 적혀있는 메소드들은 openXXX라고 바꿔줌
나중에 close 키워드를 함수 이름으로 써야할 때 주의해야함

그리고 window.close는 원래 window.open으로 열린 페이지에 대해서만 작동한다고함
그래서 로컬 환경에서는 작동하지 않았는데 왜 배포하면 이 메서드가 작동하는지는 아직 찾아보는중

vi undefined 해결

아래처럼 mocks 폴더 안에 있는 가짜모듈을 테스트 코드가 아닌 실제 코드에서 import해서 사용하고 있어서 생겼던 에러

import WebRTC from '@business/services/__mocks__/WebRTC';

그래서 tsconfig.json에서 exclude 옵션으로 __mocks__ 폴더를 제외 해주려고 했는데
그러면 테스트 코드를 작성 할 때도 vi를 찾지 못하는 문제가 생겨버림 어떻게 해야할지 생각 해 봐야할듯

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Pages

Latest commit:	6fe1433
Status:	✅  Deploy successful!
Preview URL:	https://c15d2085.web09-magicconch.pages.dev
Branch Preview URL:	https://fe-bugfix--428.web09-magicconch.pages.dev

View logs

[kimyu0218]

💬 __mocks__를 제외하면 에러가 나는 구나..!!

BE도 __mocks__를 제외하는데 jest를 위한 config 파일이 있어서 에러가 안 나는 것 같아. jest 설정을 위한 json 파일을 작성하고, --config로 해당 파일을 지정하고 있어 (nest 기본 설정값!!)

"test:e2e": "jest --config ./test/jest-e2e.json --runInBand --verbose"

[Doosies]

💬 __mocks__를 제외하면 에러가 나는 구나..!!

BE도 __mocks__를 제외하는데 jest를 위한 config 파일이 있어서 에러가 안 나는 것 같아. jest 설정을 위한 json 파일을 작성하고, --config로 해당 파일을 지정하고 있어 (nest 기본 설정값!!)

"test:e2e": "jest --config ./test/jest-e2e.json --runInBand --verbose"

글을 조금 잘못 썼던것같아 ㅋㅋㅋㅋㅋ
__mocks__는 test환경에서만 mock 메소드에 의해 불러와야 하는데
build시에 실행되는 환경에서 mocks 폴더에서 가짜 모듈을 불러와버려서 생겼던 오류였어
실수로 저깄는걸 import 해버려서 import할 때 자동완성 목록에서 지워주고 싶은데 아직 방법을 못찾았어 😢