Refactor auth controller tests to reuse common behaviour

This commit is contained in:
Rory Powell 2021-07-13 10:28:15 +01:00
parent f7d91f7cd6
commit 5f5fe92807
1 changed files with 14 additions and 14 deletions

View File

@ -61,37 +61,37 @@ describe("/api/admin/auth", () => {
auth.oidc.strategyFactory = strategyFactory
const passportSpy = jest.spyOn(auth.passport, "authenticate")
let oidcConf
describe("/api/admin/auth/oidc", () => {
it("should load the oidc config and calculate the correct callback url", async () => {
const oidcConf = await config.saveOIDCConfig()
await request.get(`/api/admin/auth/oidc`)
beforeEach(async () => {
oidcConf = await config.saveOIDCConfig()
})
afterEach(() => {
expect(strategyFactory).toBeCalledWith(
oidcConf.config,
"http://127.0.0.1:4003/api/admin/auth/oidc/callback" // calculated url
)
})
describe("/api/admin/auth/oidc", () => {
it("should load strategy and delegate to passport", async () => {
await request.get(`/api/admin/auth/oidc`)
expect(passportSpy).toBeCalledWith(mockStrategyReturn, {
scope: ["profile", "email"],
})
expect(passportSpy.mock.calls.length).toBe(1);
})
})
describe("/api/admin/auth/oidc/callback", () => {
it("should do something", async () => {
const oidcConf = await config.saveOIDCConfig()
const passportSpy = jest.spyOn(auth.passport, "authenticate")
it("should load strategy and delegate to passport", async () => {
await request.get(`/api/admin/auth/oidc/callback`)
expect(passportSpy).toBeCalledWith(mockStrategyReturn, {
successRedirect: "/", failureRedirect: "/error"
}, expect.anything())
expect(passportSpy.mock.calls.length).toBe(1);
})
})