fix: normalize artifact paths in makeResult dryRunResume (#3206)

electron · Apr 4, 2023 · dcd5476 · dcd5476
1 parent 0f73d64
commit dcd5476
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/packages/api/core/src/api/publish.ts b/packages/api/core/src/api/publish.ts
@@ -217,13 +217,16 @@ const publish = async ({
                     d('restoring publish settings from dry run');
 
                     for (const makeResult of restoredMakeResults) {
-                      for (const makePath of makeResult.artifacts) {
-                        // standardize the path to artifacts across platforms
-                        const normalizedPath = makePath.split(/\/|\\/).join(path.sep);
-                        if (!(await fs.pathExists(normalizedPath))) {
-                          throw new Error(`Attempted to resume a dry run but an artifact (${normalizedPath}) could not be found`);
-                        }
-                      }
+                      makeResult.artifacts = await Promise.all(
+                        makeResult.artifacts.map(async (makePath: string) => {
+                          // standardize the path to artifacts across platforms
+                          const normalizedPath = makePath.split(/\/|\\/).join(path.sep);
+                          if (!(await fs.pathExists(normalizedPath))) {
+                            throw new Error(`Attempted to resume a dry run, but an artifact (${normalizedPath}) could not be found`);
+                          }
+                          return normalizedPath;
+                        })
+                      );
                     }
 
                     d('publishing for given state set');