test: update references to archived repository

tniessen · tniessen · commit d3fbbe056dc9 · 2018-03-04T03:05:22.000+01:00
PR-URL: nodejs#17924 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jon Moss <me@jonathanmoss.me> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
diff --git a/test/parallel/test-assert.js b/test/parallel/test-assert.js
@@ -522,7 +522,7 @@ testAssertionMessage({a: undefined, b: null}, '{ a: undefined, b: null }');
 testAssertionMessage({a: NaN, b: Infinity, c: -Infinity},
                      '{ a: NaN, b: Infinity, c: -Infinity }');
 
-// #2893
+// https://github.com/nodejs/node-v0.x-archive/issues/2893
 try {
   // eslint-disable-next-line no-restricted-syntax
   assert.throws(function() {
@@ -534,7 +534,7 @@ try {
 }
 assert.ok(threw);
 
-// #5292
+// https://github.com/nodejs/node-v0.x-archive/issues/5292
 try {
   assert.strictEqual(1, 2);
 } catch (e) {
diff --git a/test/parallel/test-buffer-alloc.js b/test/parallel/test-buffer-alloc.js
@@ -629,7 +629,8 @@ assert.strictEqual('<Buffer 81 a3 66 6f 6f a3 62 61 72>', x.inspect());
 }
 
 {
-  // #1210 Test UTF-8 string includes null character
+  // https://github.com/nodejs/node-v0.x-archive/pull/1210
+  // Test UTF-8 string includes null character
   let buf = Buffer.from('\0');
   assert.strictEqual(buf.length, 1);
   buf = Buffer.from('\0\0');
@@ -653,7 +654,8 @@ assert.strictEqual('<Buffer 81 a3 66 6f 6f a3 62 61 72>', x.inspect());
 }
 
 {
-  // #243 Test write() with maxLength
+  // https://github.com/nodejs/node-v0.x-archive/issues/243
+  // Test write() with maxLength
   const buf = Buffer.allocUnsafe(4);
   buf.fill(0xFF);
   assert.strictEqual(buf.write('abcd', 1, 2, 'utf8'), 2);
@@ -937,11 +939,13 @@ assert.throws(() => Buffer.allocUnsafe(8).writeFloatLE(0.0, -1), RangeError);
   assert.strictEqual(buf.readIntBE(0, 5), -0x0012000000);
 }
 
-// Regression test for #5482: should throw but not assert in C++ land.
+// Regression test for https://github.com/nodejs/node-v0.x-archive/issues/5482:
+// should throw but not assert in C++ land.
 assert.throws(() => Buffer.from('', 'buffer'), TypeError);
 
-// Regression test for #6111. Constructing a buffer from another buffer
-// should a) work, and b) not corrupt the source buffer.
+// Regression test for https://github.com/nodejs/node-v0.x-archive/issues/6111.
+// Constructing a buffer from another buffer should a) work, and b) not corrupt
+// the source buffer.
 {
   const a = [...Array(128).keys()]; // [0, 1, 2, 3, ... 126, 127]
   const b = Buffer.from(a);
diff --git a/test/parallel/test-cluster-worker-init.js b/test/parallel/test-cluster-worker-init.js
@@ -21,7 +21,7 @@ if (cluster.isMaster) {
     worker.send(msg);
   });
 } else {
-  // GH #7998
+  // https://github.com/nodejs/node-v0.x-archive/issues/7998
   cluster.worker.on('message', (message) => {
     process.send(message === msg);
   });
diff --git a/test/parallel/test-crypto-binary-default.js b/test/parallel/test-crypto-binary-default.js
@@ -393,7 +393,8 @@ fileStream.on('close', common.mustCall(function() {
   );
 }));
 
-// Issue #2227: unknown digest method should throw an error.
+// Unknown digest method should throw an error:
+// https://github.com/nodejs/node-v0.x-archive/issues/2227
 assert.throws(function() {
   crypto.createHash('xyzzy');
 }, /^Error: Digest method not supported$/);
diff --git a/test/parallel/test-crypto-cipher-decipher.js b/test/parallel/test-crypto-cipher-decipher.js
@@ -70,15 +70,16 @@ testCipher1(Buffer.from('MySecretKey123'));
 testCipher2('0123456789abcdef');
 testCipher2(Buffer.from('0123456789abcdef'));
 
-// Base64 padding regression test, see #4837.
+// Base64 padding regression test, see
+// https://github.com/nodejs/node-v0.x-archive/issues/4837.
 {
   const c = crypto.createCipher('aes-256-cbc', 'secret');
   const s = c.update('test', 'utf8', 'base64') + c.final('base64');
   assert.strictEqual(s, '375oxUQCIocvxmC5At+rvA==');
 }
 
 // Calling Cipher.final() or Decipher.final() twice should error but
-// not assert. See #4886.
+// not assert. See https://github.com/nodejs/node-v0.x-archive/issues/4886.
 {
   const c = crypto.createCipher('aes-256-cbc', 'secret');
   try { c.final('xxx'); } catch (e) { /* Ignore. */ }
@@ -90,14 +91,16 @@ testCipher2(Buffer.from('0123456789abcdef'));
   try { d.final('xxx'); } catch (e) { /* Ignore. */ }
 }
 
-// Regression test for #5482: string to Cipher#update() should not assert.
+// Regression test for https://github.com/nodejs/node-v0.x-archive/issues/5482:
+// string to Cipher#update() should not assert.
 {
   const c = crypto.createCipher('aes192', '0123456789abcdef');
   c.update('update');
   c.final();
 }
 
-// #5655 regression tests, 'utf-8' and 'utf8' are identical.
+// https://github.com/nodejs/node-v0.x-archive/issues/5655 regression tests,
+// 'utf-8' and 'utf8' are identical.
 {
   let c = crypto.createCipher('aes192', '0123456789abcdef');
   c.update('update', '');  // Defaults to "utf8".
diff --git a/test/parallel/test-crypto-hash.js b/test/parallel/test-crypto-hash.js
@@ -104,7 +104,8 @@ fileStream.on('close', common.mustCall(function() {
                      'Test SHA1 of sample.png');
 }));
 
-// Issue #2227: unknown digest method should throw an error.
+// Issue https://github.com/nodejs/node-v0.x-archive/issues/2227: unknown digest
+// method should throw an error.
 assert.throws(function() {
   crypto.createHash('xyzzy');
 }, /Digest method not supported/);
diff --git a/test/parallel/test-crypto-random.js b/test/parallel/test-crypto-random.js
@@ -230,8 +230,9 @@ const expectedErrorRegexp = /^TypeError: size must be a number >= 0$/;
   }
 }
 
-// #5126, "FATAL ERROR: v8::Object::SetIndexedPropertiesToExternalArrayData()
-// length exceeds max acceptable value"
+// https://github.com/nodejs/node-v0.x-archive/issues/5126,
+// "FATAL ERROR: v8::Object::SetIndexedPropertiesToExternalArrayData() length
+// exceeds max acceptable value"
 assert.throws(function() {
   crypto.randomBytes((-1 >>> 0) + 1);
 }, /^TypeError: size must be a number >= 0$/);
diff --git a/test/parallel/test-crypto.js b/test/parallel/test-crypto.js
@@ -110,8 +110,8 @@ testImmutability(tls.getCiphers);
 testImmutability(crypto.getHashes);
 testImmutability(crypto.getCurves);
 
-// Regression tests for #5725: hex input that's not a power of two should
-// throw, not assert in C++ land.
+// Regression tests for https://github.com/nodejs/node-v0.x-archive/pull/5725:
+// hex input that's not a power of two should throw, not assert in C++ land.
 assert.throws(function() {
   crypto.createCipher('aes192', 'test').update('0', 'hex');
 }, common.hasFipsCrypto ? /not supported in FIPS mode/ : /Bad input string/);
diff --git a/test/parallel/test-dgram-ref.js b/test/parallel/test-dgram-ref.js
@@ -2,7 +2,7 @@
 const common = require('../common');
 const dgram = require('dgram');
 
-// should not hang, see #1282
+// should not hang, see https://github.com/nodejs/node-v0.x-archive/issues/1282
 dgram.createSocket('udp4');
 dgram.createSocket('udp6');
 
diff --git a/test/parallel/test-dns-regress-7070.js b/test/parallel/test-dns-regress-7070.js
@@ -3,7 +3,8 @@ require('../common');
 const assert = require('assert');
 const dns = require('dns');
 
-// Should not raise assertion error. Issue #7070
+// Should not raise assertion error.
+// Issue https://github.com/nodejs/node-v0.x-archive/issues/7070
 assert.throws(() => dns.resolveNs([]), // bad name
               /^Error: "name" argument must be a string$/);
 assert.throws(() => dns.resolveNs(''), // bad callback
diff --git a/test/parallel/test-timers-unref.js b/test/parallel/test-timers-unref.js
@@ -50,7 +50,8 @@ const check_unref = setInterval(() => {
     setInterval(() => timeout.unref(), SHORT_TIME);
 }
 
-// Should not assert on args.Holder()->InternalFieldCount() > 0. See #4261.
+// Should not assert on args.Holder()->InternalFieldCount() > 0.
+// See https://github.com/nodejs/node-v0.x-archive/issues/4261.
 {
   const t = setInterval(() => {}, 1);
   process.nextTick(t.unref.bind({}));
diff --git a/test/parallel/test-tls-set-encoding.js b/test/parallel/test-tls-set-encoding.js
@@ -42,7 +42,7 @@ server.listen(0, function() {
   client.on('close', function() {
     // readyState is deprecated but we want to make
     // sure this isn't triggering an assert in lib/net.js
-    // See issue #1069.
+    // See https://github.com/nodejs/node-v0.x-archive/issues/1069.
     assert.strictEqual('closed', client.readyState);
 
     // Confirming the buffer string is encoded in ASCII
diff --git a/test/sequential/test-child-process-execsync.js b/test/sequential/test-child-process-execsync.js
@@ -72,7 +72,8 @@ ret = execFileSync(process.execPath, args, { encoding: 'utf8' });
 
 assert.strictEqual(ret, `${msg}\n`);
 
-// Verify that the cwd option works - GH #7824
+// Verify that the cwd option works.
+// See https://github.com/nodejs/node-v0.x-archive/issues/7824.
 {
   const cwd = common.rootDir;
   const cmd = common.isWindows ? 'echo %cd%' : 'pwd';
@@ -81,7 +82,8 @@ assert.strictEqual(ret, `${msg}\n`);
   assert.strictEqual(response.toString().trim(), cwd);
 }
 
-// Verify that stderr is not accessed when stdio = 'ignore' - GH #7966
+// Verify that stderr is not accessed when stdio = 'ignore'.
+// See https://github.com/nodejs/node-v0.x-archive/issues/7966.
 {
   assert.throws(function() {
     execSync('exit -1', {stdio: 'ignore'});
diff --git a/test/sequential/test-module-loading.js b/test/sequential/test-module-loading.js
@@ -181,7 +181,8 @@ const child = require('../fixtures/module-require/child/');
 assert.strictEqual(child.loaded, parent.loaded);
 
 
-// #1357 Loading JSON files with require()
+// Loading JSON files with require()
+// See https://github.com/nodejs/node-v0.x-archive/issues/1357.
 const json = require('../fixtures/packages/main/package.json');
 assert.deepStrictEqual(json, {
   name: 'package-name',
@@ -289,7 +290,8 @@ process.on('exit', function() {
 });
 
 
-// #1440 Loading files with a byte order marker.
+// Loading files with a byte order marker.
+// See https://github.com/nodejs/node-v0.x-archive/issues/1440.
 assert.strictEqual(42, require('../fixtures/utf8-bom.js'));
 assert.strictEqual(42, require('../fixtures/utf8-bom.json'));
 

Original file line number	Diff line number	Diff line change
`@@ -629,7 +629,8 @@ assert.strictEqual('<Buffer 81 a3 66 6f 6f a3 62 61 72>', x.inspect());`
`629`	`629`	`}`
`630`	`630`
`631`	`631`	`{`
`632`		`- // #1210 Test UTF-8 string includes null character`
	`632`	`+ // https://github.com/nodejs/node-v0.x-archive/pull/1210`
	`633`	`+ // Test UTF-8 string includes null character`
`633`	`634`	`let buf = Buffer.from('\0');`
`634`	`635`	`assert.strictEqual(buf.length, 1);`
`635`	`636`	`buf = Buffer.from('\0\0');`
`@@ -653,7 +654,8 @@ assert.strictEqual('<Buffer 81 a3 66 6f 6f a3 62 61 72>', x.inspect());`
`653`	`654`	`}`
`654`	`655`
`655`	`656`	`{`
`656`		`- // #243 Test write() with maxLength`
	`657`	`+ // https://github.com/nodejs/node-v0.x-archive/issues/243`
	`658`	`+ // Test write() with maxLength`
`657`	`659`	`const buf = Buffer.allocUnsafe(4);`
`658`	`660`	`buf.fill(0xFF);`
`659`	`661`	`assert.strictEqual(buf.write('abcd', 1, 2, 'utf8'), 2);`
`@@ -937,11 +939,13 @@ assert.throws(() => Buffer.allocUnsafe(8).writeFloatLE(0.0, -1), RangeError);`
`937`	`939`	`assert.strictEqual(buf.readIntBE(0, 5), -0x0012000000);`
`938`	`940`	`}`
`939`	`941`
`940`		`-// Regression test for #5482: should throw but not assert in C++ land.`
	`942`	`+// Regression test for https://github.com/nodejs/node-v0.x-archive/issues/5482:`
	`943`	`+// should throw but not assert in C++ land.`
`941`	`944`	`assert.throws(() => Buffer.from('', 'buffer'), TypeError);`
`942`	`945`
`943`		`-// Regression test for #6111. Constructing a buffer from another buffer`
`944`		`-// should a) work, and b) not corrupt the source buffer.`
	`946`	`+// Regression test for https://github.com/nodejs/node-v0.x-archive/issues/6111.`
	`947`	`+// Constructing a buffer from another buffer should a) work, and b) not corrupt`
	`948`	`+// the source buffer.`
`945`	`949`	`{`
`946`	`950`	`const a = [...Array(128).keys()]; // [0, 1, 2, 3, ... 126, 127]`
`947`	`951`	`const b = Buffer.from(a);`
Original file line number	Diff line number	Diff line change
`@@ -70,15 +70,16 @@ testCipher1(Buffer.from('MySecretKey123'));`
`70`	`70`	`testCipher2('0123456789abcdef');`
`71`	`71`	`testCipher2(Buffer.from('0123456789abcdef'));`
`72`	`72`
`73`		`-// Base64 padding regression test, see #4837.`
	`73`	`+// Base64 padding regression test, see`
	`74`	`+// https://github.com/nodejs/node-v0.x-archive/issues/4837.`
`74`	`75`	`{`
`75`	`76`	`const c = crypto.createCipher('aes-256-cbc', 'secret');`
`76`	`77`	`const s = c.update('test', 'utf8', 'base64') + c.final('base64');`
`77`	`78`	`assert.strictEqual(s, '375oxUQCIocvxmC5At+rvA==');`
`78`	`79`	`}`
`79`	`80`
`80`	`81`	`// Calling Cipher.final() or Decipher.final() twice should error but`
`81`		`-// not assert. See #4886.`
	`82`	`+// not assert. See https://github.com/nodejs/node-v0.x-archive/issues/4886.`
`82`	`83`	`{`
`83`	`84`	`const c = crypto.createCipher('aes-256-cbc', 'secret');`
`84`	`85`	`try { c.final('xxx'); } catch (e) { /* Ignore. */ }`
`@@ -90,14 +91,16 @@ testCipher2(Buffer.from('0123456789abcdef'));`
`90`	`91`	`try { d.final('xxx'); } catch (e) { /* Ignore. */ }`
`91`	`92`	`}`
`92`	`93`
`93`		`-// Regression test for #5482: string to Cipher#update() should not assert.`
	`94`	`+// Regression test for https://github.com/nodejs/node-v0.x-archive/issues/5482:`
	`95`	`+// string to Cipher#update() should not assert.`
`94`	`96`	`{`
`95`	`97`	`const c = crypto.createCipher('aes192', '0123456789abcdef');`
`96`	`98`	`c.update('update');`
`97`	`99`	`c.final();`
`98`	`100`	`}`
`99`	`101`
`100`		`-// #5655 regression tests, 'utf-8' and 'utf8' are identical.`
	`102`	`+// https://github.com/nodejs/node-v0.x-archive/issues/5655 regression tests,`
	`103`	`+// 'utf-8' and 'utf8' are identical.`
`101`	`104`	`{`
`102`	`105`	`let c = crypto.createCipher('aes192', '0123456789abcdef');`
`103`	`106`	`c.update('update', ''); // Defaults to "utf8".`
Original file line number	Diff line number	Diff line change
`@@ -230,8 +230,9 @@ const expectedErrorRegexp = /^TypeError: size must be a number >= 0$/;`
`230`	`230`	`}`
`231`	`231`	`}`
`232`	`232`
`233`		`-// #5126, "FATAL ERROR: v8::Object::SetIndexedPropertiesToExternalArrayData()`
`234`		`-// length exceeds max acceptable value"`
	`233`	`+// https://github.com/nodejs/node-v0.x-archive/issues/5126,`
	`234`	`+// "FATAL ERROR: v8::Object::SetIndexedPropertiesToExternalArrayData() length`
	`235`	`+// exceeds max acceptable value"`
`235`	`236`	`assert.throws(function() {`
`236`	`237`	`crypto.randomBytes((-1 >>> 0) + 1);`
`237`	`238`	`}, /^TypeError: size must be a number >= 0$/);`