Skip to content

Commit 3d37efe

Browse files
kuriyoshdanielleadams
authored andcommitted
test: improve _http_outgoing coverage
PR-URL: #42213 Refs: https://coverage.nodejs.org/coverage-29bb2bb57d2a993e/lib/_http_outgoing.js.html Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Juan José Arboleda <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
1 parent 7fe16ec commit 3d37efe

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

test/parallel/test-http-outgoing-internal-headernames-getter.js

+10
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
const common = require('../common');
33

44
const { OutgoingMessage } = require('http');
5+
const assert = require('assert');
56

67
const warn = 'OutgoingMessage.prototype._headerNames is deprecated';
78
common.expectWarning('DeprecationWarning', warn, 'DEP0066');
@@ -11,3 +12,12 @@ common.expectWarning('DeprecationWarning', warn, 'DEP0066');
1112
const outgoingMessage = new OutgoingMessage();
1213
outgoingMessage._headerNames; // eslint-disable-line no-unused-expressions
1314
}
15+
16+
{
17+
// Tests _headerNames getter result after setting a header.
18+
const outgoingMessage = new OutgoingMessage();
19+
outgoingMessage.setHeader('key', 'value');
20+
const expect = Object.create(null);
21+
expect.key = 'key';
22+
assert.deepStrictEqual(outgoingMessage._headerNames, expect);
23+
}

0 commit comments

Comments
 (0)