Skip to content

Commit

Permalink
test: remove uneeded alice certs in fixtures/
Browse files Browse the repository at this point in the history
PR-URL: #27962
Reviewed-By: Sam Roberts <[email protected]>
Reviewed-By: Ujjwal Sharma <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
  • Loading branch information
reasonablytall authored and BridgeAR committed Jun 17, 2019
1 parent ee62fa1 commit af9eb96
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 50 deletions.
24 changes: 0 additions & 24 deletions test/fixtures/alice.crt

This file was deleted.

22 changes: 0 additions & 22 deletions test/fixtures/multi-alice.crt

This file was deleted.

8 changes: 4 additions & 4 deletions test/parallel/test-tls-peer-certificate-multi-keys.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,8 @@ const tls = require('tls');
const fixtures = require('../common/fixtures');

const options = {
key: fixtures.readSync('agent.key'),
cert: fixtures.readSync('multi-alice.crt')
key: fixtures.readKey('rsa_private.pem'),
cert: fixtures.readKey('rsa_cert.crt')
};

const server = tls.createServer(options, function(cleartext) {
Expand All @@ -42,7 +42,7 @@ server.once('secureConnection', common.mustCall(function(socket) {
// The server's local cert is the client's peer cert.
assert.deepStrictEqual(
cert.subject.OU,
['Information Technology', 'Engineering', 'Marketing']
['Test TLS Certificate', 'Engineering']
);
}));

Expand All @@ -54,7 +54,7 @@ server.listen(0, common.mustCall(function() {
const peerCert = socket.getPeerCertificate();
assert.deepStrictEqual(
peerCert.subject.OU,
['Information Technology', 'Engineering', 'Marketing']
['Test TLS Certificate', 'Engineering']
);
server.close();
}));
Expand Down

0 comments on commit af9eb96

Please sign in to comment.