diff --git a/packages/workbox-precaching/src/PrecacheController.ts b/packages/workbox-precaching/src/PrecacheController.ts index bb7f97e59..5573fb0fa 100644 --- a/packages/workbox-precaching/src/PrecacheController.ts +++ b/packages/workbox-precaching/src/PrecacheController.ts @@ -264,7 +264,10 @@ class PrecacheController { printCleanupDetails(deletedURLs); } - return {deletedURLs}; + return { + deletedURLs, + deletedCacheRequests: deletedURLs, + }; }); } diff --git a/packages/workbox-precaching/src/_types.ts b/packages/workbox-precaching/src/_types.ts index 506d1bc23..aa715d2dc 100644 --- a/packages/workbox-precaching/src/_types.ts +++ b/packages/workbox-precaching/src/_types.ts @@ -14,6 +14,11 @@ export interface InstallResult { } export interface CleanupResult { + deletedURLs: string[]; + /** + * @deprecated Use `deletedURLs` instead. + * TODO: Remove in the next major release. + */ deletedCacheRequests: string[]; } @@ -51,8 +56,10 @@ export type urlManipulation = ({url}: {url: URL}) => URL[]; /** * @typedef {Object} CleanupResult - * @property {Array} deletedCacheRequests List of URLs that were deleted - * while cleaning up the cache. + * @property {Array} deletedURLs List of URLs that were deleted while + * cleaning up the cache. + * @property {Array} deletedCacheRequests Deprecated alias for + * deletedURLs. * * @memberof workbox-precaching */ diff --git a/test/workbox-precaching/sw/test-PrecacheController.mjs b/test/workbox-precaching/sw/test-PrecacheController.mjs index bd52b19c7..bb15afb2e 100644 --- a/test/workbox-precaching/sw/test-PrecacheController.mjs +++ b/test/workbox-precaching/sw/test-PrecacheController.mjs @@ -834,9 +834,13 @@ describe(`PrecacheController`, function () { activateEvent, ); expect(cleanupDetailsTwo.deletedURLs.length).to.equal(1); + expect(cleanupDetailsTwo.deletedCacheRequests.length).to.equal(1); expect(cleanupDetailsTwo.deletedURLs[0]).to.eql( `${location.origin}/scripts/index.js?__WB_REVISION__=1234`, ); + expect(cleanupDetailsTwo.deletedCacheRequests[0]).to.eql( + `${location.origin}/scripts/index.js?__WB_REVISION__=1234`, + ); const keysTwo = await cache.keys(); expect(keysTwo.length).to.equal(cacheListTwo.length); @@ -894,6 +898,10 @@ describe(`PrecacheController`, function () { `${location.origin}/index.1234.html`, `${location.origin}/scripts/stress.js?test=search&foo=bar&__WB_REVISION__=1234`, ]); + expect(cleanupDetailsTwo.deletedCacheRequests).to.have.members([ + `${location.origin}/index.1234.html`, + `${location.origin}/scripts/stress.js?test=search&foo=bar&__WB_REVISION__=1234`, + ]); const keysTwo = await cache.keys(); expect(keysTwo.length).to.equal(cacheListTwo.length);