Add retry to tests

Author: extremeheat

package.json

@@ -6,7 +6,7 @@
   "types": "index.d.ts",
   "scripts": {
     "build": "cd tools && node compileProtocol.js",
-    "test": "mocha --bail --exit",
+    "test": "mocha --retries 2 --bail --exit",
     "pretest": "npm run lint",
     "lint": "standard",
     "vanillaServer": "minecraft-bedrock-server --root tools --version",

test/util.js

@@ -5,7 +5,12 @@ const getPort = () => new Promise(resolve => {
   server.listen(0, '127.0.0.1')
   server.on('listening', () => {
     const { port } = server.address()
-    server.close(() => resolve(port))
+    server.close(() => {
+      // Wait a bit for port to free as we try to bind right after freeing it
+      setTimeout(() => {
+        resolve(port)
+      }, 200)
+    })
   })
 })
 

test/vanilla.js

@@ -8,8 +8,9 @@ async function test (version) {
   // const ChunkColumn = require('bedrock-provider').chunk('bedrock_' + (version.includes('1.19') ? '1.18.30' : version)) // TODO: Fix prismarine-chunk
 
   // Start the server, wait for it to accept clients, throws on timeout
-  const port = await getPort()
-  const handle = await vanillaServer.startServerAndWait2(version, 1000 * 220, { 'server-port': port })
+  const [port, v6] = [await getPort(), await getPort()]
+  console.log('Starting vanilla server', version, 'on port', port, v6)
+  const handle = await vanillaServer.startServerAndWait2(version, 1000 * 220, { 'server-port': port, 'server-portv6': v6 })
   console.log('Started server')
 
   const client = new Client({

tools/genPacketDumps.js

@@ -24,7 +24,7 @@ async function dump (version, force = true) {
   const random = (Math.random() * 1000) | 0
   const [port, v6] = [await getPort(), await getPort()]
 
-  console.log('Starting dump server', version)
+  console.log('Starting dump server', version, 'on port', port, v6)
   const handle = await vanillaServer.startServerAndWait2(version || CURRENT_VERSION, 1000 * 120, { 'server-port': port, 'server-portv6': v6 })
 
   console.log('Started dump server', version)