From 3eb029794c61e013c236e70e99ff659f6ea0826a Mon Sep 17 00:00:00 2001
From: Anatoli Papirovski <apapirovski@mac.com>
Date: Fri, 25 Aug 2017 13:19:28 -0400
Subject: [PATCH] test: increase coverage for http2 response headers

Expanded an existing test for setting pseudo-headers on response to
include all pseudo-headers, not just :status.

PR-URL: https://github.com/nodejs/node/pull/15035
Refs: https://github.com/nodejs/node/issues/14985
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
---
 ...est-http2-compat-serverresponse-headers.js | 21 ++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/test/parallel/test-http2-compat-serverresponse-headers.js b/test/parallel/test-http2-compat-serverresponse-headers.js
index 7a9b7d66405704..512a14c1c0cf8f 100644
--- a/test/parallel/test-http2-compat-serverresponse-headers.js
+++ b/test/parallel/test-http2-compat-serverresponse-headers.js
@@ -42,13 +42,20 @@ server.listen(0, common.mustCall(function() {
     response.removeHeader(denormalised);
     assert.strictEqual(response.hasHeader(denormalised), false);
 
-    assert.throws(function() {
-      response.setHeader(':status', 'foobar');
-    }, common.expectsError({
-      code: 'ERR_HTTP2_PSEUDOHEADER_NOT_ALLOWED',
-      type: Error,
-      message: 'Cannot set HTTP/2 pseudo-headers'
-    }));
+    [
+      ':status',
+      ':method',
+      ':path',
+      ':authority',
+      ':scheme'
+    ].forEach((header) => assert.throws(
+      () => response.setHeader(header, 'foobar'),
+      common.expectsError({
+        code: 'ERR_HTTP2_PSEUDOHEADER_NOT_ALLOWED',
+        type: Error,
+        message: 'Cannot set HTTP/2 pseudo-headers'
+      })
+    ));
     assert.throws(function() {
       response.setHeader(real, null);
     }, common.expectsError({