chore: migrate from tap to node:test and c8 using borp (#182)

Author: dancastillo

package.json

@@ -12,10 +12,9 @@
     "load-data": "docker exec -it fastify-postgres psql -c 'CREATE TABLE users(id serial PRIMARY KEY, username VARCHAR (50) NOT NULL);' -U postgres -d postgres",
     "postgres": "docker run -p 5432:5432 --name fastify-postgres -e POSTGRES_PASSWORD=postgres -d postgres:11-alpine",
     "test": "npm run test:unit && npm run test:typescript",
-    "test:unit": "tap test/*.test.js",
-    "test:report": "standard && tap --coverage-report=html test/*.test.js",
-    "test:typescript": "tsd",
-    "test:verbose": "standard && tap test/*.test.js -Rspec"
+    "test:unit": "c8 --100 node --test",
+    "test:report": "standard && c8 --reporter html node --test",
+    "test:typescript": "tsd"
   },
   "repository": {
     "type": "git",
@@ -41,11 +40,11 @@
   "devDependencies": {
     "@tsconfig/node10": "^1.0.9",
     "@types/pg": "^8.11.4",
+    "c8": "^10.1.2",
     "fastify": "^5.0.0",
     "pg": "^8.11.3",
     "pg-native": "^3.0.1",
     "standard": "^17.1.0",
-    "tap": "^18.7.1",
     "tsd": "^0.31.1",
     "typescript": "~5.6.3"
   },

test/add-handler.test.js

@@ -1,32 +1,32 @@
 'use strict'
 
-const { test } = require('tap')
+const { test } = require('node:test')
 const addHandler = require('../lib/add-handler')
 
-test('The addHandler lib should return the right handlers structure', t => {
-  t.test('When the existingHandler argument is undefined', t => {
+test('The addHandler lib should return the right handlers structure', async t => {
+  await t.test('When the existingHandler argument is undefined', t => {
     t.plan(1)
 
     const handlers = addHandler(
       undefined,
       'test'
     )
 
-    t.same(handlers, ['test'])
+    t.assert.deepStrictEqual(handlers, ['test'])
   })
 
-  t.test('When the existingHandler argument is an array', t => {
+  await t.test('When the existingHandler argument is an array', t => {
     t.plan(1)
 
     const handlers = addHandler(
       ['test'],
       'again'
     )
 
-    t.same(handlers, ['test', 'again'])
+    t.assert.deepStrictEqual(handlers, ['test', 'again'])
   })
 
-  t.test('When the existingHandler argument is a function', t => {
+  await t.test('When the existingHandler argument is a function', t => {
     t.plan(2)
 
     const stub = () => 'test'
@@ -36,9 +36,7 @@ test('The addHandler lib should return the right handlers structure', t => {
       'again'
     )
 
-    t.same(handlers[0](), 'test')
-    t.same(handlers[1], 'again')
+    t.assert.deepStrictEqual(handlers[0](), 'test')
+    t.assert.deepStrictEqual(handlers[1], 'again')
   })
-
-  t.end()
 })

test/initialization-log.test.js

@@ -0,0 +1,46 @@
+const { test } = require('node:test')
+const Fastify = require('fastify')
+const pg = require('pg')
+const fastifyPostgres = require('../index')
+const { connectionString } = require('./helpers')
+
+test('Initalization log tests', async (t) => {
+  const realConsole = global.console
+  const ctx = {}
+
+  test.beforeEach(() => {
+    ctx.fastify = require('fastify')()
+    ctx.pg = {
+      ...require('pg'),
+      native: null
+    }
+    ctx.native = pg.native
+  })
+
+  test.afterEach(() => {
+  // Really would just remove the module from the
+  // cache.
+    ctx.pg.native = ctx.native
+    global.console = realConsole
+  })
+
+  await t.test('Should print warning when native module not installed', async (t) => {
+    t.plan(2)
+
+    global.console.warn = (msg) => t.assert.strictEqual(msg, "pg-native not installed, can't use native option - fallback to pg module")
+
+    const fastify = Fastify()
+    t.after(() => {
+      fastify.close()
+    })
+
+    await ctx.fastify.register(fastifyPostgres, {
+      connectionString,
+      native: true,
+      pg: ctx.pg
+    })
+
+    const ready = await ctx.fastify.ready()
+    t.assert.ok(ready)
+  })
+})