Skip to content

Commit 685bb36

Browse files
committed
Fix strict equality checks and restore missing test in fedify/fresh test
1 parent 75437d3 commit 685bb36

1 file changed

Lines changed: 9 additions & 8 deletions

File tree

packages/fresh/src/mod.test.ts

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,8 @@ Deno.test("integrateFetchOptions() - onNotFound delegates to ctx.next with corre
2929
const options = integrateFetchOptions(mockCtx);
3030
const request = new Request("https://example.com/some-page");
3131

32-
const response = await options.onNotFound!(request);
32+
assertExists(options.onNotFound);
33+
const response = await options.onNotFound(request);
3334

3435
assertStrictEquals(capturedThis, mockCtx);
3536
assertStrictEquals(passedRequest, request);
@@ -45,8 +46,8 @@ Deno.test("onNotAcceptable() returns Fresh response when not 404", async () => {
4546
assertExists(options.onNotAcceptable);
4647
const response = await options.onNotAcceptable(ctx.req);
4748

48-
assertEquals(response.status, 200);
49-
assertEquals(await response.text(), "ok");
49+
assertStrictEquals(response.status, 200);
50+
assertStrictEquals(await response.text(), "ok");
5051
});
5152

5253
Deno.test("onNotAcceptable() returns 406 when Fresh returns 404", async () => {
@@ -58,9 +59,9 @@ Deno.test("onNotAcceptable() returns 406 when Fresh returns 404", async () => {
5859
assertExists(options.onNotAcceptable);
5960
const response = await options.onNotAcceptable(ctx.req);
6061

61-
assertEquals(response.status, 406);
62-
assertEquals(response.headers.get("Vary"), "Accept");
63-
assertEquals(response.headers.get("Content-Type"), "text/plain");
62+
assertStrictEquals(response.status, 406);
63+
assertStrictEquals(response.headers.get("Vary"), "Accept");
64+
assertStrictEquals(response.headers.get("Content-Type"), "text/plain");
6465
});
6566

6667
Deno.test("integrateHandler() calls createContextData and passes result to federation.fetch()", async () => {
@@ -85,7 +86,7 @@ Deno.test("integrateHandler() calls createContextData and passes result to feder
8586
const handler = integrateHandler(mockFederation, createContextData);
8687
const response = await handler(ctx);
8788

88-
assertEquals(receivedContextData, expectedContextData);
89+
assertStrictEquals(receivedContextData, expectedContextData);
8990
assertStrictEquals(receivedRequest, ctx.req);
9091
assertEquals(await response.text(), "ok");
9192
});
@@ -107,5 +108,5 @@ Deno.test("integrateHandler() supports an async createContextData factory", asyn
107108
const handler = integrateHandler(mockFederation, createContextData);
108109
await handler(ctx);
109110

110-
assertEquals(receivedContextData, expectedContextData);
111+
assertStrictEquals(receivedContextData, expectedContextData);
111112
});

0 commit comments

Comments
 (0)