From 759220d5928af4a01c205a039eb9826456a475cd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sebasti=C3=A1n=20Ram=C3=ADrez?= <tiangolo@gmail.com>
Date: Fri, 21 Feb 2025 12:59:48 +0100
Subject: [PATCH] =?UTF-8?q?=20=F0=9F=91=B7=20Add=20Codecov=20to=20CI,=20Sm?=
 =?UTF-8?q?okeshow/Cloudflare=20has=20been=20flaky=20lately=20(#1303)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
---
 .github/workflows/test.yml | 10 ++++++++++
 scripts/test.sh            |  1 +
 2 files changed, 11 insertions(+)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 9126db9..c924e54 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -74,13 +74,23 @@ jobs:
         env:
           COVERAGE_FILE: coverage/.coverage.${{ runner.os }}-py${{ matrix.python-version }}-${{ matrix.pydantic-version }}
           CONTEXT: ${{ runner.os }}-py${{ matrix.python-version }}
+      # TODO: if removing Smokeshow, and moving only to Codecov, remove this
+      # Upload files before running Codecov, as it generates an extra file coverage/coverage.xml, and that breaks coverage-combine
       - name: Store coverage files
         uses: actions/upload-artifact@v4
         with:
           name: coverage-${{ matrix.python-version }}-${{ matrix.pydantic-version }}
           path: coverage
           include-hidden-files: true
+      - uses: codecov/codecov-action@v5
+        with:
+          fail_ci_if_error: true
+          files: ./coverage.xml
+          name: codecov-umbrella
+          token: ${{ secrets.CODECOV_TOKEN }}
+          verbose: true
 
+  # TODO: if removing Smokeshow, and moving only to Codecov, remove this
   coverage-combine:
     needs:
       - test
diff --git a/scripts/test.sh b/scripts/test.sh
index ff4b114..826ae90 100755
--- a/scripts/test.sh
+++ b/scripts/test.sh
@@ -7,3 +7,4 @@ coverage run -m pytest tests
 coverage combine
 coverage report
 coverage html
+coverage xml