Use info logic/#82

[seonghunYang]

📌 작업 내용

구현 내용 및 작업 했던 내역

• 로그아웃 및 회원탈퇴 로직, 그리고 컴포넌트 구현
• mock에서 인증 관련 로직 수정
• 미들웨어 라우팅 로직 수정

🤔 고민 했던 부분

• 인증 로직으로 테스트가 어려워지는 문제를 해결하기 위해, 개발 모드일 때만 인증 처리를 하는 devModeAuthGuard() 함수를 구현했습니다.
• 또 다른 해결책은 next/header의 쿠키를 모의하는 것입니다. 이에 대해 user-info-navigator 컴포넌트에서 작성했습니다.
• 어느 정도까지 mock을 유지하고 관리해야 하는지에 대한 고민은 항상 있는 것 같습니다. 개발의 편의성을 위해 AuthGuard()를 제거할 수도 있지만, 나중에 동작을 정확하게 모의해야 할 필요가 있을 때 사용하시면 됩니다.

🔊 도움이 필요한 부분

• user.command 파일이 너무 크네요. 로직별로 파일을 분리하는 것을 고려하고 있는데. 여러분의 의견을 듣고 싶어요.

[github-actions]

🚀storybook: https://65ccb85d5afe55a024495bc0-wuvqamumwh.chromatic.com/

[github-actions]

🚀storybook: https://65ccb85d5afe55a024495bc0-sbzfdrbkgj.chromatic.com/

[yougyung]

수고하셨습니다 !
인증 관련 테스트의 진행에 있어 수고해주신 덕분에 devModeAuthGuard()을 유용하게 사용할 수 있을 것 같습니다.

user.command 파일이 너무 크네요. 로직별로 파일을 분리하는 것을 고려하고 있는데. 여러분의 의견을 듣고 싶어요.

가현이의 auth와 user를 분리하자는 의견이 좋아보입니다.

[yougyung]

개발 모드가 아닐때는

 return {
    authId: '',
  };

을 반환해야하는 것 아닌가요?

[seonghunYang]

• 먼저, 기존의 mock 로직을 활용하려면 authId가 필요합니다. 따라서 개발 모드가 아닌 테스트 상황에서는 admin 계정으로 진행한다는 가정 하에 이를 구성하였습니다.

[seonghunYang]

• 쿠키 삭제는 서버 액션 또는 라우트 핸들러에서만 수행할 수 있고
• 서버 액션은 비동기 함수여야 합니다.
  https://nextjs.org/docs/app/api-reference/functions/cookies#deleting-cookies

[seonghunYang]

• 좋은 질문이네요. 이벤트 핸들러에 대해, 저는 반드시 wrapper 함수를 생성하여 정의하는 방식을 이용합니다. 이를 위한 이유는 두 가지입니다.
• 가독성: 컴포넌트를 살펴볼 때 어떤 이벤트 핸들러와 로직이 있는지 한눈에 볼 수 있습니다. 또한, 컴포넌트와 로직을 분리하는 장점도 있습니다(큽니다).
• 확장성: 이벤트 핸들러에서는 비즈니스 로직을 작성하는 경우가 많습니다. 처음에는 내용이 작아서 인라인 할 수 있지만, 내용이 쉽게 확장될 수 있기 때문에 별도로 분리합니다.
• 쓰고보니 확장 가능성은 그리 좋은 이유로 보이진 않고 가독성이 더 중요한 이유로 보입니다.

[seonghunYang]

나머지 사항들은 수정이 필요하므로, 수정한 후에 업로드 하겠습니다.

user.command 파일은 좋은 의견을 주신 덕분에, 로직 단위로 분리하기 전에 먼저 auth와 user를 분리하도록 하겠습니다.