http2: replace unreachable error with assertion

"That particular `emit('error', ...)` is largely defensively coded and
should not ever actually happen." Sounds like an assertion rather than
an error event. The code in question has no test coverage because it is
believed to be unreachable.

Fixes: #20673

PR-URL: #24407
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>

Author: Trott

lib/internal/http2/compat.js

@@ -1,5 +1,6 @@
 'use strict';
 
+const assert = require('assert');
 const Stream = require('stream');
 const Readable = Stream.Readable;
 const binding = process.binding('http2');
@@ -331,15 +332,12 @@ class Http2ServerRequest extends Readable {
 
   _read(nread) {
     const state = this[kState];
-    if (!state.closed) {
-      if (!state.didRead) {
-        state.didRead = true;
-        this[kStream].on('data', onStreamData);
-      } else {
-        process.nextTick(resumeStream, this[kStream]);
-      }
+    assert(!state.closed);
+    if (!state.didRead) {
+      state.didRead = true;
+      this[kStream].on('data', onStreamData);
     } else {
-      this.emit('error', new ERR_HTTP2_INVALID_STREAM());
+      process.nextTick(resumeStream, this[kStream]);
     }
   }