Do not run "Leaks ignores Counter globals" on Windows Node >= 11.

Use rimraf.sync instead of fs.rmdirSync to delete directories in tests, in order to prevent spurious ENOENT errors on Windows.

Author: Invader444

package.json

@@ -31,7 +31,8 @@
     "cpr": "3.0.x",
     "eslint-plugin-markdown": "1.0.0-rc.0",
     "lab-event-reporter": "1.x.x",
-    "rimraf": "2.6.x"
+    "rimraf": "2.6.x",
+    "semver": "5.6.x"
   },
   "bin": {
     "lab": "./bin/lab"

test/leaks.js

@@ -3,6 +3,7 @@
 // Load modules
 
 const Code = require('code');
+const Semver = require('semver');
 const _Lab = require('../test_runner');
 const Lab = require('../');
 
@@ -100,7 +101,7 @@ describe('Leaks', () => {
         expect(leaks.length).to.equal(0);
     });
 
-    it('ignores Counter globals', () => {
+    it('ignores Counter globals', { skip: process.platform === 'win32' && Semver.gte(process.version, '11.0.0') }, () => {
 
         const counterGlobals = internals.counterGlobals;
         testedKeys = internals.counterGlobals;

test/reporters.js

@@ -6,6 +6,7 @@ const Crypto = require('crypto');
 const Fs = require('fs');
 const Os = require('os');
 const Path = require('path');
+const Rimraf = require('rimraf');
 const Stream = require('stream');
 const Code = require('code');
 const _Lab = require('../test_runner');
@@ -101,8 +102,7 @@ describe('Reporter', () => {
 
         expect(code).to.equal(0);
         expect(output).to.equal(Fs.readFileSync(filename).toString());
-        Fs.unlinkSync(filename);
-        Fs.rmdirSync(folder);
+        Rimraf.sync(folder);
     });
 
     it('outputs to a file with output is passed as an array and reporter is an array', async () => {