test,assert: remove internal errorCache property

lib/assert.js

@@ -27,12 +27,14 @@ const { codes: {
   ERR_INVALID_ARG_VALUE,
   ERR_INVALID_RETURN_VALUE
 } } = require('internal/errors');
-const { AssertionError, errorCache } = require('internal/assert');
+const { AssertionError } = require('internal/assert');
 const { openSync, closeSync, readSync } = require('fs');
 const { inspect, types: { isPromise, isRegExp } } = require('util');
 const { EOL } = require('internal/constants');
 const { NativeModule } = require('internal/bootstrap/loaders');
 
+const errorCache = new Map();
+
 let isDeepEqual;
 let isDeepStrictEqual;
 let parseExpressionAt;

lib/internal/assert.js

@@ -396,6 +396,5 @@ class AssertionError extends Error {
 }
 
 module.exports = {
-  AssertionError,
-  errorCache: new Map()
+  AssertionError
 };

test/parallel/test-assert-builtins-not-read-from-filesystem.js

@@ -5,47 +5,44 @@
 
 require('../common');
 const assert = require('assert');
+const EventEmitter = require('events');
+const e = new EventEmitter();
+e.on('hello', assert);
 
 if (process.argv[2] !== 'child') {
   const tmpdir = require('../common/tmpdir');
   tmpdir.refresh();
   const { spawnSync } = require('child_process');
-  const { output, status, error } =
-    spawnSync(process.execPath,
-              ['--expose-internals', process.argv[1], 'child'],
-              { cwd: tmpdir.path, env: process.env });
-  assert.ifError(error);
-  assert.strictEqual(status, 0, `Exit code: ${status}\n${output}`);
-} else {
-  const EventEmitter = require('events');
-  const { errorCache } = require('internal/assert');
-  const { writeFileSync } = require('fs');
-  const e = new EventEmitter();
-
-  e.on('hello', assert);
 
   let threw = false;
   try {
     e.emit('hello', false);
   } catch (err) {
     const frames = err.stack.split('\n');
     const [, filename, line, column] = frames[1].match(/\((.+):(\d+):(\d+)\)/);
-    // Reset the cache to check again
-    const size = errorCache.size;
-    errorCache.delete(`${filename}${line - 1}${column - 1}`);
-    assert.strictEqual(errorCache.size, size - 1);
-    const data = `${'\n'.repeat(line - 1)}${' '.repeat(column - 1)}` +
-                 'ok(failed(badly));';
+    // Spawn a child process to avoid the error having been cached in the assert
+    // module's `errorCache` Map.
 
-    writeFileSync(filename, data);
-    assert.throws(
-      () => e.emit('hello', false),
-      {
-        message: 'false == true'
-      }
-    );
+    const { output, status, error } =
+      spawnSync(process.execPath,
+                [process.argv[1], 'child', filename, line, column],
+                { cwd: tmpdir.path, env: process.env });
+    assert.ifError(error);
+    assert.strictEqual(status, 0, `Exit code: ${status}\n${output}`);
     threw = true;
-
   }
-  assert(threw);
+  assert.ok(threw);
+} else {
+  const { writeFileSync } = require('fs');
+  const [, , , filename, line, column] = process.argv;
+  const data = `${'\n'.repeat(line - 1)}${' '.repeat(column - 1)}` +
+               'ok(failed(badly));';
+
+  writeFileSync(filename, data);
+  assert.throws(
+    () => e.emit('hello', false),
+    {
+      message: 'false == true'
+    }
+  );
 }