Update Smithy to 1.52.1

.changelog/f2df419d89d44856b08d3e5afb013b83.json

@@ -0,0 +1,12 @@
+{
+    "id": "f2df419d-89d4-4856-b08d-3e5afb013b83",
+    "type": "dependency",
+    "description": "Update Smithy to 1.52.1 and update tests",
+    "modules": [
+        "internal/protocoltest/awsrestjson",
+        "internal/protocoltest/jsonrpc",
+        "internal/protocoltest/jsonrpc10",
+        "internal/protocoltest/restxml",
+        "internal/protocoltest/smithyrpcv2cbor"
+    ]
+}

SMITHY_GO_CODEGEN_VERSION

@@ -1 +1 @@
-e6338ca9b5ee30c6317ea908173a369a9d3eb236
+48250f49d98ad6bed6c302a8af97f62cc7f3bd83

codegen/gradle.properties

@@ -1,2 +1,2 @@
-smithyVersion=1.50.0
+smithyVersion=1.52.1
 smithyGradleVersion=0.7.0

codegen/smithy-aws-go-codegen/src/main/java/software/amazon/smithy/aws/go/codegen/AwsProtocolUtils.java

@@ -150,18 +150,6 @@ public static void generateHttpProtocolTests(GenerationContext context) {
                         .service(ShapeId.from("aws.protocoltests.json10#JsonRpc10"))
                         .operation(ShapeId.from("aws.protocoltests.json10#OperationWithNestedStructure"))
                         .addTestName("AwsJson10ClientPopulatesNestedDefaultValuesWhenMissing")
-                        .build(),
-
-                // TODO these tests are currently failing due to tests not being updated, see #2825
-                HttpProtocolUnitTestGenerator.SkipTest.builder()
-                        .service(ShapeId.from("aws.protocoltests.restjson#RestJson"))
-                        .operation(ShapeId.from("aws.protocoltests.restjson#NullAndEmptyHeadersClient"))
-                        .addTestName("RestJsonNullAndEmptyHeaders")
-                        .build(),
-                HttpProtocolUnitTestGenerator.SkipTest.builder()
-                        .service(ShapeId.from("aws.protocoltests.restxml#RestXml"))
-                        .operation(ShapeId.from("aws.protocoltests.restxml#NullAndEmptyHeadersClient"))
-                        .addTestName("NullAndEmptyHeaders")
                         .build()
         ));
 
@@ -227,6 +215,7 @@ public static void generateHttpProtocolTests(GenerationContext context) {
                         .addTestName("AwsJson10ClientPopulatesDefaultsValuesWhenMissingInResponse")
                         .addTestName("AwsJson10ClientIgnoresDefaultValuesIfMemberValuesArePresentInResponse")
                         .build(),
+                // We don't populate default values if none are sent by the server
                 HttpProtocolUnitTestGenerator.SkipTest.builder()
                         .service(ShapeId.from("aws.protocoltests.json10#JsonRpc10"))
                         .operation(ShapeId.from("aws.protocoltests.json10#OperationWithNestedStructure"))
@@ -236,6 +225,11 @@ public static void generateHttpProtocolTests(GenerationContext context) {
                         .service(ShapeId.from("aws.protocoltests.json10#JsonRpc10"))
                         .operation(ShapeId.from("aws.protocoltests.json10#OperationWithRequiredMembers"))
                         .addTestName("AwsJson10ClientErrorCorrectsWhenServerFailsToSerializeRequiredValues")
+                        .build(),
+                HttpProtocolUnitTestGenerator.SkipTest.builder()
+                        .service(ShapeId.from("aws.protocoltests.json10#JsonRpc10"))
+                        .operation(ShapeId.from("aws.protocoltests.json10#OperationWithRequiredMembersWithDefaults"))
+                        .addTestName("AwsJson10ClientErrorCorrectsWithDefaultValuesWhenServerFailsToSerializeRequiredValues")
                         .build()
         ));