fastify 4.0.1 → 4.0.2

package/build/build-error-serializer.js

@@ -1,3 +1,4 @@
+/* istanbul ignore file */
 'use strict'
 
 const FJS = require('fast-json-stringify')

package/docs/Guides/Migration-Guide-V4.md

@@ -14,7 +14,7 @@ Starting this version of Fastify, we have deprecated the use of `app.use()`. We
 have decided not to support the use of middlewares. Both
 [`@fastify/middie`](https://github.com/fastify/middie) and
 [`@fastify/express`](https://github.com/fastify/fastify-express) will still be
-there and maintained. Use Fastify's [hooks](./Reference/Hooks.md) instead.
+there and maintained. Use Fastify's [hooks](../Reference/Hooks.md) instead.
 
 ## Non Breaking Changes
 

package/fastify.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const VERSION = '4.0.1'
+const VERSION = '4.0.2'
 
 const Avvio = require('avvio')
 const http = require('http')

package/lib/reply.js

@@ -291,7 +291,7 @@ Reply.prototype.removeTrailer = function (key) {
 
 Reply.prototype.code = function (code) {
   const intValue = parseInt(code)
-  if (isNaN(intValue) || intValue < 100 || intValue > 600) {
+  if (isNaN(intValue) || intValue < 100 || intValue > 599) {
     throw new FST_ERR_BAD_STATUS_CODE(code || String(code))
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fastify",
-  "version": "4.0.1",
+  "version": "4.0.2",
   "description": "Fast and low overhead web framework, for Node.js",
   "main": "fastify.js",
   "type": "commonjs",
@@ -17,7 +17,7 @@
     "lint:markdown": "markdownlint-cli2",
     "lint:standard": "standard | snazzy",
     "lint:typescript": "eslint -c types/.eslintrc.json types/**/*.d.ts test/types/**/*.test-d.ts",
-    "prepublishOnly": "tap --no-check-coverage test/internals/version.test.js",
+    "prepublishOnly": "tap --no-check-coverage test/build/**.test.js",
     "test": "npm run lint && npm run unit && npm run test:typescript",
     "test:ci": "npm run unit -- -R terse --cov --coverage-report=lcovonly && npm run test:typescript",
     "test:report": "npm run lint && npm run unit:report && npm run test:typescript",
@@ -146,7 +146,6 @@
     "eslint-plugin-n": "^15.2.0",
     "eslint-plugin-promise": "^6.0.0",
     "fast-json-body": "^1.1.0",
-    "fast-json-stringify": "^4.1.0",
     "fastify-plugin": "^3.0.1",
     "fluent-json-schema": "^3.1.0",
     "form-data": "^4.0.0",
@@ -182,6 +181,7 @@
     "@fastify/fast-json-stringify-compiler": "^3.0.0",
     "abstract-logging": "^2.0.1",
     "avvio": "^8.1.3",
+    "fast-json-stringify": "^4.1.0",
     "find-my-way": "^6.3.0",
     "light-my-request": "^5.0.0",
     "pino": "^8.0.0",

package/test/build/error-serializer.test.js

@@ -0,0 +1,28 @@
+'use strict'
+
+const t = require('tap')
+const test = t.test
+const fs = require('fs')
+const path = require('path')
+
+const { code } = require('../../build/build-error-serializer')
+
+test('check generated code syntax', async (t) => {
+  t.plan(1)
+
+  // standard is a esm, we import it like this
+  const { default: standard } = await import('standard')
+  const result = await standard.lintText(code)
+
+  // if there are any invalid syntax
+  // fatal count will be greater than 0
+  t.equal(result[0].fatalErrorCount, 0)
+})
+
+test('ensure the current error serializer is latest', async (t) => {
+  t.plan(1)
+
+  const current = await fs.promises.readFile(path.resolve('lib/error-serializer.js'))
+
+  t.equal(current.toString(), code)
+})

package/test/{internals → build}/version.test.js

@@ -4,7 +4,7 @@ const fs = require('fs')
 const path = require('path')
 const t = require('tap')
 const test = t.test
-const fastify = require('../..')()
+const fastify = require('../../fastify')()
 
 test('should be the same as package.json', t => {
   t.plan(1)

package/test/reply-error.test.js

@@ -520,7 +520,13 @@ const invalidErrorCodes = [
   undefined,
   null,
   'error_code',
-  700 // out of the 100-600 range
+
+  // out of the 100-599 range:
+  0,
+  1,
+  99,
+  600,
+  700
 ]
 invalidErrorCodes.forEach((invalidCode) => {
   test(`should throw error if error code is ${invalidCode}`, t => {