test: use relative path in pipePrefix

rhanford · targos · commit 8173b0c36371 · 2017-10-17T23:33:37.000-07:00
Modified pipePrefix to use relative path on windows, previously tests failed when the full path was 120+ characters PR-URL: #15988 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/test/common/index.js b/test/common/index.js
@@ -258,7 +258,8 @@ Object.defineProperty(exports, 'hasFipsCrypto', {
 });
 
 {
-  const pipePrefix = exports.isWindows ? '\\\\.\\pipe\\' : `${exports.tmpDir}/`;
+  const localRelative = path.relative(process.cwd(), `${exports.tmpDir}/`);
+  const pipePrefix = exports.isWindows ? '\\\\.\\pipe\\' : localRelative;
   const pipeName = `node-test.${process.pid}.sock`;
   exports.PIPE = pipePrefix + pipeName;
 }

Original file line number	Diff line number	Diff line change
`@@ -258,7 +258,8 @@ Object.defineProperty(exports, 'hasFipsCrypto', {`
`258`	`258`	`});`
`259`	`259`
`260`	`260`	`{`
`261`		- const pipePrefix = exports.isWindows ? '\\\\.\\pipe\\' : `${exports.tmpDir}/`;
	`261`	+ const localRelative = path.relative(process.cwd(), `${exports.tmpDir}/`);
	`262`	`+ const pipePrefix = exports.isWindows ? '\\\\.\\pipe\\' : localRelative;`
`262`	`263`	const pipeName = `node-test.${process.pid}.sock`;
`263`	`264`	`exports.PIPE = pipePrefix + pipeName;`
`264`	`265`	`}`