Run Typescript Test CI on all supported Node versions (#1261)

Fix tests for the following packages:

- @smithy/abort-controller: remove unused `useFakeTimers()` call
- @smithy/util-stream-browser: update test global writable for Blob and ReadableStream
- @aws-smithy/server-node: loosen test matcher for connection header

Author: syall

.changeset/fuzzy-lions-flow.md

@@ -0,0 +1,2 @@
+---
+---

.github/workflows/ci.yml

@@ -44,12 +44,16 @@ jobs:
 
   test-typescript:
     runs-on: smithy-typescript_ubuntu-latest_8-core
-    name: TypeScript Test
+    name: TypeScript Test ${{ matrix.node }}
+    strategy:
+      matrix:
+        node: [16, 18, 20, 22]
+
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
-          node-version: 16
+          node-version: ${{ matrix.node }}
           cache: "yarn"
       - uses: gradle/wrapper-validation-action@v2
       - name: Set up JDK 17

packages/abort-controller/src/AbortController.spec.ts

@@ -1,8 +1,6 @@
 import { AbortController } from "./AbortController";
 import { AbortSignal } from "./AbortSignal";
 
-jest.useFakeTimers();
-
 describe("AbortController", () => {
   it("should communicate cancellation via its signal", () => {
     const source = new AbortController();

packages/util-stream-browser/src/sdk-stream-mixin.spec.ts

@@ -14,6 +14,12 @@ jest.mock("@smithy/util-utf8");
 
 const mockStreamCollectorReturn = Uint8Array.from([117, 112, 113]);
 (streamCollector as jest.Mock).mockReturnValue(mockStreamCollectorReturn);
+Object.defineProperty(global, "Blob", {
+  writable: true,
+});
+Object.defineProperty(global, "ReadableStream", {
+  writable: true,
+});
 
 describe(sdkStreamMixin.name, () => {
   const expectAllTransformsToFail = async (sdkStream: SdkStreamMixin) => {

smithy-typescript-ssdk-libs/server-node/src/index.spec.ts

@@ -69,9 +69,9 @@ describe("convertRequest", () => {
     expect(convertedReq.path).toEqual("/");
     expect(convertedReq.protocol).toEqual("http:");
     expect(convertedReq.query).toEqual({});
-    expect(convertedReq.headers).toEqual({
-      connection: "close",
+    expect(convertedReq.headers).toMatchObject({
       host: "example.com",
+      // From LTS 18 -> 20, the connection header defaults from "close" to "keep-alive", so don't test explicitly
     });
     expect(await streamToString(convertedReq.body)).toEqual("");
   });
@@ -92,10 +92,10 @@ describe("convertRequest", () => {
       q: "hello",
       a: "world",
     });
-    expect(convertedReq.headers).toEqual({
-      connection: "close",
+    expect(convertedReq.headers).toMatchObject({
       host: "example.com",
       "content-length": "5",
+      // From LTS 18 -> 20, the connection header defaults from "close" to "keep-alive", so don't test explicitly
     });
     expect(await streamToString(convertedReq.body)).toEqual("hello");
   });
@@ -116,12 +116,12 @@ describe("convertRequest", () => {
     expect(convertedReq.path).toEqual("/some/resource");
     expect(convertedReq.protocol).toEqual("http:");
     expect(convertedReq.query).toEqual({});
-    expect(convertedReq.headers).toEqual({
+    expect(convertedReq.headers).toMatchObject({
       "access-control-request-headers": "origin, x-requested-with",
       "access-control-request-method": "DELETE",
       origin: "https://example.com",
-      connection: "close",
       host: "example.com",
+      // From LTS 18 -> 20, the connection header defaults from "close" to "keep-alive", so don't test explicitly
     });
     expect(await streamToString(convertedReq.body)).toEqual("");
   });