From 384c529067856e575c9a3867eaf2fe69fbbfc956 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 4 Feb 2026 12:16:44 +0000 Subject: [PATCH 1/2] Initial plan From 93c136b59c03d262291297775f8107a89fa816e0 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 4 Feb 2026 12:43:10 +0000 Subject: [PATCH 2/2] fix: ensure test results are uploaded even if tests fail The canary_go job checks test coverage by analyzing test result artifacts. When integration tests fail, the artifact upload was skipped, causing canary_go to report missing test coverage. This fix adds `if: always()` to both unit and integration test result uploads to ensure artifacts are created even when tests fail, allowing canary_go to accurately track which tests are being executed. Fixes the 137 missing tests issue in CI. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --- .github/workflows/ci.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index ca545516f7..563b902abb 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -98,6 +98,7 @@ jobs: retention-days: 7 - name: Upload unit test results + if: always() # Upload even if tests fail so canary_go can track coverage uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4 with: name: test-result-unit @@ -287,6 +288,7 @@ jobs: fi - name: Upload integration test results + if: always() # Upload even if tests fail so canary_go can track coverage uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4 with: name: test-result-integration-${{ matrix.test-group.name }}