Skip to content

Commit

Permalink
test: fix flaky test-tls-multiple-cas-as-string
Browse files Browse the repository at this point in the history
The following error is emitted in a nondeterministic way on the server
side socket on macOS:

```
events.js:173
      throw er; // Unhandled 'error' event
      ^

Error: read ECONNRESET
    at TLSWrap.onStreamRead (internal/stream_base_commons.js:183:27)
Emitted 'error' event at:
    at emitErrorNT (internal/streams/destroy.js:91:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:59:3)
    at processTicksAndRejections (internal/process/task_queues.js:84:9)
```

Prevent the error from being emitted by moving the `socket.end()` call
to the client. Also, run tests in parallel and use `common.mustCall()`.

PR-URL: #27569
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
  • Loading branch information
lpinca authored and targos committed May 9, 2019
1 parent c53a674 commit b6e540a
Showing 1 changed file with 18 additions and 15 deletions.
33 changes: 18 additions & 15 deletions test/parallel/test-tls-multiple-cas-as-string.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,26 +15,29 @@ const ca2 = fixtures.readKey('ca2-cert.pem', 'utf8');
const cert = fixtures.readKey('agent3-cert.pem', 'utf8');
const key = fixtures.readKey('agent3-key.pem', 'utf8');

function test(ca, next) {
const server = tls.createServer({ ca, cert, key }, function(conn) {
this.close();
conn.end();
});
function test(ca) {
const server = tls.createServer({ ca, cert, key });

server.addContext('agent3', { ca, cert, key });

const host = common.localhostIPv4;
server.listen(0, host, function() {
tls.connect({ servername: 'agent3', host, port: this.address().port, ca });
});

if (next) {
server.once('close', next);
}
server.listen(0, host, common.mustCall(() => {
const socket = tls.connect({
servername: 'agent3',
host,
port: server.address().port,
ca
}, common.mustCall(() => {
socket.end();
}));

socket.on('close', () => {
server.close();
});
}));
}

// `ca1` is not actually necessary for the certificate validation -- maybe
// the fixtures should be written in a way that requires it?
const array = [ca1, ca2];
const string = `${ca1}\n${ca2}`;
test(array, common.mustCall(() => test(string)));
test([ca1, ca2]);
test(`${ca1}\n${ca2}`);

0 comments on commit b6e540a

Please sign in to comment.