Update dev deps

package.json

@@ -16,7 +16,7 @@
   "scripts": {
     "coverage-html": "npm run tap -- --coverage-report=html",
     "lint": "eslint bin/* lib/ test/ --cache",
-    "tap": "tap -J --timeout 480 \"test/**/test-*.js\"",
+    "tap": "tap -J \"test/**/test-*.js\"",
     "test": "npm run lint && npm run tap",
     "test-ci": "npm run lint && npm run tap -- --coverage-report=lcov"
   },
@@ -64,12 +64,12 @@
     "eslint": "^7.21.0",
     "eslint-config-prettier": "^8.1.0",
     "eslint-plugin-prettier": "^3.3.1",
-    "fs-extra": "^9.1.0",
+    "fs-extra": "^10.0.0",
     "prettier": "^2.2.1",
     "proxyquire": "^2.1.3",
     "rewire": "^5.0.0",
     "string-to-stream": "^3.0.1",
-    "tap": "^14.11.0",
+    "tap": "^15.0.9",
     "tap-parser": "^10.1.0",
     "xml2js": "^0.4.23"
   },
@@ -85,5 +85,9 @@
         }
       }
     ]
+  },
+  "tap": {
+    "timeout": 480,
+    "check-coverage": false
   }
 }

test/bin/test-citgm-all.js

@@ -22,7 +22,7 @@ test('citgm-all: /w markdown /w -j', (t) => {
     t.fail('we should not get an error testing omg-i-pass');
   });
   proc.on('close', (code) => {
-    t.equals(code, 0, 'citgm-all should run all the tests in the lookup');
+    t.equal(code, 0, 'citgm-all should run all the tests in the lookup');
   });
 });
 
@@ -36,11 +36,7 @@ test('citgm-all: /w missing lookup.json', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(
-      code,
-      1,
-      'citgm-all should fail if the lookup.json does not exist'
-    );
+    t.equal(code, 1, 'citgm-all should fail if the lookup.json does not exist');
   });
 });
 
@@ -54,7 +50,7 @@ test('citgm-all: /w bad lookup.json', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(
+    t.equal(
       code,
       1,
       'citgm-all should fail if the lookup.json contains errors'
@@ -74,7 +70,7 @@ test('citgm-all: fail /w tap /w junit', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 1, 'citgm-all should have failed');
+    t.equal(code, 1, 'citgm-all should have failed');
   });
 });
 
@@ -88,7 +84,7 @@ test('citgm-all: flaky-fail', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 0, 'citgm-all should exit with signal 0');
+    t.equal(code, 0, 'citgm-all should exit with signal 0');
   });
 });
 
@@ -102,7 +98,7 @@ test('citgm-all: fail expectFail', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 0, 'citgm-all should exit with signal 0');
+    t.equal(code, 0, 'citgm-all should exit with signal 0');
   });
 });
 
@@ -116,7 +112,7 @@ test('citgm-all: pass expectFail', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 1, 'citgm-all should exit with signal 1');
+    t.equal(code, 1, 'citgm-all should exit with signal 1');
   });
 });
 
@@ -130,7 +126,7 @@ test('citgm-all: test with replace', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 0, 'citgm-all should exit with signal 0');
+    t.equal(code, 0, 'citgm-all should exit with signal 0');
   });
 });
 
@@ -145,7 +141,7 @@ test('citgm-all: flaky-fail ignoring flakyness', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 1, 'citgm-all should exit with signal 1');
+    t.equal(code, 1, 'citgm-all should exit with signal 1');
   });
 });
 
@@ -161,7 +157,7 @@ test('citgm-all: includeTags', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 0, 'citgm-all should only run omg-i-pass');
+    t.equal(code, 0, 'citgm-all should only run omg-i-pass');
   });
 });
 
@@ -177,7 +173,7 @@ test('citgm-all: excludeTags', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 0, 'citgm-all should not run omg-i-fail');
+    t.equal(code, 0, 'citgm-all should not run omg-i-fail');
   });
 });
 
@@ -193,7 +189,7 @@ test('citgm-all: includeTags multiple', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 0, 'citgm-all should only run omg-i-pass');
+    t.equal(code, 0, 'citgm-all should only run omg-i-pass');
   });
 });
 
@@ -209,7 +205,7 @@ test('citgm-all: excludeTags modulename', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 0, 'citgm-all should not run omg-i-fail');
+    t.equal(code, 0, 'citgm-all should not run omg-i-fail');
   });
 });
 
@@ -225,7 +221,7 @@ test('citgm-all: includeTags modulename multiple', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 0, 'citgm-all should only run omg-i-pass');
+    t.equal(code, 0, 'citgm-all should only run omg-i-pass');
   });
 });
 
@@ -245,7 +241,7 @@ test('citgm-all: skip /w rootcheck /w tap to fs /w junit to fs /w append', (t) =
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 0, 'it should run omg-i-pass and skip omg-i-fail');
+    t.equal(code, 0, 'it should run omg-i-pass and skip omg-i-fail');
   });
 });
 
@@ -260,7 +256,7 @@ test('citgm-all: install with yarn', (t) => {
     t.error(err);
   });
   proc.on('close', (code) => {
-    t.equals(code, 0, 'citgm-all should only run omg-i-pass');
+    t.equal(code, 0, 'citgm-all should only run omg-i-pass');
   });
 });
 

test/npm/test-npm-author-name.js

@@ -16,8 +16,8 @@ test('npm.test() authorName:', (t) => {
     url: 'omg.html'
   };
   const authorExpected = 'Randy Savage <randy@wwe.rekt> (omg.html)';
-  t.equals(authorName(name), name, 'it should return any string');
-  t.equals(
+  t.equal(authorName(name), name, 'it should return any string');
+  t.equal(
     authorName(author),
     authorExpected,
     'it should return the expected' + ' string when given an object'
@@ -36,13 +36,13 @@ test('npm.test() authorName partial data:', (t) => {
     email: 'thedude@abides.net'
   };
   const authorTwoExpected = '<thedude@abides.net>';
-  t.equals(authorName(name), name, 'it should return any string');
-  t.equals(
+  t.equal(authorName(name), name, 'it should return any string');
+  t.equal(
     authorName(authorOne),
     authorOneExpected,
     'it should return the' + ' expected string when given an object'
   );
-  t.equals(
+  t.equal(
     authorName(authorTwo),
     authorTwoExpected,
     'it should return the' + ' expected string when given an object'

test/npm/test-npm-install.js

@@ -76,7 +76,7 @@ test('npm-install: no package.json', async (t) => {
   try {
     await packageManagerInstall('npm', context);
   } catch (err) {
-    t.equals(err && err.message, 'Install Failed');
+    t.equal(err && err.message, 'Install Failed');
     t.notOk(context.module.flaky, 'Module failed but is not flaky');
   }
 });
@@ -96,7 +96,7 @@ test('npm-install: timeout', async (t) => {
     await packageManagerInstall('npm', context);
   } catch (err) {
     t.notOk(context.module.flaky, 'Time out should not mark module flaky');
-    t.equals(err && err.message, 'Install Timed Out');
+    t.equal(err && err.message, 'Install Timed Out');
   }
 });
 
@@ -118,7 +118,7 @@ test('npm-install: failed install', async (t) => {
     await packageManagerInstall('npm', context);
   } catch (err) {
     t.notOk(context.module.flaky, 'Module failed is not flaky');
-    t.equals(err && err.message, 'Install Failed');
+    t.equal(err && err.message, 'Install Failed');
     t.match(context, expected, 'Install error reported');
   }
 });

test/npm/test-npm-test.js

@@ -71,7 +71,7 @@ test('npm-test: basic module failing', async (t) => {
   try {
     await packageManagerTest('npm', context);
   } catch (err) {
-    t.equals(err && err.message, 'The canary is dead:');
+    t.equal(err && err.message, 'The canary is dead:');
   }
 });
 
@@ -85,7 +85,7 @@ test('npm-test: basic module no test script', async (t) => {
   try {
     await packageManagerTest('npm', context);
   } catch (err) {
-    t.equals(err && err.message, 'Module does not support npm-test!');
+    t.equal(err && err.message, 'Module does not support npm-test!');
   }
 });
 
@@ -99,7 +99,7 @@ test('npm-test: no package.json', async (t) => {
   try {
     await packageManagerTest('npm', context);
   } catch (err) {
-    t.equals(err && err.message, 'Package.json Could not be found');
+    t.equal(err && err.message, 'Package.json Could not be found');
   }
 });
 
@@ -118,7 +118,7 @@ test('npm-test: alternative test-path', async (t) => {
   try {
     await packageManagerTest('npm', context);
   } catch (err) {
-    t.equals(err && err.message, 'The canary is dead:');
+    t.equal(err && err.message, 'The canary is dead:');
   }
 });
 
@@ -137,7 +137,7 @@ test('npm-test: timeout', async (t) => {
     await packageManagerTest('npm', context);
   } catch (err) {
     t.notOk(context.module.flaky, 'Time out should not mark module flaky');
-    t.equals(err && err.message, 'Test Timed Out');
+    t.equal(err && err.message, 'Test Timed Out');
   }
 });
 
@@ -171,7 +171,7 @@ test('npm-test: module with no test script failing', async (t) => {
   try {
     await packageManagerTest('npm', context);
   } catch (err) {
-    t.equals(err && err.message, 'Module does not support npm-test!');
+    t.equal(err && err.message, 'Module does not support npm-test!');
   }
 });
 

test/reporter/test-reporter-junit.js

@@ -64,7 +64,7 @@ test('reporter.junit(): passing', (t) => {
   }
 
   junit(logger, passingInput);
-  t.equals(
+  t.equal(
     output,
     passingExpected,
     'we should get expected output when all' + ' modules pass'
@@ -105,7 +105,7 @@ test('reporter.junit(): failing', (t) => {
   }
 
   junit(logger, failingInput);
-  t.equals(output, failingExpected),
+  t.equal(output, failingExpected),
     'we should get the expected output when a' + ' module fails';
   t.end();
 });
@@ -120,7 +120,7 @@ test('reporter.junit(): parser', async (t) => {
 
   junit(logger, failingInput);
   const result = await parseString(output);
-  t.deepEquals(
+  t.same(
     result,
     junitParserExpected,
     'we should get the expected output when a module fails'
@@ -131,7 +131,7 @@ test('reporter.junit(): write to disk', (t) => {
   t.plan(1);
   junit(outputFile, passingInput);
   const expected = fs.readFileSync(outputFile, 'utf8');
-  t.equals(expected, passingExpected),
+  t.equal(expected, passingExpected),
     'the file on disk should match the' + ' expected output';
   t.end();
 });
@@ -142,7 +142,7 @@ test('reporter.junit(): append to disk', (t) => {
   fs.writeFileSync(outputFileAppend, appendStartFile);
   junit(outputFileAppend, passingInput, true);
   const expected = fs.readFileSync(outputFileAppend, 'utf-8');
-  t.equals(expected, passingExpectedAppend),
+  t.equal(expected, passingExpectedAppend),
     'the file on disk should match the' + ' expected output';
   t.end();
 });

test/reporter/test-reporter-logger.js

@@ -36,7 +36,7 @@ test('single passing module:', (t) => {
   expected += 'The smoke test has passed.';
   output = '';
   loggerReporter(logger, fixtures.iPass);
-  t.equals(output, expected, 'we should have the expected logged output');
+  t.equal(output, expected, 'we should have the expected logged output');
   t.end();
 });
 
@@ -51,7 +51,7 @@ test('single failing module:', (t) => {
   expected += 'The smoke test has failed.';
   output = '';
   loggerReporter(logger, fixtures.iFail);
-  t.equals(output, expected, 'we should have the expected logged output');
+  t.equal(output, expected, 'we should have the expected logged output');
   t.end();
 });
 
@@ -75,6 +75,6 @@ test('multiple modules passing, with a flaky module that fails:', (t) => {
     fixtures.iFlakyPass,
     fixtures.iFlakyFail
   ]);
-  t.equals(output, expected, 'we should have the expected logged output');
+  t.equal(output, expected, 'we should have the expected logged output');
   t.end();
 });

test/reporter/test-reporter-markdown.js

@@ -14,7 +14,7 @@ test('single passing module:', (t) => {
   let expected = '## 🎉🎉 CITGM Passed 🎉🎉';
   expected += '### Passing Modules';
   expected += '  * iPass v4.2.2 duration:50ms';
-  t.equals(output, expected, 'we should have the expected markdown output');
+  t.equal(output, expected, 'we should have the expected markdown output');
   t.end();
 });
 
@@ -29,7 +29,7 @@ test('single failing module:', (t) => {
   expected += '  * iFail v3.0.1 duration:50ms';
   expected += '    - I dun wurk';
   expected += '    -  Thanks for testing!';
-  t.equals(output, expected, 'we should have the expected markdown output');
+  t.equal(output, expected, 'we should have the expected markdown output');
   t.end();
 });
 
@@ -51,6 +51,6 @@ test('multiple modules passing, with a flaky module that fails:', (t) => {
   expected += '  * iFlakyFail v3.3.3 duration:50ms';
   expected += '    - I dun wurk';
   expected += '    -  Thanks for testing!';
-  t.equals(output, expected, 'we should have the expected markdown output');
+  t.equal(output, expected, 'we should have the expected markdown output');
   t.end();
 });