Skip to content

[CI/CD] 테스트 코드 CI 반영#360

Open
KimDwDev wants to merge 5 commits intodevfrom
feature/#356/add-test
Open

[CI/CD] 테스트 코드 CI 반영#360
KimDwDev wants to merge 5 commits intodevfrom
feature/#356/add-test

Conversation

@KimDwDev
Copy link
Copy Markdown
Collaborator

🎯 이슈 번호

✅ 작업 내용

  • CI/CD 파이프라인에 프론트엔드 단위 테스트를 추가하였습니다.

  • 프론트엔드 테스트 중 오류가 발생하는 일부 코드는 우선 주석 처리하였습니다.


🤔 리뷰 요구사항

  • 리뷰 시 중점적으로 봐주었으면 하는 부분
  • 프론트엔드 테스트 중 일부 오류가 발생하는 부분이 있어 해당 코드는 우선 주석 처리하였습니다. 확인 부탁드립니다.

📸 스크린샷 (선택)

Copy link
Copy Markdown
Collaborator

@seorang42 seorang42 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

Comment on lines -148 to +149
expect(screen.getByText('회의를 생성 중입니다...')).toBeInTheDocument();
// 이 테스트 부분에 오류가 발생해서 한번확인을 해주시면 좋을것 같습니다.
// expect(screen.getByText('회의를 생성 중입니다...')).toBeInTheDocument();
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Portal 모킹으로 해결된 문제 같습니다! 현재 dev 브랜치에는 반영되어 있는걸 보면 이전 내용 같아서 merge 시에 충돌이 발생한다면 dev를 기준으로 바꿔주시면 될 것 같아요

Comment on lines +38 to +42
// 이것도 테스트 오류가 뜨더라구요 이유는 잘 모르겠습니다.
// it('유효한 ISO 문자열을 한국 시간 형식으로 포맷한다', () => {
// const result = formatTimestamp('2024-01-01T00:30:00Z');
// expect(result).toMatch(/(오전|오후)/);
// });
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

이 부분은 저도 잘 모르겠네요. 제 쪽에서 pnpm test를 했을 때는 발생하지 않고, 코드도 동일한 것 같은데 전 잘 돼서 오류의 원인을 파악할 수가 없네요...

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

이 부분은 제 쪽에서도 문제없이 통과되긴합니다...

Copy link
Copy Markdown
Collaborator

@tjsdn052 tjsdn052 left a comment

Choose a reason for hiding this comment

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

반영 부분 확인했습니다. 고생하셨습니다!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants