update for Fastify v5 (#505)

Author: gurgunday

.github/workflows/ci.yml

@@ -17,7 +17,7 @@ on:
 
 jobs:
   test:
-    uses: fastify/workflows/.github/workflows/plugins-ci.yml@v3
+    uses: fastify/workflows/.github/workflows/plugins-ci.yml@v4.0.0
     with:
       license-check: true
       lint: true

.gitignore

@@ -23,6 +23,9 @@ lib-cov
 coverage
 *.lcov
 
+# .tap output
+.tap
+
 # nyc test coverage
 .nyc_output
 

index.js

@@ -7,12 +7,11 @@ const { createWriteStream } = require('node:fs')
 const { unlink } = require('node:fs/promises')
 const path = require('node:path')
 const { generateId } = require('./lib/generateId')
-const util = require('node:util')
 const createError = require('@fastify/error')
-const sendToWormhole = require('stream-wormhole')
+const { sendToWormhole } = require('stream-wormhole')
 const deepmergeAll = require('@fastify/deepmerge')({ all: true })
-const { PassThrough, pipeline, Readable } = require('node:stream')
-const pump = util.promisify(pipeline)
+const { PassThrough, Readable } = require('node:stream')
+const { pipeline: pump } = require('node:stream/promises')
 const secureJSON = require('secure-json-parse')
 
 const kMultipart = Symbol('multipart')
@@ -95,7 +94,6 @@ function fastifyMultipart (fastify, options, done) {
             const field = req.body[key]
 
             /* Don't modify the body if a field doesn't have a value or an attached buffer */
-            /* istanbul ignore else */
             if (field.value !== undefined) {
               body[key] = field.value
             } else if (field._buf) {
@@ -454,10 +452,10 @@ function fastifyMultipart (fastify, options, done) {
       const filepath = this.tmpUploads[i]
       try {
         await unlink(filepath)
-      } catch (error) {
-        /* istanbul ignore next */
+      } /* c8 ignore start */ catch (error) {
         this.log.error(error, 'Could not delete file')
       }
+      /* c8 ignore stop */
     }
   }
 
@@ -466,7 +464,6 @@ function fastifyMultipart (fastify, options, done) {
     let part
     while ((part = await parts()) != null) {
       /* Only return a part if the file property exists */
-      /* istanbul ignore else */
       if (part.file) {
         return part
       }

package.json

@@ -6,37 +6,37 @@
   "type": "commonjs",
   "types": "types/index.d.ts",
   "dependencies": {
-    "@fastify/busboy": "^1.0.0",
-    "@fastify/deepmerge": "^1.0.0",
-    "@fastify/error": "^3.0.0",
-    "fastify-plugin": "^4.0.0",
-    "secure-json-parse": "^2.4.0",
-    "stream-wormhole": "^1.1.0"
+    "@fastify/busboy": "^1.2.1",
+    "@fastify/deepmerge": "^1.3.0",
+    "@fastify/error": "^3.4.1",
+    "fastify-plugin": "^4.5.1",
+    "secure-json-parse": "^2.7.0",
+    "stream-wormhole": "^2.0.1"
   },
   "devDependencies": {
-    "@fastify/pre-commit": "^2.0.2",
-    "@fastify/swagger": "^8.10.1",
-    "@fastify/swagger-ui": "^2.0.1",
-    "@types/node": "^20.1.0",
-    "@typescript-eslint/eslint-plugin": "^6.3.0",
-    "@typescript-eslint/parser": "^6.3.0",
+    "@fastify/pre-commit": "^2.1.0",
+    "@fastify/swagger": "^8.14.0",
+    "@fastify/swagger-ui": "^2.1.0",
+    "@types/node": "^20.11.6",
+    "@typescript-eslint/eslint-plugin": "^6.19.1",
+    "@typescript-eslint/parser": "^6.19.1",
     "benchmark": "^2.1.4",
     "climem": "^2.0.0",
     "concat-stream": "^2.0.0",
-    "eslint": "^8.20.0",
-    "eslint-plugin-import": "^2.26.0",
-    "eslint-plugin-n": "^16.0.1",
-    "eslint-plugin-promise": "^6.0.0",
-    "fastify": "^4.0.0",
+    "eslint": "^8.56.0",
+    "eslint-plugin-import": "^2.29.1",
+    "eslint-plugin-n": "^16.6.2",
+    "eslint-plugin-promise": "^6.1.1",
+    "fastify": "^4.25.2",
     "form-data": "^4.0.0",
     "h2url": "^0.2.0",
     "noop-stream": "^0.1.0",
     "pump": "^3.0.0",
-    "readable-stream": "^4.5.1",
+    "readable-stream": "^4.5.2",
     "snazzy": "^9.0.0",
-    "standard": "^17.0.0",
-    "tap": "^16.0.0",
-    "tsd": "^0.30.0"
+    "standard": "^17.1.0",
+    "tap": "^18.6.1",
+    "tsd": "^0.30.4"
   },
   "scripts": {
     "coverage": "npm run test:unit -- --coverage-report=html",
@@ -48,7 +48,7 @@
     "start": "CLIMEM=8999 node -r climem ./examples/example",
     "test": "npm run test:unit && npm run test:typescript",
     "test:typescript": "tsd",
-    "test:unit": "tap -t 90"
+    "test:unit": "tap -t 120"
   },
   "repository": {
     "type": "git",

test/multipart-big-stream.test.js

@@ -7,7 +7,7 @@ const multipart = require('..')
 const http = require('node:http')
 const crypto = require('node:crypto')
 const { Readable } = require('readable-stream')
-const sendToWormhole = require('stream-wormhole')
+const { sendToWormhole } = require('stream-wormhole')
 const EventEmitter = require('node:events')
 const { once } = EventEmitter
 

test/multipart-http2.test.js

@@ -7,7 +7,7 @@ const multipart = require('..')
 const h2url = require('h2url')
 const path = require('node:path')
 const fs = require('node:fs')
-const sendToWormhole = require('stream-wormhole')
+const { sendToWormhole } = require('stream-wormhole')
 
 const filePath = path.join(__dirname, '../README.md')
 

test/multipart-small-stream.test.js

@@ -8,7 +8,7 @@ const http = require('node:http')
 const path = require('node:path')
 const fs = require('node:fs')
 const EventEmitter = require('node:events')
-const sendToWormhole = require('stream-wormhole')
+const { sendToWormhole } = require('stream-wormhole')
 const { once } = EventEmitter
 
 const filePath = path.join(__dirname, '../README.md')

test/multipart.test.js

@@ -12,7 +12,7 @@ const concat = require('concat-stream')
 const stream = require('node:stream')
 const { once } = require('node:events')
 const pump = util.promisify(stream.pipeline)
-const sendToWormhole = require('stream-wormhole')
+const { sendToWormhole } = require('stream-wormhole')
 
 const filePath = path.join(__dirname, '../README.md')