test(quickbooks): cover attachment MIME fallback

Author: Bill Leoutsakos

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

@@ -346,6 +346,28 @@ describe('QuickBooks document API routes', () => {
     )
   })
 
+  it('uses the binary fallback MIME type when an attachment response omits Content-Type', async () => {
+    mockFetch.mockResolvedValueOnce(
+      new Response('https://intuit-download.example/attachment.bin', {
+        headers: { 'content-type': 'text/plain' },
+      })
+    )
+    mockSecureFetchWithPinnedIP.mockResolvedValueOnce(new Response(new Uint8Array([1, 2, 3])))
+
+    const response = await downloadAttachment(
+      createMockRequest('POST', { ...auth, attachmentId: '16' })
+    )
+    const body = await response.json()
+
+    expect(response.status).toBe(200)
+    expect(body.output).toMatchObject({
+      attachmentId: '16',
+      fileName: 'attachment.bin',
+      mimeType: 'application/octet-stream',
+      size: 3,
+    })
+  })
+
   it('rejects note-only and oversized attachment downloads', async () => {
     mockFetch.mockResolvedValueOnce(new Response(''))
     const note = await downloadAttachment(

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

@@ -104,7 +104,7 @@ export const POST = withRouteHandler(async (request: NextRequest) => {
     })
     if (buffer.length === 0) throw new Error('QuickBooks attachment file is empty')
     const mimeType =
-      downloadResponse.headers.get('content-type')?.split(';', 1)[0].trim().toLowerCase() ||
+      downloadResponse.headers.get('content-type')?.split(';', 1)[0]?.trim().toLowerCase() ||
       'application/octet-stream'
     let fallbackName = `quickbooks-attachment-${attachmentId}`
     try {