Merge pull request #1954 from actions/robherley/miss-msg

Cache miss as debug, not warning annotation

Author: robherley

packages/cache/__tests__/restoreCacheV2.test.ts

@@ -115,7 +115,6 @@ test('restore with restore keys and no cache found', async () => {
   const paths = ['node_modules']
   const key = 'node-test'
   const restoreKeys = ['node-']
-  const logWarningMock = jest.spyOn(core, 'warning')
 
   jest
     .spyOn(CacheServiceClientJSON.prototype, 'GetCacheEntryDownloadURL')
@@ -130,7 +129,7 @@ test('restore with restore keys and no cache found', async () => {
   const cacheKey = await restoreCache(paths, key, restoreKeys)
 
   expect(cacheKey).toBe(undefined)
-  expect(logWarningMock).toHaveBeenCalledWith(
+  expect(logDebugMock).toHaveBeenCalledWith(
     `Cache not found for keys: ${[key, ...restoreKeys].join(', ')}`
   )
 })

packages/cache/src/cache.ts

@@ -256,7 +256,7 @@ async function restoreCacheV2(
     const response = await twirpClient.GetCacheEntryDownloadURL(request)
 
     if (!response.ok) {
-      core.warning(`Cache not found for keys: ${keys.join(', ')}`)
+      core.debug(`Cache not found for keys: ${keys.join(', ')}`)
       return undefined
     }