Test with unique artifact name for v4 update

github · Sep 4, 2024 · af7457e · af7457e
1 parent 2992a95
commit af7457e
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 16 deletions.
diff --git a/lib/debug-artifacts.js b/lib/debug-artifacts.js
diff --git a/lib/debug-artifacts.js.map b/lib/debug-artifacts.js.map
diff --git a/src/debug-artifacts.ts b/src/debug-artifacts.ts
@@ -60,7 +60,7 @@ export async function uploadDebugArtifacts(
   try {
     if (ghVariant === GitHubVariant.GHES) {
       await artifactLegacy.create().uploadArtifact(
-        sanitizeArifactName(`${artifactName}${suffix}`),
+        sanitizeArifactName(`${artifactName}${suffix}}`),
         toUpload.map((file) => path.normalize(file)),
         path.normalize(rootDir),
         {
@@ -70,16 +70,20 @@ export async function uploadDebugArtifacts(
         },
       );
     } else {
+      const normalizedFilePaths = toUpload.map((file) => path.normalize(file));
       const artifactClient = new artifact.DefaultArtifactClient();
-      await artifactClient.uploadArtifact(
-        sanitizeArifactName(`${artifactName}${suffix}`),
-        toUpload.map((file) => path.normalize(file)),
-        path.normalize(rootDir),
-        {
-          // ensure we don't keep the debug artifacts around for too long since they can be large.
-          retentionDays: 7,
-        },
-      );
+
+      for (const file of normalizedFilePaths) {
+        await artifactClient.uploadArtifact(
+          sanitizeArifactName(`${artifactName}${suffix}-${file}`),
+          [file],
+          path.normalize(rootDir),
+          {
+            // ensure we don't keep the debug artifacts around for too long since they can be large.
+            retentionDays: 7,
+          },
+        );
+      }
     }
   } catch (e) {
     // A failure to upload debug artifacts should not fail the entire action.