http2: general cleanups in core.js

* fixup js debug messages
* simplify and improve rstStream
* improve and simplify _read
* simplify and improve priority
* simplify on ready a bit
* simplify and improve respond/push
* reduce duplication with _unrefActive
* simplify stream close handling

PR-URL: https://github.com/nodejs/node/pull/17209
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>

1 files changed, 0 insertions, 57 deletions

diff --git a/test/parallel/test-http2-priority-parent-self.js b/test/parallel/test-http2-priority-parent-self.js
deleted file mode 100644
index 55a161bf17..0000000000
--- a/test/parallel/test-http2-priority-parent-self.js
+++ /dev/null
@@ -1,57 +0,0 @@
-'use strict';
-
-const common = require('../common');
-if (!common.hasCrypto)
-  common.skip('missing crypto');
-const h2 = require('http2');
-
-const server = h2.createServer();
-const invalidOptValueError = (value) => ({
-  type: TypeError,
-  code: 'ERR_INVALID_OPT_VALUE',
-  message: `The value "${value}" is invalid for option "parent"`
-});
-
-// we use the lower-level API here
-server.on('stream', common.mustCall((stream) => {
-  common.expectsError(
-    () => stream.priority({
-      parent: stream.id,
-      weight: 1,
-      exclusive: false
-    }),
-    invalidOptValueError(stream.id)
-  );
-  stream.respond({
-    'content-type': 'text/html',
-    ':status': 200
-  });
-  stream.end('hello world');
-}));
-
-server.listen(0, common.mustCall(() => {
-
-  const client = h2.connect(`http://localhost:${server.address().port}`);
-  const req = client.request({ ':path': '/' });
-
-  req.on(
-    'ready',
-    () => common.expectsError(
-      () => req.priority({
-        parent: req.id,
-        weight: 1,
-        exclusive: false
-      }),
-      invalidOptValueError(req.id)
-    )
-  );
-
-  req.on('response', common.mustCall());
-  req.resume();
-  req.on('end', common.mustCall(() => {
-    server.close();
-    client.destroy();
-  }));
-  req.end();
-
-}));