From 0101a8f1a6eb363e1ae5ac7aca19223e37025f78 Mon Sep 17 00:00:00 2001 From: coreybeaumont Date: Sat, 22 Apr 2017 13:22:57 -0700 Subject: test: add relative path to accommodate limit Found that libuv had a path character limit of 108. Used path.relative() to set prefix with relative path. Also added comments explaining the use of a relative path. PR-URL: https://github.com/nodejs/node/pull/12601 Reviewed-By: Anna Henningsen Reviewed-By: James M Snell Reviewed-By: Rich Trott Reviewed-By: Colin Ihrig Reviewed-By: Gibson Fahnestock --- test/parallel/test-net-connect-options-fd.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'test/parallel/test-net-connect-options-fd.js') diff --git a/test/parallel/test-net-connect-options-fd.js b/test/parallel/test-net-connect-options-fd.js index 5a0db83184..9cc581243f 100644 --- a/test/parallel/test-net-connect-options-fd.js +++ b/test/parallel/test-net-connect-options-fd.js @@ -2,6 +2,7 @@ const common = require('../common'); const assert = require('assert'); const net = require('net'); +const path = require('path'); const Pipe = process.binding('pipe_wrap').Pipe; if (common.isWindows) { @@ -32,7 +33,9 @@ const forAllClients = (cb) => common.mustCall(cb, CLIENT_VARIANTS); // Test Pipe fd is wrapped correctly { - const prefix = `${common.PIPE}-net-connect-options-fd`; + // Use relative path to avoid hitting 108-char length limit + // for socket paths in libuv. + const prefix = path.relative('.', `${common.PIPE}-net-connect-options-fd`); const serverPath = `${prefix}-server`; let counter = 0; let socketCounter = 0; -- cgit v1.2.1