test: fix test-cluster-net-listen-relative-path.js to run in / · nodejs/node@e16b3e7 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 632
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit e16b3e7

Browse filesBrowse the repository at this point in the historyBrowse files

Trott

authored and

BethGriggs

committed

test: fix test-cluster-net-listen-relative-path.js to run in /

test-cluster-net-listen-relative-path fails if run from the root directory on POSIX because the socket filename isn't quite long enough. Increase it by 2 so that the path length always exceeds 100 characters. PR-URL: #34820 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ricky Zhou <0x19951125@gmail.com>

1 parent b1c3fb7 commit e16b3e7
Copy full SHA for e16b3e7

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-cluster-net-listen-relative-path.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-cluster-net-listen-relative-path.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-cluster-net-listen-relative-path.js

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -17,7 +17,7 @@ const tmpdir = require('../common/tmpdir');
17	17
18	18	// Choose a socket name such that the absolute path would exceed 100 bytes.
19	19	const socketDir = './unix-socket-dir';
20		- const socketName = 'A'.repeat(100 - socketDir.length - 1);
	20	+ const socketName = 'A'.repeat(101 - socketDir.length);
21	21
22	22	// Make sure we're not in a weird environment.
23	23	assert.ok(path.resolve(socketDir, socketName).length > 100,

0 commit comments

Comments

0 (0)