Skip to content
Permalink
Browse files

test: verify order of error in h2 server stream

Currently the order of error / closing of an h2 stream is consistent
in 10.x, 11.x, and master. There appears to be an unexpected behavior
difference in 8.x. This test will be used to bisect the commit that will
fix this behavior change and ensure there are no future regressions.

PR-URL: #24685
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information...
MylesBorins authored and BethGriggs committed Nov 27, 2018
1 parent 288a421 commit e74345b2f5b4bf8c5033754191c88fd074e311f3
Showing with 43 additions and 0 deletions.
  1. +43 −0 test/parallel/test-http2-error-order.js
@@ -0,0 +1,43 @@
'use strict';

const common = require('../common');
if (!common.hasCrypto)
common.skip('missing crypto');

const assert = require('assert');
const { createServer, connect } = require('http2');

const messages = [];
const expected = [
'Stream:created',
'Stream:error',
'Stream:close',
'Request:error'
];

const server = createServer();

server.on('stream', (stream) => {
messages.push('Stream:created');
stream
.on('close', () => messages.push('Stream:close'))
.on('error', (err) => messages.push('Stream:error'))
.respondWithFile('dont exist');
});

server.listen(0);

const client = connect(`http://localhost:${server.address().port}`);
const req = client.request();

req.on('response', common.mustNotCall());

req.on('error', () => {
messages.push('Request:error');
client.close();
});

client.on('close', common.mustCall(() => {
assert.deepStrictEqual(messages, expected);
server.close();
}));

0 comments on commit e74345b

Please sign in to comment.
You can’t perform that action at this time.