julien/remanso-jetstream
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(test): match getSession URL by suffix, not /getSession path
All checks were successful
CI / check (push) Successful in 13s
Details

Browse Source 
The PDS endpoint URL is /xrpc/com.atproto.server.getSession —
those are dots before getSession, not slashes, so the previous
includes("/getSession") filter never matched and the Authorization
header assertion silently short-circuited via optional chaining.
This commit is contained in:
Julien Calixte
2026-06-07 22:31:26 +02:00
parent 2105d75cb5
commit 7623fff227
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/auth_verify_test.ts
View File

@@ -52,7 +52,7 @@ Deno.test(
`Bearer ${makeJwt({ sub: did })}`, `Bearer ${makeJwt({ sub: did })}`,
); );
assertEquals(result, did); assertEquals(result, did);
const sessionCall = stub.calls.find((c) => c.url.includes("/getSession")); const sessionCall = stub.calls.find((c) => c.url.endsWith("getSession"));
assertEquals( assertEquals(
sessionCall?.headers.get("Authorization")?.startsWith("Bearer "), sessionCall?.headers.get("Authorization")?.startsWith("Bearer "),
true, true,