test(quickbooks): cover missing PDF content type

Author: Bill Leoutsakos

apps/sim/app/api/tools/quickbooks/documents.test.ts

@@ -116,6 +116,19 @@ describe('QuickBooks document API routes', () => {
     )
     expect(nonPdf.status).toBe(500)
 
+    mockFetch.mockResolvedValueOnce(new Response(new Uint8Array(Buffer.from('%PDF-1.4 fixture'))))
+    const missingContentType = await downloadTransactionPdf(
+      createMockRequest('POST', {
+        ...auth,
+        transactionType: 'invoice',
+        transactionId: '1',
+      })
+    )
+    expect(missingContentType.status).toBe(500)
+    await expect(missingContentType.json()).resolves.toMatchObject({
+      error: 'QuickBooks returned a non-PDF response',
+    })
+
     mockFetch.mockResolvedValueOnce(
       new Response('%PDF-', {
         headers: {

apps/sim/app/api/tools/quickbooks/download-transaction-pdf/route.ts

@@ -55,7 +55,8 @@ export const POST = withRouteHandler(async (request: NextRequest) => {
     })
     if (!response.ok) throw await getQuickBooksDocumentError(response)
 
-    const mimeType = response.headers.get('content-type')?.split(';', 1)[0].trim().toLowerCase()
+    const mimeType =
+      response.headers.get('content-type')?.split(';', 1)[0]?.trim().toLowerCase() ?? ''
     if (mimeType !== 'application/pdf') throw new Error('QuickBooks returned a non-PDF response')
     const buffer = await readResponseToBufferWithLimit(response, {
       maxBytes: MAX_FILE_SIZE,