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 <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
This commit is contained in:
parent
1052383f7c
commit
0101a8f1a6
@ -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;
|
||||
|
Loading…
x
Reference in New Issue
Block a user