Skip to content

Commit 25bb930

Browse files
priyanshu92Copilot
andcommitted
Separate integration header contract tests
Keep default omission and explicit inclusion independently visible in test output. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent b8bac6b commit 25bb930

1 file changed

Lines changed: 17 additions & 6 deletions

File tree

plugins/power-pages/scripts/tests/integration/discover-integration.test.js

Lines changed: 17 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,24 +28,35 @@ function makeSiteRoot(entityLogicalNames) {
2828
return root;
2929
}
3030

31-
test('integration transport mirrors makeRequest includeHeaders behavior', async () => {
32-
const mock = await startMock([
31+
function startHeaderMock() {
32+
return startMock([
3333
{
3434
method: 'GET',
3535
matcher: '/headers',
3636
headers: { 'x-test-header': 'present' },
3737
body: { ok: true },
3838
},
3939
]);
40+
}
41+
42+
test('integration transport omits response headers by default', async () => {
43+
const mock = await startHeaderMock();
4044
try {
41-
const withoutHeaders = await makeLocalRequest({ url: `${mock.baseUrl}/headers` });
42-
assert.equal(Object.hasOwn(withoutHeaders, 'headers'), false);
45+
const result = await makeLocalRequest({ url: `${mock.baseUrl}/headers` });
46+
assert.equal(Object.hasOwn(result, 'headers'), false);
47+
} finally {
48+
await mock.close();
49+
}
50+
});
4351

44-
const withHeaders = await makeLocalRequest({
52+
test('integration transport includes response headers when requested', async () => {
53+
const mock = await startHeaderMock();
54+
try {
55+
const result = await makeLocalRequest({
4556
url: `${mock.baseUrl}/headers`,
4657
includeHeaders: true,
4758
});
48-
assert.equal(withHeaders.headers['x-test-header'], 'present');
59+
assert.equal(result.headers['x-test-header'], 'present');
4960
} finally {
5061
await mock.close();
5162
}

0 commit comments

Comments
 (0)