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

외부 서비스에 영향을 받는 테스트 비활성화 #17

Closed
2 tasks done
Wo-ogie opened this issue Jul 27, 2023 · 0 comments · Fixed by #18
Closed
2 tasks done

외부 서비스에 영향을 받는 테스트 비활성화 #17

Wo-ogie opened this issue Jul 27, 2023 · 0 comments · Fixed by #18
Assignees
Labels
🗃 infra 인프라, 배포 ✅ test 테스트 관련

Comments

@Wo-ogie
Copy link
Member

Wo-ogie commented Jul 27, 2023

🔥 Issue

KakaoPlaceScrapingController7e2ab3d 에서 "실제 가게 정보는 수시로 바뀔 수 있으므로 기능의 정상적인 동작을 확인하기에 불안정하다. 따라서 평소에는 disabled 해놓고 필요할 때 테스트하는 방식으로 사용한다"는 이유로 비활성화 되었다.

해당 테스트를 활성화/비활성화하고 CI에 실제 스크래핑 로직을 검증할 것인지에 대해서는 다음과 같은 고려할 점들이 있다.

  • 실제 가게 정보가 변경되어 테스트에 영향이 갈 수 있다. 이는 해당 테스트의 목적에 맞지 않는다.
  • 데이터를 스크래핑하는 웹 페이지의 코드(HTML structure)가 변할 수 있다. 이는 검증 및 모니터링이 필요한 사항이나, CI 단계에서 확인할 내용은 아니라고 생각된다.
  • Chrome version이 update 됨에 따라 CI가 실패할 수 있다.

이러한 사항들을 고민했을 때 테스트는 비활성화하고 CI 단계에서 실제 스크래핑 로직을 검증하는 단계는 빼는 것이 맞다는 판단을 했다. 하지만, 실제로 스크래핑 로직이 원하는 결과대로 동작함을 보장하는 것은 안정적인 서비스 제공을 위해 중요하다. 그 때문에 주기적으로 스크래핑 기능을 테스트하는 로직을 수행하는 배치 프로그램을 개발하는 것으로 기능의 안정성을 검증하기로 한다.

이 issue에서는 우선 외부 서비스에 영향을 받는 테스트를 비활성화하고, CI 단계에서 관련된 내용을 제거하고자 한다.

✅ TODO

  • 외부 서비스에 영향을 받는 테스트 비활성화
  • gradle.yml에서 chromedriver를 세팅하는 단계 제거
@Wo-ogie Wo-ogie added 🗃 infra 인프라, 배포 ✅ test 테스트 관련 labels Jul 27, 2023
@Wo-ogie Wo-ogie self-assigned this Jul 27, 2023
@Wo-ogie Wo-ogie changed the title CI - Chromedriver를 세팅하는 단계 제거 외부 서비스에 영향을 받는 테스트 비활성화 Jul 27, 2023
@github-project-automation github-project-automation bot moved this from Todo to Done in Eatery - Backend Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🗃 infra 인프라, 배포 ✅ test 테스트 관련
Projects
Archived in project
1 participant