Skip to content

Commit

Permalink
test: replaced fs.readSync with fixtures.readSync
Browse files Browse the repository at this point in the history
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: #15882
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
  • Loading branch information
Lam Chan authored and MylesBorins committed Nov 28, 2017
1 parent aefaed4 commit 2ab826c
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/parallel/test-https-socket-options.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,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';
Expand Down

0 comments on commit 2ab826c

Please sign in to comment.