Skip to content

Commit

Permalink
test: add test-http2-large-file sequential test
Browse files Browse the repository at this point in the history
Refs: #19141

PR-URL: #22254
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: George Adams <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
  • Loading branch information
jasnell authored and rvagg committed Aug 15, 2018
1 parent cee78bf commit a414b07
Showing 1 changed file with 39 additions and 0 deletions.
39 changes: 39 additions & 0 deletions test/sequential/test-http2-large-file.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
'use strict';

// Test to ensure sending a large stream with a large initial window size works
// See: https://github.com/nodejs/node/issues/19141

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

const http2 = require('http2');

const server = http2.createServer({ settings: { initialWindowSize: 6553500 } });
server.on('stream', (stream) => {
stream.resume();
stream.respond();
stream.end('ok');
});

server.listen(0, common.mustCall(() => {
let remaining = 1e8;
const chunk = 1e6;
const client = http2.connect(`http://localhost:${server.address().port}`,
{ settings: { initialWindowSize: 6553500 } });
const request = client.request({ ':method': 'POST' });
function writeChunk() {
if (remaining > 0) {
remaining -= chunk;
request.write(Buffer.alloc(chunk, 'a'), writeChunk);
} else {
request.end();
}
}
writeChunk();
request.on('close', common.mustCall(() => {
client.close();
server.close();
}));
request.resume();
}));

0 comments on commit a414b07

Please sign in to comment.