1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: http2 client operations after destroy

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/18845
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
This commit is contained in:
Trivikram 2018-02-17 18:29:22 -08:00 committed by James M Snell
parent e83adf87f5
commit 808c05858a
1 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
test/parallel/test-http2-client-destroy.js
View File

@ -59,7 +59,6 @@ const Countdown = require('../common/countdown');
assert(socket.destroyed); assert(socket.destroyed);
})); }));
const req = client.request(); const req = client.request();
req.on('error', common.expectsError({ req.on('error', common.expectsError({
code: 'ERR_HTTP2_STREAM_CANCEL', code: 'ERR_HTTP2_STREAM_CANCEL',
@ -77,15 +76,21 @@ const Countdown = require('../common/countdown');
message: 'The session has been destroyed' message: 'The session has been destroyed'
}; };
common.expectsError(() => client.request(), sessionError); common.expectsError(() => client.setNextStreamID(), sessionError);
common.expectsError(() => client.ping(), sessionError);
common.expectsError(() => client.settings({}), sessionError); common.expectsError(() => client.settings({}), sessionError);
common.expectsError(() => client.goaway(), sessionError);
common.expectsError(() => client.request(), sessionError);
client.close(); // should be a non-op at this point client.close(); // should be a non-op at this point
// Wait for setImmediate call from destroy() to complete // Wait for setImmediate call from destroy() to complete
// so that state.destroyed is set to true // so that state.destroyed is set to true
setImmediate(() => { setImmediate(() => {
common.expectsError(() => client.request(), sessionError); common.expectsError(() => client.setNextStreamID(), sessionError);
common.expectsError(() => client.ping(), sessionError);
common.expectsError(() => client.settings({}), sessionError); common.expectsError(() => client.settings({}), sessionError);
common.expectsError(() => client.goaway(), sessionError);
common.expectsError(() => client.request(), sessionError);
client.close(); // should be a non-op at this point client.close(); // should be a non-op at this point
}); });