Skip to content
This repository has been archived by the owner on Aug 31, 2018. It is now read-only.

Commit

Permalink
test: use common/fixtures in tls-connect-no-host
Browse files Browse the repository at this point in the history
PR-URL: nodejs/node#15986
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
dtex authored and addaleax committed Oct 15, 2017
1 parent 67aa2d5 commit 0e6746b
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions test/parallel/test-tls-connect-no-host.js
Original file line number Diff line number Diff line change
@@ -1,17 +1,16 @@
'use strict';
const common = require('../common');
const fixtures = require('../common/fixtures');

if (!common.hasCrypto)
common.skip('missing crypto');

const tls = require('tls');

const assert = require('assert');
const fs = require('fs');
const path = require('path');

const cert = fs.readFileSync(path.join(common.fixturesDir, 'test_cert.pem'));
const key = fs.readFileSync(path.join(common.fixturesDir, 'test_key.pem'));
const cert = fixtures.readSync('test_cert.pem');
const key = fixtures.readSync('test_key.pem');

// https://github.com/nodejs/node/issues/1489
// tls.connect(options) with no options.host should accept a cert with
Expand Down

0 comments on commit 0e6746b

Please sign in to comment.