From b56a44ad8a32f060159b2833a88f28fa8184da50 Mon Sep 17 00:00:00 2001
From: David Losert <davelosert@github.com>
Date: Sun, 1 Sep 2024 19:54:31 +0200
Subject: [PATCH] ci: Fix test for dependabot by using different artifact name

---
 .github/workflows/test.yml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index e651c7c..41e69ac 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -32,7 +32,8 @@ jobs:
       - name: "Upload Coverage"
         uses: actions/upload-artifact@v4
         with:
-          name: coverage-${{ matrix.branch }}
+          ## We can't use the branch name as it can contain invalid characters (e.g. dependatbot hast forwared slashes in it's branch name)
+          name: "coverage-${{ matrix.branch == 'main' && 'main' || 'test-branch' }}"
           path: coverage
 
   build-and-report:
@@ -56,7 +57,7 @@ jobs:
       - name: "Download Coverage Artifacts for ${{ github.head_ref}}"
         uses: actions/download-artifact@v4
         with:
-          name: coverage-${{ github.head_ref }}
+          name: coverage-test-branch
           path: coverage 
 
       - name: "Download Coverage Artifacts for main"