diff --git a/dist/cache-save/index.js b/dist/cache-save/index.js index cb0db11..e00ca19 100644 --- a/dist/cache-save/index.js +++ b/dist/cache-save/index.js @@ -49203,16 +49203,13 @@ const cachePackages = () => __awaiter(void 0, void 0, void 0, function* () { const primaryKey = core.getState(constants_1.State.CachePrimaryKey); const packageManagerInfo = yield cache_utils_1.getPackageManagerInfo(packageManager); const cachePaths = yield cache_utils_1.getCacheDirectoryPath(packageManagerInfo); - let pathsCounter = cachePaths.length; - for (let path of cachePaths) { - if (!fs_1.default.existsSync(path)) { - logWarning(`Cache folder path is retrieved but doesn't exist on disk: ${path}`); - pathsCounter--; - } - } - if (!pathsCounter) { + const nonExistingPaths = cachePaths.filter(cachePath => !fs_1.default.existsSync(cachePath)); + if (nonExistingPaths.length === cachePaths.length) { throw new Error(`No cache folders exist on disk`); } + if (nonExistingPaths.length) { + logWarning(`Cache folder path is retrieved but doesn't exist on disk: ${nonExistingPaths.join(', ')}`); + } if (primaryKey === state) { core.info(`Cache hit occurred on the primary key ${primaryKey}, not saving cache.`); return; diff --git a/src/cache-save.ts b/src/cache-save.ts index 4d4f279..a3883fb 100644 --- a/src/cache-save.ts +++ b/src/cache-save.ts @@ -35,19 +35,20 @@ const cachePackages = async () => { const cachePaths = await getCacheDirectoryPath(packageManagerInfo); - let pathsCounter = cachePaths.length; + const nonExistingPaths = cachePaths.filter( + cachePath => !fs.existsSync(cachePath) + ); - for (let path of cachePaths) { - if (!fs.existsSync(path)) { - logWarning( - `Cache folder path is retrieved but doesn't exist on disk: ${path}` - ); - pathsCounter--; - } + if (nonExistingPaths.length === cachePaths.length) { + throw new Error(`No cache folders exist on disk`); } - if (!pathsCounter) { - throw new Error(`No cache folders exist on disk`); + if (nonExistingPaths.length) { + logWarning( + `Cache folder path is retrieved but doesn't exist on disk: ${nonExistingPaths.join( + ', ' + )}` + ); } if (primaryKey === state) {