Skip to content

Commit 2291709

Browse files
v1rtlclaude
andcommitted
test: add medium difficulty coverage tests
- Add test for enableCaching with maxAge only (no immutable, non-zero) - Add test for sendFile callback on successful stream completion Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 6a1b717 commit 2291709

1 file changed

Lines changed: 21 additions & 0 deletions

File tree

tests/modules/send.test.ts

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -270,4 +270,25 @@ describe('sendFile(path)', () => {
270270
const app = runServer((req, res) => sendFile(req, res)(testFilePath, { caching: { maxAge: 0 } }))
271271
await makeFetch(app)('/').expectHeader('Cache-Control', 'public,max-age=0,must-revalidate')
272272
})
273+
it('should set cache header with just maxAge when not immutable and not 0', async () => {
274+
const app = runServer((req, res) => sendFile(req, res)(testFilePath, { caching: { maxAge: 3600 } }))
275+
await makeFetch(app)('/').expectHeader('Cache-Control', 'public,max-age=3600')
276+
})
277+
it('should call callback on successful stream completion', async () => {
278+
let callbackCalled = false
279+
let callbackError: Error | undefined
280+
281+
const app = runServer((req, res) => {
282+
sendFile(req, res)(testFilePath, {}, (err) => {
283+
callbackCalled = true
284+
callbackError = err
285+
})
286+
})
287+
288+
await makeFetch(app)('/').expect('Hello World')
289+
// Give stream time to complete and callback to fire
290+
await new Promise((resolve) => setTimeout(resolve, 50))
291+
expect(callbackCalled).toBe(true)
292+
expect(callbackError).toBeUndefined()
293+
})
273294
})

0 commit comments

Comments
 (0)