jayvenn21 commented on code in PR #524:
URL: https://github.com/apache/airavata-custos/pull/524#discussion_r3591414788
##########
web/tests/fixtures/auth.ts:
##########
@@ -79,6 +78,7 @@ export async function signInAs(page: Page, persona: Persona =
"admin") {
throw new Error("NEXTAUTH_SECRET must be set for the cookie-injection
fixture");
}
+ const { encode } = await import("next-auth/jwt");
Review Comment:
i changed the import because the JWT encoder is only needed inside the test
helper when we mint the session cookie. also, there wasn’t a runtime behavior
reason beyond keeping it scoped to the helper call. i can switch it back to a
normal static import if preferred
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
