Smithy 1.17.0 update and server protocol test workaround (#3288)

* chore(codegen): update to Smithy-1.17.0

* fix(codegen): workaround server protocol tests

Author: adamthom-amzn

Diff for: codegen/build.gradle.kts

@@ -31,7 +31,7 @@ allprojects {
     version = "0.8.0"
 }
 
-extra["smithyVersion"] = "[1.16.3,1.17.0["
+extra["smithyVersion"] = "[1.17.0,1.18.0["
 
 // The root project doesn't produce a JAR.
 tasks["jar"].enabled = false

Diff for: codegen/smithy-aws-typescript-codegen/src/main/java/software/amazon/smithy/aws/typescript/codegen/AwsProtocolUtils.java

@@ -314,6 +314,12 @@ private static boolean filterProtocolTests(
             return true;
         }
 
+        // TODO remove after 1.17.1 or later
+        // https://github.com/awslabs/smithy/pull/1084
+        if (testCase.getId().equals("RestJsonOutputUnionWithUnitMember")) {
+            return true;
+        }
+
         return false;
     }
 

Diff for: private/aws-protocoltests-restjson/src/models/models_0.ts

@@ -1435,7 +1435,7 @@ export namespace PlayerAction {
 }
 
 export interface PostPlayerActionInput {
-  action: PlayerAction | undefined;
+  action?: PlayerAction;
 }
 
 export namespace PostPlayerActionInput {

Diff for: private/aws-protocoltests-restxml/test/functional/restxml.spec.ts

@@ -6473,7 +6473,7 @@ it("XmlUnionsWithBooleanMember:Request", async () => {
 });
 
 /**
- * Serializes union union member
+ * Serializes union member
  */
 it("XmlUnionsWithUnionMember:Request", async () => {
   const client = new RestXmlProtocolClient({
@@ -6680,7 +6680,7 @@ it("XmlUnionsWithBooleanMember:Response", async () => {
 });
 
 /**
- * Serializes union union member
+ * Serializes union member
  */
 it("XmlUnionsWithUnionMember:Response", async () => {
   const client = new RestXmlProtocolClient({

Diff for: scripts/generate-clients/copy-to-clients.js

@@ -177,13 +177,30 @@ const copyServerTests = async (sourceDir, destinationDir) => {
             directory: `private/${testName}`,
           },
         };
+        if (!mergedManifest.scripts.test) {
+          mergedManifest.scripts.test = "jest --coverage --passWithNoTests";
+        }
         writeFileSync(destSubPath, JSON.stringify(mergedManifest, null, 2).concat(`\n`));
       } else if (overWritableSubs.includes(packageSub) || !existsSync(destSubPath)) {
         if (lstatSync(packageSubPath).isDirectory()) removeSync(destSubPath);
         copySync(packageSubPath, destSubPath, {
           overwrite: true,
         });
       }
+      const jestConfigPath = join(destPath, "jest.config.js");
+      writeFileSync(
+        jestConfigPath,
+        'const base = require("../../jest.config.base.js");\n' +
+          "\n" +
+          "module.exports = {\n" +
+          "  ...base,\n" +
+          "  globals: {\n" +
+          "    'ts-jest': {\n" +
+          "      isolatedModules: true\n" +
+          "    }\n" +
+          "  }\n" +
+          "};\n"
+      );
     }
   }
 };