diff options
author | James M Snell <jasnell@gmail.com> | 2018-03-19 09:17:23 -0700 |
---|---|---|
committer | James M Snell <jasnell@gmail.com> | 2018-03-22 13:48:47 -0700 |
commit | cef909797af1d716b802f04264e51cf7f26c1b51 (patch) | |
tree | 0aec47d99dc8faf488f8f0e673b6f2113216887b /test/parallel/test-http2-options-max-reserved-streams.js | |
parent | ab755484a8a5113cb6d74f8e8ae5a26c44ae4582 (diff) | |
download | android-node-v8-cef909797af1d716b802f04264e51cf7f26c1b51.tar.gz android-node-v8-cef909797af1d716b802f04264e51cf7f26c1b51.tar.bz2 android-node-v8-cef909797af1d716b802f04264e51cf7f26c1b51.zip |
http2: do not emit our own close emit in Http2Stream
Streams were recently updated to emit their own close event. The
Http2Stream was an exception because it included the close argument
with the close event. Refactor that to use the built in close.
PR-URL: https://github.com/nodejs/node/pull/19451
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Diffstat (limited to 'test/parallel/test-http2-options-max-reserved-streams.js')
-rw-r--r-- | test/parallel/test-http2-options-max-reserved-streams.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/parallel/test-http2-options-max-reserved-streams.js b/test/parallel/test-http2-options-max-reserved-streams.js index 994a881745..6dbf54bf31 100644 --- a/test/parallel/test-http2-options-max-reserved-streams.js +++ b/test/parallel/test-http2-options-max-reserved-streams.js @@ -37,8 +37,8 @@ server.on('stream', common.mustCall((stream) => { pushedStream.respond(); pushedStream.on('aborted', common.mustCall()); pushedStream.on('error', common.mustNotCall()); - pushedStream.on('close', common.mustCall((code) => { - assert.strictEqual(code, 8); + pushedStream.on('close', common.mustCall(() => { + assert.strictEqual(pushedStream.rstCode, 8); countdown.dec(); })); })); |