test: http2 client operations after destroy

PR-URL: #18845 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
nodejs · Feb 21, 2018 · f90490d · f90490d
1 parent e1f8273
commit f90490d
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/test/parallel/test-http2-client-destroy.js b/test/parallel/test-http2-client-destroy.js
@@ -59,7 +59,6 @@ const Countdown = require('../common/countdown');
       assert(socket.destroyed);
     }));
 
-
     const req = client.request();
     req.on('error', common.expectsError({
       code: 'ERR_HTTP2_STREAM_CANCEL',
@@ -77,15 +76,21 @@ const Countdown = require('../common/countdown');
       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.goaway(), sessionError);
+    common.expectsError(() => client.request(), sessionError);
     client.close();  // should be a non-op at this point
 
     // Wait for setImmediate call from destroy() to complete
     // so that state.destroyed is set to true
     setImmediate(() => {
-      common.expectsError(() => client.request(), sessionError);
+      common.expectsError(() => client.setNextStreamID(), sessionError);
+      common.expectsError(() => client.ping(), 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
     });