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

✅ test: SSE 최초 접속 테스트 코드 작성 #273

Merged
merged 3 commits into from
Dec 4, 2024

Conversation

Jo-Minseok
Copy link
Collaborator

@Jo-Minseok Jo-Minseok commented Dec 3, 2024

🔨 테스크

Issue

SSE 테스트 방식

  • SSE가 현재 서비스에서는 30초마다 돌아가도록 구현되어 있다.
  • 30초를 기다릴 경우 jest의 timeout이 발생한다. 별도의 jest timeout 셋팅을 해줘야한다.
  • 매번 CI/CD를 할 때 테스트를 할 때 시간이 너무 많이 발생한다.
  • 최초로 SSE 연결했을 때, 응답을 하게 했기에 그 응답에 대해서만 비교하도록 했다.
  • 실제 서비스처럼 주기적으로 데이터 받는 것을 검증하는 것이 가능하긴 하다. test 환경일 경우 Cron의 시간을 줄여서 처리할 수는 있겠지만, 이때, Redis에 데이터가 들어가는 타이밍이 잘 못 된다면 문제가 발생한다. 테스트를 최소 조건에 대해서만 테스트하기로 했으므로 이 부분에 대해서는 생략했다.

📋 작업 내용

  • eventSource 의존성 추가
  • SSE 최초 접속에 대한 테스트 코드 작성

📷 스크린 샷(선택 사항)

image

@Jo-Minseok Jo-Minseok added the ✅ Test 테스트 관련 (storybook, vitest, jest 등) label Dec 3, 2024
@Jo-Minseok Jo-Minseok requested a review from asn6878 December 3, 2024 09:19
@Jo-Minseok Jo-Minseok self-assigned this Dec 3, 2024
@Jo-Minseok Jo-Minseok requested a review from CodeVac513 December 3, 2024 09:19
Copy link

netlify bot commented Dec 3, 2024

Deploy Preview for denamu canceled.

Name Link
🔨 Latest commit 105274a
🔍 Latest deploy log https://app.netlify.com/sites/denamu/deploys/674ecd2da5b85100083016df

@Jo-Minseok Jo-Minseok changed the title ✅ test: sSE 최초 접속 테스트 코드 작성 ✅ test: SSE 최초 접속 테스트 코드 작성 Dec 3, 2024
Copy link
Collaborator

@CodeVac513 CodeVac513 left a comment

Choose a reason for hiding this comment

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

아까 보고 승인을 안 눌렀네요. (저는 민석님 코드보고 race라는 메서드를 처음 알았어요)
LGTM!

@Jo-Minseok Jo-Minseok merged commit 253c88f into main Dec 4, 2024
1 check passed
@Jo-Minseok Jo-Minseok deleted the test/trend-sse-api branch December 4, 2024 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅ Test 테스트 관련 (storybook, vitest, jest 등)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] 트렌딩 포스트 스케줄러 구현
2 participants