Skip to content

Commit

Permalink
test: shorten path for bogus socket
Browse files Browse the repository at this point in the history
This fixes CI failures for test-net-pipe-connect-errors on Raspberry Pi
devices.

PR-URL: #4478
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
Trott authored and Fishrock123 committed Jan 6, 2016
1 parent eee5082 commit d46d850
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions test/parallel/test-net-pipe-connect-errors.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,12 @@ if (common.isWindows) {
// file instead
emptyTxt = path.join(common.fixturesDir, 'empty.txt');
} else {
// use common.PIPE to ensure we stay within POSIX socket path length
// restrictions, even on CI
common.refreshTmpDir();
emptyTxt = common.PIPE + '.txt';
// Keep the file name very short so tht we don't exceed the 108 char limit
// on CI for a POSIX socket. Even though this isn't actually a socket file,
// the error will be different from the one we are expecting if we exceed the
// limit.
emptyTxt = common.tmpDir + '0.txt';

function cleanup() {
try {
Expand Down

0 comments on commit d46d850

Please sign in to comment.