테스트 커버리지 수집을 위한 CodeCov 셋업

[DaleSeo]

[hyoseong1994]

@DaleSeo 안녕하세요 달레님
이 티켓의 목적을 확인하기위해서 댓글 추가했습니다.
제가 알아본 바로는 codecov 는 테스트 커버리지 결과를 수집해서, 비교하고, 시각화해주는 SaaS 서비스로 이해했습니다.

1. test coverage 생성
   • test coverage 생성 script 추가
   • vite.config.ts 설정 (codecov 에 올라가는 형태 / lcov.info)
2. github workflow 수정(integration.yml)
   • test 실행시 coverage 생성
   • codecov에 coverage upload
3. codecov 토큰 설정

이 정도로 예상되는데 맞을까요?
codecov 토큰은 달레님이 생성해주시는걸까요?

생각하신 방향이 다르다면 편히 말씀해주세요

[DaleSeo]

@hyoseong1994 네, 맞게 이해하시고 계십니다. 적당한 Theshold 제안 및 설정도 같이 해주시면 좋을 것 같습니다. 우선은 프로젝트에 관련 npm script 추가하시고, CI를 통해 Test Coverage이 자동으로 수집되도록 작업을 진행해주세요. 저는 CodeCov 토큰 생성해서 저장소에 설정 후에 효성님께 공유드리도록 하겠습니다.

[hyoseong1994]

현재 main에서 분기하여 확인한 결과 아래 이미지와 같은 테스트 커버리지가 나왔습니다.

리서치를 진행한 결과 일반적으로 coverage threshold를 약 80% 수준으로 설정하는 경우가 많으며, 회사마다 정책에 따라 신규 기능에 대해서만 85~90% 이상을 요구하는 방식도 있습니다.

저희는 이미 높은 수준의 커버리지를 유지하고 있기 때문에, 아래와 같은 threshold 기준을 유지하는 방향으로 제안드립니다.

     thresholds: {
        statements: 90,
        branches: 85,
        functions: 95,
        lines: 90,
      },

[hyoseong1994]

안녕하세요 달레님
전에 말씀드린 codecov 토큰 설정 하지 않는 방법도 있어서 공유드립니다.

https://docs.codecov.com/docs/codecov-tokens#uploading-without-a-token

[DaleSeo]

@hyoseong1994 토큰 없이 커버리지 업로드가 가능하도록 설정하였습니다. (편의를 위해서는 좋겠으나, 혹시 외부에서 저희인 척하고 업로드 할 수도 있어서 좀 꺼림직하긴 하네요. 그럴 가능성은 매우 희박하겠지만요 ㅎㅎ)

커버리지는 현재 이미 훌륭하네요! 👍 제안해주신 threshold로 시작하고, 앞으로 진행하면서 전 항목 90%에 도전해보시죠 ㅎㅎ Vitest 수준에서 그렇게 설정할 수도 있지만, CodeCov 수준에서도 설정을 해줘야 PR에서 Check으로 달릴 것입니다. 관련 문서 참고하세요: https://docs.codecov.com/docs/commit-status

[hyoseong1994]

@DaleSeo 혼란을 드려 죄송합니다. public이면 되는지 알았는데 보호된 브랜치일 경우도 token이 필요하네요
토큰을 설정하는 방향으로 가는게 좋을거 같은데 어떻게 생각하시나요?

error - 2025-12-12 07:09:21,698 -- Upload failed: {"message":"Token required because branch is protected"}

https://github.com/DaleStudy/daleui/actions/runs/20159184453/job/57867749878?pr=660

[DaleSeo]

@hyoseong1994 역시, 뭔가 이상하더라고요 ㅋㅋ 어떻게 생각하고 말 것도 없는 것 같은데요? 토큰 디스코드로 보내드렸습니다.

[hyoseong1994]

@hyoseong1994 역시, 뭔가 이상하더라고요 ㅋㅋ 어떻게 생각하고 말 것도 없는 것 같은데요? 토큰 디스코드로 보내드렸습니다.
빠른 피드백 감사합니다. 작업해서 PR open 하도록하겠습니다.