test(audit-core,predicates,otp,xo-server): move from tap to test

Support of Mocha syntax is much easier than in latest versions of tap

@vates/otp/index.test.mjs

@@ -1,5 +1,5 @@
 import { strict as assert } from 'node:assert'
-import { describe, it } from 'tap/mocha'
+import test from 'test'
 
 import {
   generateHotp,
@@ -11,6 +11,8 @@ import {
   verifyTotp,
 } from './index.mjs'
 
+const { describe, it } = test
+
 describe('generateSecret', function () {
   it('generates a string of 32 chars', async function () {
     const secret = generateSecret()

@vates/otp/package.json

@@ -31,9 +31,9 @@
   },
   "scripts": {
     "postversion": "npm publish --access public",
-    "test": "tap"
+    "test": "node--test"
   },
   "devDependencies": {
-    "tap": "^16.3.0"
+    "test": "^3.3.0"
   }
 }

@vates/predicates/index.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const assert = require('assert/strict')
-const { describe, it } = require('tap').mocha
+const { describe, it } = require('test')
 
 const { every, not, some } = require('./')
 

@vates/predicates/package.json

@@ -32,9 +32,9 @@
   },
   "scripts": {
     "postversion": "npm publish --access public",
-    "test": "tap"
+    "test": "node--test"
   },
   "devDependencies": {
-    "tap": "^16.0.1"
+    "test": "^3.3.0"
   }
 }

@xen-orchestra/audit-core/index.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const assert = require('assert/strict')
-const { afterEach, describe, it } = require('tap').mocha
+const { afterEach, describe, it } = require('test')
 
 const { AlteredRecordError, AuditCore, MissingRecordError, NULL_ID, Storage } = require('.')
 

@xen-orchestra/audit-core/package.json

@@ -13,7 +13,7 @@
   },
   "scripts": {
     "postversion": "npm publish --access public",
-    "test": "tap --lines 67 --functions 92 --branches 52 --statements 67"
+    "test": "node--test"
   },
   "dependencies": {
     "@vates/decorate-with": "^2.1.0",
@@ -28,6 +28,6 @@
     "url": "https://vates.fr"
   },
   "devDependencies": {
-    "tap": "^16.0.1"
+    "test": "^3.3.0"
   }
 }

packages/xo-server/package.json

@@ -151,7 +151,7 @@
     "babel-plugin-transform-dev": "^2.0.1",
     "cross-env": "^7.0.2",
     "index-modules": "^0.4.3",
-    "tap": "^16.0.1"
+    "test": "^3.3.0"
   },
   "scripts": {
     "_build": "index-modules --index-file index.mjs src/api src/xapi/mixins src/xo-mixins && babel --delete-dir-on-start --keep-file-extension --source-maps --out-dir=dist/ src/",
@@ -160,7 +160,7 @@
     "prepublishOnly": "yarn run build",
     "start": "node dist/cli.mjs",
     "pretest": "yarn run build",
-    "test": "tap 'dist/**/*.spec.mjs'"
+    "test": "node--test dist"
   },
   "author": {
     "name": "Vates SAS",

packages/xo-server/src/_ensureArray.test.mjs

@@ -1,9 +1,9 @@
 import assert from 'assert/strict'
-import tap from 'tap'
+import test from 'test'
 
 import ensureArray from './_ensureArray.mjs'
 
-const { describe, it } = tap.mocha
+const { describe, it } = test
 
 describe('ensureArray()', function () {
   it('wrap the value in an array', function () {

packages/xo-server/src/_normalizeVmNetworks.test.mjs

@@ -1,9 +1,10 @@
-import tap from 'tap'
+import assert from 'node:assert'
+import test from 'test'
 
 import normalizeVmNetworks from './_normalizeVmNetworks.mjs'
 
-tap.test('normalizeVmNetworks', async t => {
+test('normalizeVmNetworks', async t => {
-  t.strictSame(
+  assert.deepStrictEqual(
     normalizeVmNetworks({
       // legacy protocol
       '0/ip': '127.0.0.1 127.0.0.2',

packages/xo-server/src/_pDebounceWithKey.test.mjs

@@ -1,8 +1,8 @@
 import assert from 'assert/strict'
-import tap from 'tap'
+import test from 'test'
 import { debounceWithKey, REMOVE_CACHE_ENTRY } from './_pDebounceWithKey.mjs'
 
-const { describe, it } = tap.mocha
+const { describe, it } = test
 
 describe('REMOVE_CACHE_ENTRY', () => {
   it('clears the cache', async () => {

packages/xo-server/src/math.test.mjs

@@ -1,10 +1,10 @@
 import assert from 'assert/strict'
 import forEach from 'lodash/forEach.js'
-import tap from 'tap'
+import test from 'test'
 import { thunkToArray } from './utils.mjs'
 import { crossProduct, mergeObjects } from './math.mjs'
 
-const { describe, it } = tap.mocha
+const { describe, it } = test
 
 describe('mergeObjects', function () {
   forEach(

packages/xo-server/src/stream-to-existing-buffer.test.mjs

@@ -1,11 +1,11 @@
 import assert from 'assert/strict'
-import tap from 'tap'
+import test from 'test'
 import { createReadStream, readFile } from 'fs'
 import { fromCallback } from 'promise-toolbox'
 
 import streamToExistingBuffer from './stream-to-existing-buffer.mjs'
 
-const { describe, it } = tap.mocha
+const { describe, it } = test
 
 describe('streamToExistingBuffer()', () => {
   it('read the content of a stream in a buffer', async () => {

packages/xo-server/src/utils.test.mjs

@@ -1,9 +1,9 @@
 import assert from 'assert/strict'
-import tap from 'tap'
+import test from 'test'
 
 import { camelToSnakeCase, diffItems, extractProperty, generateToken, parseSize, parseXml } from './utils.mjs'
 
-const { describe, it } = tap.mocha
+const { describe, it } = test
 
 // ===================================================================
 

packages/xo-server/src/xo-mixins/jobs/execute-call.test.mjs

@@ -1,9 +1,9 @@
 import assert from 'assert/strict'
 import forEach from 'lodash/forEach.js'
-import tap from 'tap'
+import test from 'node:test'
 import { resolveParamsVector } from './execute-call.mjs'
 
-const { describe, it } = tap.mocha
+const { describe, it } = test
 
 describe('resolveParamsVector', function () {
   forEach(