test: use maxRetries with tests too (#3150)

Author: lukekarrys

test/test-download.js

@@ -163,7 +163,7 @@ describe('download', function () {
     this.timeout(platformTimeout(1, { win32: 5 }))
 
     const expectedDir = path.join(devDir, process.version.replace(/^v/, ''))
-    await fs.rm(expectedDir, { recursive: true, force: true })
+    await fs.rm(expectedDir, { recursive: true, force: true, maxRetries: 3 })
 
     const prog = gyp()
     prog.parseArgv([])

test/test-install.js

@@ -63,7 +63,7 @@ describe('install', function () {
     })
 
     afterEach(async () => {
-      await rm(prog.devDir, { recursive: true, force: true })
+      await rm(prog.devDir, { recursive: true, force: true, maxRetries: 3 })
       prog = null
     })
 
@@ -74,14 +74,16 @@ describe('install', function () {
       }
 
       return it(name, async function () {
-        this.timeout(platformTimeout(1, { win32: 20 }))
-        const start = Date.now()
+        this.timeout(platformTimeout(2, { win32: 20 }))
         await fn.call(this)
         const expectedDir = path.join(prog.devDir, process.version.replace(/^v/, ''))
-        await rm(expectedDir, { recursive: true, force: true })
+        await rm(expectedDir, { recursive: true, force: true, maxRetries: 3 })
         await Promise.all(new Array(10).fill(0).map(async (_, i) => {
+          const title = `${' '.repeat(8)}${name} ${(i + 1).toString().padEnd(2, ' ')}`
+          console.log(`${title} : Start`)
+          console.time(title)
           await install(prog, [])
-          console.log(`${' '.repeat(8)}${name} ${(i + 1).toString().padEnd(2, ' ')} (${Date.now() - start}ms)`)
+          console.timeEnd(title)
         }))
       })
     }