From 73d713b16e764a2b52028803e14ce4ed8343a0a5 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Thu, 30 Jul 2020 10:46:22 -0700 Subject: [PATCH] test: change Fixes: to Refs: Tests don't fix things generally, so use "Refs:" to refer people to GitHub issues. PR-URL: https://github.com/nodejs/node/pull/34568 Reviewed-By: Richard Lau Reviewed-By: James M Snell Reviewed-By: Luigi Pinca --- test/parallel/test-fs-symlink.js | 2 +- test/parallel/test-http-outgoing-message-inheritance.js | 4 ++-- test/parallel/test-http-server-response-standalone.js | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/test/parallel/test-fs-symlink.js b/test/parallel/test-fs-symlink.js index 28938f41d66010..45bc99b48980a8 100644 --- a/test/parallel/test-fs-symlink.js +++ b/test/parallel/test-fs-symlink.js @@ -58,7 +58,7 @@ fs.symlink(linkData, linkPath, common.mustCall(function(err) { })); tmpdir.refresh(); - // Fixes: https://github.com/nodejs/node/issues/34514 + // Refs: https://github.com/nodejs/node/issues/34514 fs.symlinkSync(Buffer.from(linkData), linkPath); })); diff --git a/test/parallel/test-http-outgoing-message-inheritance.js b/test/parallel/test-http-outgoing-message-inheritance.js index 335a9a28956108..d0da4c68c3118f 100644 --- a/test/parallel/test-http-outgoing-message-inheritance.js +++ b/test/parallel/test-http-outgoing-message-inheritance.js @@ -6,8 +6,8 @@ const { Writable } = require('stream'); const assert = require('assert'); // Check that OutgoingMessage can be used without a proper Socket -// Fixes: https://github.com/nodejs/node/issues/14386 -// Fixes: https://github.com/nodejs/node/issues/14381 +// Refs: https://github.com/nodejs/node/issues/14386 +// Refs: https://github.com/nodejs/node/issues/14381 class Response extends OutgoingMessage { _implicitHeader() {} diff --git a/test/parallel/test-http-server-response-standalone.js b/test/parallel/test-http-server-response-standalone.js index 3c91dd0889b066..ec6d1e89e38525 100644 --- a/test/parallel/test-http-server-response-standalone.js +++ b/test/parallel/test-http-server-response-standalone.js @@ -6,8 +6,8 @@ const { Writable } = require('stream'); const assert = require('assert'); // Check that ServerResponse can be used without a proper Socket -// Fixes: https://github.com/nodejs/node/issues/14386 -// Fixes: https://github.com/nodejs/node/issues/14381 +// Refs: https://github.com/nodejs/node/issues/14386 +// Refs: https://github.com/nodejs/node/issues/14381 const res = new ServerResponse({ method: 'GET',