-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
✨ Modulize mysql pool #56
Conversation
@can019
|
Mysql get connection과 destroy가 필요없어 beforeAll 과 afterAll의 mysql 관련 구문들을 지웠더니 supertest 관련 이슈를 읽어보면 express가 random port를 할당받아서 생기는 문제라고 합니다. 이슈 생성할까요? |
local에서 test 확인했습니다 |
현재 작업중인 feature/nginx에서는 위와 같은 오류가 발생하지 않고 있습니다.
|
네 |
넵 알겠습니다. test 파일에 남긴 코드리뷰 참고해주시면 감사하겠습니다 :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@201702083 :)
WorkList
Issue
Description
InitiateMysqlPool.ts type 지정 및 async / await 함수 작성에 미숙함이 있어 review 부탁합니다.
refer to https://mayajuni.github.io/2016/07/12/typescript-nodejs-mysql/