fix tests

Author: talentlessguy

src/index.ts

@@ -12,7 +12,7 @@ export type ReqWithBody<T = any> = IncomingMessage & {
 
 export const hasBody = (method: string) => ['POST', 'PUT', 'PATCH', 'DELETE'].includes(method)
 
-const defaultPayloadLimit = 104857600 // 100KB
+const defaultPayloadLimit = 102400 // 100KB
 
 export type LimitErrorFn = (limit: number) => Error
 
@@ -132,10 +132,16 @@ const getBoundary = (contentType: string) => {
 
 const defaultFileSizeLimitErrorFn: LimitErrorFn = (limit) => new Error(`File too large. Limit: ${limit} bytes`)
 
+const defaultFileSizeLimit = 200 * 1024 * 1024
+
 const parseMultipart = (
   body: string,
   boundary: string,
-  { fileCountLimit, fileSizeLimit, fileSizeLimitErrorFn = defaultFileSizeLimitErrorFn }: MultipartOptions
+  {
+    fileCountLimit,
+    fileSizeLimit = defaultFileSizeLimit,
+    fileSizeLimitErrorFn = defaultFileSizeLimitErrorFn
+  }: MultipartOptions
 ) => {
   const parts = body.split(new RegExp(`${boundary}(--)?`)).filter((part) => !!part && /content-disposition/i.test(part))
   const parsedBody: Record<string, (File | string)[]> = {}
@@ -147,7 +153,7 @@ const parseMultipart = (
     const [headers, ...lines] = part.split('\r\n').filter((part) => !!part)
     const data = lines.join('\r\n').trim()
 
-    if (fileSizeLimit && data.length > fileSizeLimit) throw fileSizeLimitErrorFn(fileSizeLimit)
+    if (data.length > fileSizeLimit) throw fileSizeLimitErrorFn(fileSizeLimit)
 
     // Extract the name and filename from the headers
     const name = /name="(.+?)"/.exec(headers)![1]
@@ -184,7 +190,7 @@ type MultipartOptions = Partial<{
 /**
  * Parse multipart form data (supports files as well)
  *
- * Does not restrict total payload size by default
+ * Does not restrict total payload size by default.
  * @param options
  */
 const multipart =
@@ -200,7 +206,6 @@ const multipart =
           payloadLimit,
           payloadLimitErrorFn
         )(req, res, next)
-        next()
       } else next()
     }
 

test.ts

@@ -431,7 +431,7 @@ describe('Limits', () => {
         Accept: 'text/plain',
         'Content-Type': 'text/plain'
       }
-    }).expect(413, 'Payload too large. Limit: 104857600 bytes')
+    }).expect(413, 'Payload too large. Limit: 102400 bytes')
   })
 
   it('should throw on custom payloadLimit', async () => {
@@ -513,21 +513,21 @@ describe('Limits', () => {
   it('should throw multipart if exceeds allowed file size with a custom error', async () => {
     const server = createServer(async (req: ReqWithBody, res) => {
       await multipart({
-        fileSizeLimit: 10,
+        fileSizeLimit: 20,
         fileSizeLimitErrorFn: (limit) => new Error(`File too large. Limit: ${limit / 1024}KB`)
       })(req, res, (err) => {
         if (err) res.writeHead(413).end(err.message)
-        else res.end(req.body)
+        else res.end('ok')
       })
     })
 
     const fd = new FormData()
 
-    fd.set('file', new File(['hello world'], 'hello.txt', { type: 'text/plain' }))
+    fd.set('file', new File(['hello world to everyone'], 'hello.txt', { type: 'text/plain' }))
 
     await makeFetch(server)('/', {
       body: fd,
       method: 'POST'
-    }).expect(413, 'File too large. Limit: 0.009765625KB')
+    }).expect(413, 'File too large. Limit: 0.01953125KB')
   })
 })