nodejs
diff --git a/Collapse file
‎lib/url.js‎
Copy file name to clipboardExpand all lines: lib/url.js
-3Lines changed: 0 additions & 3 deletions b/Collapse file
‎lib/url.js‎
Copy file name to clipboardExpand all lines: lib/url.js
-3Lines changed: 0 additions & 3 deletions
diff --git a/Collapse file
‎test/parallel/test-cluster-http-pipe.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-cluster-http-pipe.js
-1Lines changed: 0 additions & 1 deletion b/Collapse file
‎test/parallel/test-cluster-http-pipe.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-cluster-http-pipe.js
-1Lines changed: 0 additions & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-fs-error-messages.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-fs-error-messages.js
+11-12Lines changed: 11 additions & 12 deletions b/Collapse file
‎test/parallel/test-fs-error-messages.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-fs-error-messages.js
+11-12Lines changed: 11 additions & 12 deletions
diff --git a/Collapse file
‎test/parallel/test-util-isDeepStrictEqual.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-util-isDeepStrictEqual.js
-2Lines changed: 0 additions & 2 deletions b/Collapse file
‎test/parallel/test-util-isDeepStrictEqual.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-util-isDeepStrictEqual.js
-2Lines changed: 0 additions & 2 deletions
@@ -281,9 +281,6 @@ Url.prototype.parse = function parse(url, parseQueryString, slashesDenoteHost) {
     // http://a@b@c/ => user:a@b host:c
     // http://a@b?@c => user:a host:b path:/?@c
 
-    // v0.12 TODO(isaacs): This is not quite how Chrome does things.
-    // Review our test case against browsers more comprehensively.
-
     var hostEnd = -1;
     var atSign = -1;
     var nonHost = -1;
 
@@ -45,7 +45,6 @@ if (cluster.isMaster) {
 http.createServer(common.mustCall((req, res) => {
   assert.strictEqual(req.connection.remoteAddress, undefined);
   assert.strictEqual(req.connection.localAddress, undefined);
-  // TODO common.PIPE?
 
   res.writeHead(200);
   res.end('OK');
 
@@ -636,22 +636,21 @@ if (!common.isAIX) {
   );
 }
 
-// copyFile with invalid flags
+// Check copyFile with invalid flags.
 {
-  const validateError = (err) => {
-    assert.strictEqual(err.message,
-                       'EINVAL: invalid argument, copyfile ' +
-                       `'${existingFile}' -> '${nonexistentFile}'`);
-    assert.strictEqual(err.errno, uv.UV_EINVAL);
-    assert.strictEqual(err.code, 'EINVAL');
-    assert.strictEqual(err.syscall, 'copyfile');
-    return true;
+  const validateError = {
+    // TODO: Make sure the error message always also contains the src.
+    message: `EINVAL: invalid argument, copyfile -> '${nonexistentFile}'`,
+    errno: uv.UV_EINVAL,
+    code: 'EINVAL',
+    syscall: 'copyfile'
   };
 
-  // TODO(joyeecheung): test fs.copyFile() when uv_fs_copyfile does not
-  // keep the loop open when the flags are invalid.
-  // See https://github.com/libuv/libuv/pull/1747
+  fs.copyFile(existingFile, nonexistentFile, -1,
+              common.expectsError(validateError));
 
+  validateError.message = 'EINVAL: invalid argument, copyfile ' +
+                          `'${existingFile}' -> '${nonexistentFile}'`;
   assert.throws(
     () => fs.copyFileSync(existingFile, nonexistentFile, -1),
     validateError
 
@@ -419,8 +419,6 @@ notUtilIsDeepStrict([1, , , 3], [1, , , 3, , , ]);
   const err3 = new TypeError('foo1');
   notUtilIsDeepStrict(err1, err2, assert.AssertionError);
   notUtilIsDeepStrict(err1, err3, assert.AssertionError);
-  // TODO: evaluate if this should throw or not. The same applies for RegExp
-  // Date and any object that has the same keys but not the same prototype.
   notUtilIsDeepStrict(err1, {}, assert.AssertionError);
 }
Original file line number	Diff line number	Diff line change
`@@ -419,8 +419,6 @@ notUtilIsDeepStrict([1, , , 3], [1, , , 3, , , ]);`
`419`	`419`	`const err3 = new TypeError('foo1');`
`420`	`420`	`notUtilIsDeepStrict(err1, err2, assert.AssertionError);`
`421`	`421`	`notUtilIsDeepStrict(err1, err3, assert.AssertionError);`
`422`		`- // TODO: evaluate if this should throw or not. The same applies for RegExp`
`423`		`- // Date and any object that has the same keys but not the same prototype.`
`424`	`422`	`notUtilIsDeepStrict(err1, {}, assert.AssertionError);`
`425`	`423`	`}`
`426`	`424`