Explicit async

Author: zachleat

src/AssetCache.js

@@ -275,7 +275,7 @@ class AssetCache {
 		return fs.readFileSync(contentPath, type !== "buffer" ? "utf8" : null);
 	}
 
-	getCachedContents(type) {
+	async getCachedContents(type) {
 		if(!this.#rawContents[type]) {
 			this.#rawContents[type] = this.#getCachedContents(type);
 		}

src/RemoteAssetCache.js

@@ -207,7 +207,7 @@ class RemoteAssetCache extends AssetCache {
 
 	// async but not explicitly declared for promise equality checks
 	// returns a Promise
-	fetch(optionsOverride = {}) {
+	async fetch(optionsOverride = {}) {
 		if(!this.#fetchPromise) {
 			// one at a time. clear when finished
 			this.#fetchPromise = this.#fetch(optionsOverride).finally(() => {

test/QueueTest.js

@@ -202,11 +202,9 @@ test("Raw Fetch using fetch method (check parallel fetch promise reuse)", async
 	// Make sure the instance is the same
 	t.is(ac1, ac2);
 
-	let fetch1 = ac1.fetch();
-	let fetch2 = ac1.fetch();
-	t.is(fetch1, fetch2);
+	let [result1, result2] = await Promise.all([ac1.fetch(), ac1.fetch()]);
 
-	t.is(await fetch1, await fetch2);
+	t.is(result1, result2);
 
 	t.false(ac1.wasLastFetchCacheHit())