From 0990fc3fa26e208a8d8084d2b37b185407c79f65 Mon Sep 17 00:00:00 2001 From: Eric Freiberg Date: Fri, 6 Oct 2017 10:11:05 -0700 Subject: [PATCH 1/3] test: replaced fixturesDir usage with fixtures module --- test/parallel/test-net-pipe-connect-errors.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/parallel/test-net-pipe-connect-errors.js b/test/parallel/test-net-pipe-connect-errors.js index 5ad77b7209fd4e..097279ad587f5e 100644 --- a/test/parallel/test-net-pipe-connect-errors.js +++ b/test/parallel/test-net-pipe-connect-errors.js @@ -21,6 +21,7 @@ 'use strict'; const common = require('../common'); +const fixtures = require('../common/fixtures'); const fs = require('fs'); const net = require('net'); const path = require('path'); @@ -34,7 +35,7 @@ let emptyTxt; if (common.isWindows) { // on Win, common.PIPE will be a named pipe, so we use an existing empty // file instead - emptyTxt = path.join(common.fixturesDir, 'empty.txt'); + emptyTxt = path.join(fixtures.fixturesDir, 'empty.txt'); } else { common.refreshTmpDir(); // Keep the file name very short so tht we don't exceed the 108 char limit From f6c5553d7a630bdd37821e13d8fcecede54cbdf6 Mon Sep 17 00:00:00 2001 From: Eric Freiberg Date: Sun, 8 Oct 2017 21:31:37 -0500 Subject: [PATCH 2/3] Use fixtures.path function instead of using fixturesDir --- test/parallel/test-net-pipe-connect-errors.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/parallel/test-net-pipe-connect-errors.js b/test/parallel/test-net-pipe-connect-errors.js index 097279ad587f5e..60a269615962cd 100644 --- a/test/parallel/test-net-pipe-connect-errors.js +++ b/test/parallel/test-net-pipe-connect-errors.js @@ -35,7 +35,7 @@ let emptyTxt; if (common.isWindows) { // on Win, common.PIPE will be a named pipe, so we use an existing empty // file instead - emptyTxt = path.join(fixtures.fixturesDir, 'empty.txt'); + emptyTxt = fixtures.path('empty.txt'); } else { common.refreshTmpDir(); // Keep the file name very short so tht we don't exceed the 108 char limit From 8944f3938fe739d3f3515b59e6d066b5741a8e25 Mon Sep 17 00:00:00 2001 From: Eric Freiberg Date: Sun, 8 Oct 2017 21:56:58 -0500 Subject: [PATCH 3/3] Remove unused path import --- test/parallel/test-net-pipe-connect-errors.js | 1 - 1 file changed, 1 deletion(-) diff --git a/test/parallel/test-net-pipe-connect-errors.js b/test/parallel/test-net-pipe-connect-errors.js index 60a269615962cd..91c4f7efd07966 100644 --- a/test/parallel/test-net-pipe-connect-errors.js +++ b/test/parallel/test-net-pipe-connect-errors.js @@ -24,7 +24,6 @@ const common = require('../common'); const fixtures = require('../common/fixtures'); const fs = require('fs'); const net = require('net'); -const path = require('path'); const assert = require('assert'); // Test if ENOTSOCK is fired when trying to connect to a file which is not