From bb9ed92f0a4ada8c0ed297c4b8042c853481fb3c Mon Sep 17 00:00:00 2001 From: Lam Chan Date: Fri, 6 Oct 2017 10:04:46 -0700 Subject: [PATCH] test: replaced fs.readSync with fixtures.readSync NINA 2017 Code and Learn session for first time contributors. This touched test/parallel/test-https-socket-options.js to replace common.fixturesDir PR-URL: https://github.com/nodejs/node/pull/15882 Reviewed-By: Colin Ihrig Reviewed-By: James M Snell Reviewed-By: Luigi Pinca Reviewed-By: Gireesh Punathil --- test/parallel/test-https-socket-options.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/parallel/test-https-socket-options.js b/test/parallel/test-https-socket-options.js index 2a6fc8f84f..2ba1b88368 100644 --- a/test/parallel/test-https-socket-options.js +++ b/test/parallel/test-https-socket-options.js @@ -25,13 +25,13 @@ const common = require('../common'); if (!common.hasCrypto) common.skip('missing crypto'); +const fixtures = require('../common/fixtures'); const https = require('https'); -const fs = require('fs'); const http = require('http'); const options = { - key: fs.readFileSync(`${common.fixturesDir}/keys/agent1-key.pem`), - cert: fs.readFileSync(`${common.fixturesDir}/keys/agent1-cert.pem`) + key: fixtures.readKey('agent1-key.pem'), + cert: fixtures.readKey('agent1-cert.pem') }; const body = 'hello world\n';