Skip to content

Commit

Permalink
test: replace fixturesDir with fixtures module
Browse files Browse the repository at this point in the history
`test/parallel/test-tls-ecdh.js` was loading fixture keys using
`common.fixturesDir`. Now it's using `fixtures.readKey` instead.

PR-URL: #15893
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
  • Loading branch information
Ivan Etchart authored and MylesBorins committed Oct 11, 2017
1 parent 36a0d3f commit f7ab126
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/parallel/test-tls-ecdh.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@

'use strict';
const common = require('../common');
const fixtures = require('../common/fixtures');

if (!common.hasCrypto)
common.skip('missing crypto');
Expand All @@ -32,11 +33,10 @@ const assert = require('assert');
const tls = require('tls');

const exec = require('child_process').exec;
const fs = require('fs');

const options = {
key: fs.readFileSync(`${common.fixturesDir}/keys/agent2-key.pem`),
cert: fs.readFileSync(`${common.fixturesDir}/keys/agent2-cert.pem`),
key: fixtures.readKey('agent2-key.pem'),
cert: fixtures.readKey('agent2-cert.pem'),
ciphers: '-ALL:ECDHE-RSA-AES128-SHA256',
ecdhCurve: 'prime256v1'
};
Expand Down

0 comments on commit f7ab126

Please sign in to comment.