From fd541197c8538ceff8149f3e72f18c83869760d5 Mon Sep 17 00:00:00 2001 From: Bruce Fletcher Date: Fri, 6 Oct 2017 09:25:00 -0700 Subject: [PATCH 1/3] test: add common.fixtures to https-req-split --- test/parallel/test-https-req-split.js | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/test/parallel/test-https-req-split.js b/test/parallel/test-https-req-split.js index d0306cb714fb71..eae7982a64d807 100644 --- a/test/parallel/test-https-req-split.js +++ b/test/parallel/test-https-req-split.js @@ -21,6 +21,7 @@ 'use strict'; const common = require('../common'); +const fixtures = require('../common/fixtures'); if (!common.hasCrypto) common.skip('missing crypto'); @@ -33,8 +34,8 @@ const tls = require('tls'); const fs = require('fs'); const options = { - key: fs.readFileSync(`${common.fixturesDir}/keys/agent1-key.pem`), - cert: fs.readFileSync(`${common.fixturesDir}/keys/agent1-cert.pem`) + key: fs.readFileSync(`${fixtures.fixturesDir}/keys/agent1-key.pem`), + cert: fs.readFileSync(`${fixtures.fixturesDir}/keys/agent1-cert.pem`) }; // Force splitting incoming data From ec9ab4dd0c9153da5112ce372b029d8f5237d01e Mon Sep 17 00:00:00 2001 From: Bruce Fletcher Date: Fri, 6 Oct 2017 09:56:14 -0700 Subject: [PATCH 2/3] test: replaced readFileSync with fixtures.readKey --- test/parallel/test-https-req-split.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/parallel/test-https-req-split.js b/test/parallel/test-https-req-split.js index eae7982a64d807..b47ac21926bbb9 100644 --- a/test/parallel/test-https-req-split.js +++ b/test/parallel/test-https-req-split.js @@ -34,8 +34,8 @@ const tls = require('tls'); const fs = require('fs'); const options = { - key: fs.readFileSync(`${fixtures.fixturesDir}/keys/agent1-key.pem`), - cert: fs.readFileSync(`${fixtures.fixturesDir}/keys/agent1-cert.pem`) + key: fixtures.readKey(`agent1-key.pem`), + cert: fixtures.readKey(`agent1-cert.pem`) }; // Force splitting incoming data From 3e56bfef1003ec138488371b1134d961e0ae5365 Mon Sep 17 00:00:00 2001 From: Bruce Fletcher Date: Fri, 6 Oct 2017 10:17:19 -0700 Subject: [PATCH 3/3] test: fixed linting issues I caused with test-https-req-split --- test/parallel/test-https-req-split.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/test/parallel/test-https-req-split.js b/test/parallel/test-https-req-split.js index b47ac21926bbb9..05fc85fc07b781 100644 --- a/test/parallel/test-https-req-split.js +++ b/test/parallel/test-https-req-split.js @@ -31,11 +31,10 @@ process.env.NODE_TLS_REJECT_UNAUTHORIZED = '0'; const https = require('https'); const tls = require('tls'); -const fs = require('fs'); const options = { - key: fixtures.readKey(`agent1-key.pem`), - cert: fixtures.readKey(`agent1-cert.pem`) + key: fixtures.readKey('agent1-key.pem'), + cert: fixtures.readKey('agent1-cert.pem') }; // Force splitting incoming data