benchmark: fix race condition on readfile

benchmark/fs/readfile-partitioned.js

@@ -11,7 +11,7 @@
 const path = require('path');
 const common = require('../common.js');
 const filename = path.resolve(__dirname,
-                              `.removeme-benchmark-garbage-${process.pid}`);
+  `.removeme-benchmark-garbage-${process.pid}`);
 const fs = require('fs');
 const zlib = require('zlib');
 const assert = require('assert');
@@ -43,11 +43,17 @@ function main({ len, duration, concurrent, encoding }) {
     const totalOps = reads + zips;
     benchEnded = true;
     bench.end(totalOps);
-    try {
-      fs.unlinkSync(filename);
-    } catch {
-      // Continue regardless of error.
-    }
+
+    // this delay is needed because on windows this can cause
+    // race condition with afterRead, which makes this benchmark
+    // fails to delete the temp file
+    setTimeout(() => {
+      try {
+        fs.unlinkSync(filename);
+      } catch {
+        // Continue regardless of error.
+      }
+    }, 10);
   }, duration * 1000);
 
   function read() {

benchmark/fs/readfile-permission-enabled.js

@@ -39,14 +39,20 @@ function main({ len, duration, concurrent, encoding }) {
   let benchEnded = false;
   bench.start();
   setTimeout(() => {
-    benchEnded = true;
+    benchEnded = true
     bench.end(reads);
-    try {
-      fs.unlinkSync(filename);
-    } catch {
-      // Continue regardless of error.
-    }
-    process.exit(0);
+
+    // this delay is needed because on windows this can cause
+    // race condition with afterRead, which makes this benchmark
+    // fails to delete the temp file
+    setTimeout(() => {
+      try {
+        fs.unlinkSync(filename);
+      } catch {
+        // Continue regardless of error.
+      }
+      process.exit(0);
+    }, 10);
   }, duration * 1000);
 
   function read() {

benchmark/fs/readfile-promises.js

@@ -41,12 +41,18 @@ function main({ len, duration, concurrent, encoding }) {
   setTimeout(() => {
     benchEnded = true;
     bench.end(writes);
-    try {
-      fs.unlinkSync(filename);
-    } catch {
-      // Continue regardless of error.
-    }
-    process.exit(0);
+
+    // this delay is needed because on windows this can cause
+    // race condition with afterRead, which makes this benchmark
+    // fails to delete the temp file
+    setTimeout(() => {
+      try {
+        fs.unlinkSync(filename);
+      } catch {
+        // Continue regardless of error.
+      }
+      process.exit(0);
+    }, 10);
   }, duration * 1000);
 
   function read() {

benchmark/fs/readfile.js

@@ -34,12 +34,18 @@ function main({ len, duration, concurrent, encoding }) {
   setTimeout(() => {
     benchEnded = true;
     bench.end(reads);
-    try {
-      fs.unlinkSync(filename);
-    } catch {
-      // Continue regardless of error.
-    }
-    process.exit(0);
+
+    // this delay is needed because on windows this can cause
+    // race condition with afterRead, which makes this benchmark
+    // fails to delete the temp file
+    setTimeout(() => {
+      try {
+        fs.unlinkSync(filename);
+      } catch {
+        // Continue regardless of error.
+      }
+      process.exit(0);
+    }, 10);
   }, duration * 1000);
 
   function read() {

benchmark/fs/writefile-promises.js

@@ -43,14 +43,20 @@ function main({ encodingType, duration, concurrent, size }) {
   setTimeout(() => {
     benchEnded = true;
     bench.end(writes);
-    for (let i = 0; i < filesWritten; i++) {
-      try {
-        fs.unlinkSync(`${filename}-${i}`);
-      } catch {
-        // Continue regardless of error.
+
+    // this delay is needed because on windows this can cause
+    // race condition with afterRead, which makes this benchmark
+    // fails to delete the temp file
+    setTimeout(() => {
+      for (let i = 0; i < filesWritten; i++) {
+        try {
+          fs.unlinkSync(`${filename}-${i}`);
+        } catch {
+          // Continue regardless of error.
+        }
       }
-    }
-    process.exit(0);
+      process.exit(0);
+    }, 10);
   }, duration * 1000);
 
   function write() {