Fascinated/node-fetch-cache
Archived
1
0
Fork 0
Code Issues Pull Requests 1 Actions Activity

dont error if ejectFromCache is called when the response is already not cached

Browse Source
This commit is contained in:
Randall Schmidt 2020-11-27 23:31:39 -05:00
parent 558d63fd40
commit 8e1b57382a
2 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
classes/response.js
View File

@ -25,8 +25,14 @@ class Response {
return this.bodyBuffer; return this.bodyBuffer;
} }
ejectFromCache() { async ejectFromCache() {
return fs.promises.unlink(this.cacheFilePath); try {
await fs.promises.unlink(this.cacheFilePath);
} catch (err) {
if (err.code !== 'ENOENT') {
throw err;
}
}
} }
} }

8
test/tests.js
View File

@ -96,6 +96,14 @@ describe('Cache tests', function() {
res = await fetch(TWO_HUNDRED_URL); res = await fetch(TWO_HUNDRED_URL);
assert.strictEqual(res.fromCache, true); assert.strictEqual(res.fromCache, true);
}); });
it('Does not error if rejecting from cache twice', async function() {
res = await fetch(TWO_HUNDRED_URL);
assert.strictEqual(res.fromCache, false);
await res.ejectFromCache();
await res.ejectFromCache();
});
}).timeout(10000); }).timeout(10000);
describe('Data tests', function() { describe('Data tests', function() {