Skip to content
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

Feature/#097 MongoDB 테스트 환경 구축 #113

Merged
merged 2 commits into from
Nov 14, 2024

Conversation

minjungw00
Copy link
Collaborator

📝 변경 사항

  • 배포 및 테스트용 MongoDB 환경 설정 #97
  • Github Secrets 환경 변수 이름 변경
  • 테스트 환경에서 테스트용 MongoDB 서버를 열 수 있도록 코드 수정
  • 모든 브랜치에서 Push할 때마다 lint와 test를 수행하도록 Github Actions 변경

🔍 변경 사항 설명

  • 환경 변수 수정하는 데 있어 기타 자잘한 부분도 같이 변경했습니다.
  • 개발 환경에서 NestJS 애플리케이션을 실행할 때에는 로컬에 MongoDB가 27017 포트로 실행되고 있어야 합니다.

🙏 질문 사항

  • 리뷰어에게 부탁하고싶은 체크리스트를 추가합니다.

📷 스크린샷 (선택)

  • UI 변경이 있는 경우 스크린샷이나 GIF를 첨부합니다.

✅ 작성자 체크리스트

  • Self-review: 코드가 스스로 검토됨
  • Unit tests 추가 또는 수정
  • 로컬에서 모든 기능이 정상 작동함
  • 린터 및 포맷터로 코드 정리됨
  • 의존성 업데이트 확인
  • 문서 업데이트 또는 주석 추가 (필요 시)

@minjungw00 minjungw00 added Bug 기능 또는 UI에서 발생한 오류나 예상치 못한 동작을 해결 Build 개발 환경 세팅(eslint, 컨벤션 등) 또는 패키지 매니저 수정 labels Nov 13, 2024
@minjungw00 minjungw00 self-assigned this Nov 13, 2024
@minjungw00 minjungw00 changed the title Feature/#097_Github_Secrets_환경_변수_설정 Feature/#097 MongoDB 테스트 환경 구축 Nov 13, 2024
it("should connect to MongoDB successfully", () => {
// expect(mongoose.connection.readyState).toBe(1);
expect(true).toBe(true);
it("should connect to the MongoDB instance provided by jest-mongodb", async () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

이런식으로 몽고DB를 테스트 하는군요..
나중에는 client에서 삽입연산을 했다고 가정한 다음, mongoDB에서의 값이 어떻게 바뀌나까지 보게 된다면 테스트를 제대로 한다고 말할 수 있겠네요..!
수고하셨습니다.

Copy link
Collaborator

@hyonun321 hyonun321 left a comment

Choose a reason for hiding this comment

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

몽고DB테스트 환경이 어떻게 돌아갈지 기대가 됩니다..!

Copy link
Collaborator

@Ludovico7 Ludovico7 left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@github-actions github-actions bot merged commit 8d17632 into dev Nov 14, 2024
6 checks passed
@hyonun321 hyonun321 deleted the Feature/#097_Github_Secrets_환경_변수_설정 branch November 15, 2024 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 기능 또는 UI에서 발생한 오류나 예상치 못한 동작을 해결 Build 개발 환경 세팅(eslint, 컨벤션 등) 또는 패키지 매니저 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants