Skip to content

Commit

Permalink
test: add a test for Expect & checkExpectation
Browse files Browse the repository at this point in the history
New test case for Expect header & checkExpectation event based on the
existing http test case.

PR-URL: nodejs#15040
Refs: nodejs#14985
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
apapirovski authored and cjihrig committed Aug 30, 2017
1 parent d728442 commit 952ab8c
Showing 1 changed file with 46 additions and 0 deletions.
46 changes: 46 additions & 0 deletions test/parallel/test-http2-compat-expect-handling.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
// Flags: --expose-http2
'use strict';

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

const expectValue = 'meoww';

const server = http2.createServer(common.mustNotCall());

server.once('checkExpectation', common.mustCall((req, res) => {
assert.strictEqual(req.headers['expect'], expectValue);
res.statusCode = 417;
res.end();
}));

server.listen(0, common.mustCall(() => nextTest(2)));

function nextTest(testsToRun) {
if (!testsToRun) {
return server.close();
}

const port = server.address().port;
const client = http2.connect(`http://localhost:${port}`);
const req = client.request({
':path': '/',
':method': 'GET',
':scheme': 'http',
':authority': `localhost:${port}`,
expect: expectValue
});

req.on('response', common.mustCall((headers) => {
assert.strictEqual(headers[':status'], 417);
req.resume();
}));

req.on('end', common.mustCall(() => {
client.destroy();
nextTest(testsToRun - 1);
}));
}

0 comments on commit 952ab8c

Please sign in to comment.