Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions fe/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,8 @@
]
},
"msw": {
"workerDirectory": "public"
}
}
"workerDirectory": [
"public"
]
}
}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

변경 사항에 대해 몇 가지 우려가 있습니다. workerDirectory 속성을 배열로 변경하였는데, 이로 인해 코드의 기능이 제대로 작동하는지 확인해야 합니다. 배열로 변경한 이유와 구조에 대한 명확한 설명이 필요하며, 테스트를 통해 기능이 정상적으로 작동하는지 확인하는 것이 좋습니다. 또한 파일 끝에 새 줄이 없는 부분은 코드 스타일 가이드에 맞지 않을 수 있으며, 추가하는 것을 추천합니다.

7 changes: 0 additions & 7 deletions fe/src/app/mocks/handlers.ts
Original file line number Diff line number Diff line change
@@ -1,10 +1,3 @@
import { testGetHandler, testPostHandler, usersHandler } from "./handler/test";

/**
* 핸들러 목록
* - `testGetHandler`: 간단한 GET 요청 핸들러
* - `testPostHandler`: 간단한 POST 요청 핸들러
* - `usersHandler`: 사용자 정보를 반환하는 핸들러
*/

export const handlers = [testGetHandler, testPostHandler, usersHandler];
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

주석이 제거되었습니다. 핸들러의 역할을 이해하는 데 도움이 되는 주석이 중요한 경우 이를 유지하는 것이 좋습니다. 코드의 가독성을 높이기 위해 주석을 남기는 것을 권장합니다. 이 주석들은 사용자나 다른 개발자가 코드를 더 쉽게 이해할 수 있도록 도와줄 것입니다. 따라서 주석을 복원하는 것을 고려해 주세요.

1 change: 0 additions & 1 deletion fe/src/app/mocks/server.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import { setupServer } from "msw/node";
import { handlers } from "./handlers";

// 서버 초기화
export const server = setupServer(...handlers);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

서버 초기화 부분이 코드에서 주석으로 설명되어 있지만, 주석 내용이 불필요하게 보입니다. 주석을 삭제하거나 더 상세한 설명으로 교체해 주세요. 예를 들어, 서버가 어떤 용도로 사용되는지나 핸들러가 무엇을 처리하는지에 대한 설명이 추가되면 좋습니다. 또한, 핸들러가 적절히 설정되어 있는지, 그리고 특정 코드 테스트 환경과의 호환성도 검토해볼 필요가 있습니다.