Infrastructure/astro
0
Fork 0
mirror of https://github.com/withastro/astro.git synced 2025-03-10 23:01:26 -05:00
Code Issues Activity

refactor: followRedirect -> followExpectedRedirect

Browse source
This commit is contained in:
bholmesdev 2024-08-07 15:44:56 -04:00
parent 7db24bbe47
commit 39bdff72d9
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
packages/astro/test/actions.test.js
View file

@ -180,7 +180,7 @@ describe('Astro Actions', () => {
Referer: 'http://example.com/user', Referer: 'http://example.com/user',
}, },
}); });
const res = await followRedirect(req, app); const res = await followExpectedRedirect(req, app);
assert.equal(res.ok, true); assert.equal(res.ok, true);
const html = await res.text(); const html = await res.text();
@ -196,7 +196,7 @@ describe('Astro Actions', () => {
Referer: 'http://example.com/user-or-throw', Referer: 'http://example.com/user-or-throw',
}, },
}); });
const res = await followRedirect(req, app); const res = await followExpectedRedirect(req, app);
assert.equal(res.status, 401); assert.equal(res.status, 401);
const html = await res.text(); const html = await res.text();
@ -217,7 +217,7 @@ describe('Astro Actions', () => {
Referer: 'http://example.com/user', Referer: 'http://example.com/user',
}, },
}); });
const res = await followRedirect(req, app); const res = await followExpectedRedirect(req, app);
assert.equal(res.ok, true); assert.equal(res.ok, true);
const html = await res.text(); const html = await res.text();
@ -235,7 +235,7 @@ describe('Astro Actions', () => {
Referer: 'http://example.com/user-or-throw', Referer: 'http://example.com/user-or-throw',
}, },
}); });
const res = await followRedirect(req, app); const res = await followExpectedRedirect(req, app);
assert.equal(res.status, 401); assert.equal(res.status, 401);
const html = await res.text(); const html = await res.text();
@ -346,7 +346,7 @@ const validRedirectStatuses = new Set([301, 302, 303, 304, 307, 308]);
* @param {*} app * @param {*} app
* @returns {Promise<Response>} * @returns {Promise<Response>}
*/ */
async function followRedirect(req, app) { async function followExpectedRedirect(req, app) {
const redirect = await app.render(req, { addCookieHeader: true }); const redirect = await app.render(req, { addCookieHeader: true });
assert.ok( assert.ok(
validRedirectStatuses.has(redirect.status), validRedirectStatuses.has(redirect.status),