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

Measure test coverage #678

Closed
computerphilosopher opened this issue Aug 18, 2021 · 1 comment
Closed

Measure test coverage #678

computerphilosopher opened this issue Aug 18, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@computerphilosopher
Copy link
Member

computerphilosopher commented Aug 18, 2021

What would you like to be enhanced
: 현재 github에서 테스트 커버리지를 측정하는 자동 프로세스가 없습니다. 커버리지를 측정하고 표시하는 것이 좋겠습니다.

Why is this needed
: 테스트가 누락된 부분을 찾아 보강할 수 있게 됩니다.

Proposed solution
:

  • github action을 이용해 테스트 커버리지 측정
  • coveralls, codecov와 같은 서비스와 연동해 커버리지 기록
  • README에 커버리지 뱃지 삽입

github action을 만드는 김에 go fmt도 github action으로 자동 적용하는게 어떨까요?

@computerphilosopher computerphilosopher added the enhancement New feature or request label Aug 18, 2021
@seokho-son
Copy link
Member

@computerphilosopher 제안 감사합니다!! ^0^
그렇습니다.. CB-Tumblebug repository에는 아직 적절한 테스트 커버리지 측정 방법이 없습니다..ㅠㅠ
(테스트 코드 자체도 부족 .. )

이를 보완하기 위해서 현재 cicd 를 위한 github action 워크플로우 개발이 진행되고 있어요.. ^^
현재는 아래 정도의 내용을 커버할 것 같습니다. (coveralls, codecov와 같은 서비스와 연동은 하고 있지 않고, 데시보드 스타일의 자체 리포트 도구가 개발되고 있습니다.)

  • golint 리포트
  • API 수준의 테스트 커버리지 리포트
  • API 기반 주요 기능 테스트 자동화 (+self hosted runner)

참고로, 해당 프로젝트는 다른 임시 Repo에서 개발되고 있습니다. https://github.com/cloud-barista/poc-cicd-tumblebug/blob/main/.github/workflows/cb-ci-actions.yaml

아직은 개발 및 테스트가 완료되지 않아서, 바로 활용은 어려울 것 같습니다.

일단.. 현황은 이렇습니다.. ^^;; @computerphilosopher 께서 더 좋은 방법이 있으시면, 편하게 제안해주시면 좋을 것 같아요.

gofmt 자동 적용 action도 있으면 좋을 것 같습니다.. ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants