Fix slightly incorrect remotePort test assertions

The ephemeral port range starts at 32768, not 32769, so we should accept
32768 as a valid client port in these tests - it should be gte, not gt.

Author: pimterry

package.json

@@ -119,8 +119,8 @@
     "browserify-zlib": "^0.2.0",
     "buffer": "^6.0.3",
     "catch-uncommitted": "^1.0.0",
-    "chai": "^3.5.0",
-    "chai-as-promised": "^6.0.0",
+    "chai": "^4.3.7",
+    "chai-as-promised": "^7.1.1",
     "chai-fetch": "^0.3.1",
     "crypto-browserify": "^3.12.0",
     "dns2": "1.4.2",

test/integration/http2.spec.ts

@@ -290,7 +290,9 @@ nodeOnly(() => {
 
                     const proxiedResponseHeaders = await getHttp2Response(proxiedRequest);
 
-                    expect(_.omit(proxiedResponseHeaders, 'date')).to.deep.equal({
+                    expect(
+                        _.omit(proxiedResponseHeaders, ['date', http2.sensitiveHeaders])
+                    ).to.deep.equal({
                         ':status': 200,
                         'header-key': 'HEADER-VALUE' // We lowercase all header keys
                         // Connection: close is omitted
@@ -414,7 +416,7 @@ nodeOnly(() => {
                     '::ffff:127.0.0.1', // IPv4 localhost
                     '::1' // IPv6 localhost
                 ]);
-                expect(seenRequest.remotePort).to.be.greaterThan(32768);
+                expect(seenRequest.remotePort).to.be.greaterThanOrEqual(32768);
 
                 await cleanup(proxiedClient, client);
             });

test/integration/proxying/https-proxying.spec.ts

@@ -780,7 +780,8 @@ nodeOnly(() => {
                 const response = await http2ProxyRequest(server, `https://localhost:${targetPort}/`);
 
                 expect(
-                    _.omit(response.headers, 'date') // https://github.com/nodejs/node/issues/34841
+                    // N.b. https://github.com/nodejs/node/issues/34841
+                    _.omit(response.headers, ['date', http2.sensitiveHeaders])
                 ).to.deep.equal({
                     ':status': 200,
                     'replacement-header': 'added'

test/integration/subscriptions/client-error-events.spec.ts

@@ -61,7 +61,7 @@ describe("Client error subscription", () => {
                 '::ffff:127.0.0.1', // IPv4 localhost
                 '::1' // IPv6 localhost
             ]);
-            expect(clientError.request.remotePort).to.be.greaterThan(32768);
+            expect(clientError.request.remotePort).to.be.greaterThanOrEqual(32768);
 
             const response = clientError.response as CompletedResponse;
             expect(response.statusCode).to.equal(431);

test/integration/subscriptions/request-events.spec.ts

@@ -64,7 +64,7 @@ describe("Request initiated subscriptions", () => {
                 '::ffff:127.0.0.1', // IPv4 localhost
                 '::1' // IPv6 localhost
             ]);
-            expect(seenRequest.remotePort).to.be.greaterThan(32768);
+            expect(seenRequest.remotePort).to.be.greaterThanOrEqual(32768);
             expect((seenRequest as any).body).to.equal(undefined); // No body included yet
 
             const matchableHeaders = _.omit(seenRequest.headers, INCONSISTENT_HEADERS);
@@ -261,7 +261,7 @@ describe("Request subscriptions", () => {
                 '::ffff:127.0.0.1', // IPv4 localhost
                 '::1' // IPv6 localhost
             ]);
-            expect(seenRequest.remotePort).to.be.greaterThan(32768);
+            expect(seenRequest.remotePort).to.be.greaterThanOrEqual(32768);
             expect(await seenRequest.body.getText()).to.equal('body-text');
             expect(seenRequest.tags).to.deep.equal([]);
         });

test/integration/subscriptions/tls-error-events.spec.ts

@@ -98,7 +98,7 @@ describe("TLS error subscriptions", () => {
             '::ffff:127.0.0.1', // IPv4 localhost
             '::1' // IPv6 localhost
         ]);
-        expect(tlsError.remotePort).to.be.greaterThan(32768);
+        expect(tlsError.remotePort).to.be.greaterThanOrEqual(32768);
         expect(tlsError.tags).to.deep.equal([]);
 
         expect(tlsError.timingEvents.startTime).to.be.greaterThan(0);
@@ -137,7 +137,7 @@ describe("TLS error subscriptions", () => {
                 '::ffff:127.0.0.1', // IPv4 localhost
                 '::1' // IPv6 localhost
             ]);
-            expect(tlsError.remotePort).to.be.greaterThan(32768);
+            expect(tlsError.remotePort).to.be.greaterThanOrEqual(32768);
 
             expect(tlsError.timingEvents.startTime).to.be.greaterThan(0);
             expect(tlsError.timingEvents.connectTimestamp).to.be.greaterThan(0);