Skip to content

Commit cd91260

Browse files
cboldisclaude
andcommitted
fix(it): remove externalId IT tests — externalId is now server-derived (LLMO-5566)
Mirrors the unit test fix: externalId is no longer accepted from the client (it is derived server-side from the site's IMS org id), so the IT tests that expected a 400 for a missing externalId are removed for both log-delivery and log-rescan. Introduced by: N/A Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 096fdb7 commit cd91260

1 file changed

Lines changed: 1 addition & 17 deletions

File tree

test/it/shared/tests/llmo-cloudfront-log-delivery.js

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -106,19 +106,9 @@ export default function llmoCloudFrontLogDeliveryTests(getHttpClient, resetData)
106106
expect(res.status).to.equal(400);
107107
});
108108

109-
it('returns 400 when externalId is missing', async () => {
110-
const http = getHttpClient();
111-
const res = await http.llmoAdmin.post(path, {
112-
accountId: VALID_ACCOUNT_ID, distributionId: VALID_DISTRIBUTION_ID,
113-
});
114-
expect(res.status).to.equal(400);
115-
});
116-
117109
it('returns 400 when distributionId is missing', async () => {
118110
const http = getHttpClient();
119-
const res = await http.llmoAdmin.post(path, {
120-
accountId: VALID_ACCOUNT_ID, externalId: VALID_EXTERNAL_ID,
121-
});
111+
const res = await http.llmoAdmin.post(path, { accountId: VALID_ACCOUNT_ID });
122112
expect(res.status).to.equal(400);
123113
});
124114

@@ -141,12 +131,6 @@ export default function llmoCloudFrontLogDeliveryTests(getHttpClient, resetData)
141131
});
142132
expect(res.status).to.equal(400);
143133
});
144-
145-
it('returns 400 when externalId is missing', async () => {
146-
const http = getHttpClient();
147-
const res = await http.llmoAdmin.post(path, { accountId: VALID_ACCOUNT_ID });
148-
expect(res.status).to.equal(400);
149-
});
150134
});
151135
});
152136
}

0 commit comments

Comments
 (0)