Add a few tests

Author: zachleat

test/QueueTest.js

@@ -1,5 +1,4 @@
 const test = require("ava");
-const fs = require("fs");
 const Cache = require("../");
 const RemoteAssetCache = require("../src/RemoteAssetCache");
 
@@ -46,16 +45,16 @@ test("Double Fetch async function (dry run)", async (t) => {
 		return Promise.resolve(expected);
 	};
 
-let ac1 = Cache(fetch, {
-	dryRun: true,
-	requestId: "fetch-1",
-});
-let ac2 = Cache(fetch, {
-	dryRun: true,
-	requestId: "fetch-2",
-});
+	let ac1 = Cache(fetch, {
+		dryRun: true,
+		requestId: "fetch-1",
+	});
+	let ac2 = Cache(fetch, {
+		dryRun: true,
+		requestId: "fetch-2",
+	});
 
-	// Make sure we only fetch once!
+	// two distinct fetches
 	t.not(ac1, ac2);
 
 	let result1 = await ac1;
@@ -65,3 +64,18 @@ let ac2 = Cache(fetch, {
 	t.deepEqual(result1, expected);
 	t.deepEqual(result2, expected);
 });
+
+test("Double Fetch 404 errors should only fetch once", async (t) => {
+	let ac1 = Cache("https://httpstat.us/404", {
+		dryRun: true,
+	});
+	let ac2 = Cache("https://httpstat.us/404", {
+		dryRun: true,
+	});
+
+	// Make sure we only fetch once!
+	t.is(ac1, ac2);
+
+	await t.throwsAsync(async () => await ac1);
+	await t.throwsAsync(async () => await ac2);
+});

test/RemoteAssetCacheTest.js

@@ -343,7 +343,7 @@ test("type: parsed-xml", async (t) => {
 	t.is(xml.children.length, 1)
 });
 
-test("raw: true, cache miss", async (t) => {
+test("returnType: response, cache miss", async (t) => {
 	let feedUrl = "https://www.11ty.dev/blog/feed.xml";
 	let ac = new RemoteAssetCache(feedUrl, ".cache", {
 		type: "xml",
@@ -365,7 +365,7 @@ test("raw: true, cache miss", async (t) => {
 });
 
 
-test("raw: true, cache hit", async (t) => {
+test("returnType: response, cache hit", async (t) => {
 	let feedUrl = "https://www.11ty.dev/blog/feed.xml";
 	let ac = new RemoteAssetCache(feedUrl, ".cache", {
 		type: "xml",
@@ -384,3 +384,4 @@ test("raw: true, cache hit", async (t) => {
 
 	t.is(response.cache, "hit");
 });
+