actions
/
toolkit
mirror of https://github.com/actions/toolkit
1
0
Fork 0
Code

Fix empty cache save on using or github.workspace(#833 in actions/cache) as path

pull/1132/head
Lovepreet Singh 2022-07-13 11:39:29 +00:00 committed by GitHub
parent c5278cdd08
commit 120202a68c
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
packages/cache/__tests__/cacheUtils.test.ts vendored
View File

@ -32,3 +32,8 @@ test('assertDefined throws if undefined', () => {
test('assertDefined returns value', () => { test('assertDefined returns value', () => {
expect(cacheUtils.assertDefined('test', 5)).toBe(5) expect(cacheUtils.assertDefined('test', 5)).toBe(5)
}) })
test('resolvePaths works on current directory', async () => {
const resolvedPath = await cacheUtils.resolvePaths(['.'])
expect(resolvedPath).toBe(['.'])
})

7
packages/cache/src/internal/cacheUtils.ts vendored
View File

@ -52,7 +52,12 @@ export async function resolvePaths(patterns: string[]): Promise<string[]> {
.replace(new RegExp(`\\${path.sep}`, 'g'), '/') .replace(new RegExp(`\\${path.sep}`, 'g'), '/')
core.debug(`Matched: ${relativeFile}`) core.debug(`Matched: ${relativeFile}`)
// Paths are made relative so the tar entries are all relative to the root of the workspace. // Paths are made relative so the tar entries are all relative to the root of the workspace.
paths.push(`${relativeFile}`) if (relativeFile === "") {
// path.relative returns empty string if workspace and file are equal
paths.push(".")
} else {
paths.push(`${relativeFile}`)
}
} }
return paths return paths