test: rename regression tests

test/parallel/test-regress-GH-1899.js → test/parallel/test-child-process-can-write-to-stdout.js

@@ -1,4 +1,7 @@
 'use strict';
+// Tests that a spawned child process can write to stdout without throwing.
+// See https:/nodejs/node-v0.x-archive/issues/1899.
+
 require('../common');
 const fixtures = require('../common/fixtures');
 const assert = require('assert');

test/parallel/test-regress-GH-819.js → test/parallel/test-net-connect-after-destroy.js

@@ -1,4 +1,6 @@
 'use strict';
+// Regression test for https:/nodejs/node-v0.x-archive/issues/819.
+
 require('../common');
 const net = require('net');
 

test/parallel/test-regress-GH-746.js → test/parallel/test-net-listen-after-destroying-stdin.js

@@ -1,6 +1,7 @@
 'use strict';
 // Just test that destroying stdin doesn't mess up listening on a server.
-// This is a regression test for GH-746.
+// This is a regression test for
+// https:/nodejs/node-v0.x-archive/issues/746.
 
 const common = require('../common');
 const net = require('net');

test/parallel/test-os-userinfo-handles-getter-errors.js

@@ -0,0 +1,18 @@
+'use strict';
+// Tests that os.userInfo correctly handles errors thrown by option property
+// getters. See https:/nodejs/node/issues/12370.
+
+const common = require('../common');
+const assert = require('assert');
+const execFile = require('child_process').execFile;
+
+const script = `os.userInfo({
+ get encoding() {
+ throw new Error('xyz');
+ }
+})`;
+
+const node = process.execPath;
+execFile(node, [ '-e', script ], common.mustCall((err, stdout, stderr) => {
+ assert(stderr.includes('Error: xyz'), 'userInfo crashes');
+}));

test/parallel/test-regress-GH-7511.js → test/parallel/test-vm-access-process-env.js

@@ -20,6 +20,10 @@
 // USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 'use strict';
+// Tests that node does neither crash nor throw an error when accessing
+// process.env when inside a VM context.
+// See https:/nodejs/node-v0.x-archive/issues/7511.
+
 require('../common');
 const assert = require('assert');
 const vm = require('vm');

test/parallel/test-regress-GH-12371.js → test/parallel/test-vm-api-handles-getter-errors.js

@@ -1,15 +1,13 @@
 'use strict';
+// Tests that vm.createScript and runInThisContext correctly handle errors
+// thrown by option property getters.
+// See https:/nodejs/node/issues/12369.
+
 const common = require('../common');
 const assert = require('assert');
 const execFile = require('child_process').execFile;
 
-const scripts = [
- `os.userInfo({
- get encoding() {
- throw new Error('xyz');
- }
- })`
-];
+const scripts = [];
 
 ['filename', 'cachedData', 'produceCachedData', 'lineOffset', 'columnOffset']
  .forEach((prop) => {