adam/shoc-frontend-new
1
0
Fork 0
mirror of https://github.com/Sea-Haven-Industries/shoc-frontend-new.git synced 2026-10-07 16:19:01 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(api): unblock FormData Content-Type test typecheck

Browse source 
Avoid CFA null narrowing on the fetch-mock capture so tsc -b passes on this tip.
This commit is contained in:
Arthur Bassi 2026-07-22 13:39:51 -03:00
parent fada679b5c
commit e2687672e2
1 changed files with 6 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/test/api/api-formdata-content-type.test.ts
View file

@ -47,13 +47,12 @@ describe("api.post FormData Content-Type", () => {
await api.post("workorders/9/media", { body: form, throwHttpErrors: false });
expect(capturedContentType).not.toBe("application/json");
// Browser sets multipart/form-data; boundary=... when Content-Type was stripped.
// In jsdom/fetch mock the runtime may leave it null or set multipart — both are OK
// as long as it is not forced JSON.
const contentType: string | null = capturedContentType;
if (contentType !== null) {
expect(contentType.toLowerCase()).toContain("multipart/form-data");
// Fetch mock assigns inside a nested callback; CFA may keep the outer binding as null.
// Contract under test: do not force JSON Content-Type for FormData bodies.
const observedContentType = capturedContentType as string | null;
expect(observedContentType).not.toBe("application/json");
if (observedContentType) {
expect(observedContentType.toLowerCase()).toContain("multipart/form-data");
}
});
});