chore: update protocol tests

aws · Sep 24, 2024 · a9e0c6f · a9e0c6f
1 parent f18e5d0
commit a9e0c6f
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 3 deletions.
diff --git a/...codegen/src/main/java/software/amazon/smithy/aws/typescript/codegen/AwsProtocolUtils.java b/...codegen/src/main/java/software/amazon/smithy/aws/typescript/codegen/AwsProtocolUtils.java
@@ -312,6 +312,11 @@ private static boolean filterProtocolTests(
             return true;
         }
 
+        // TODO: Remove when upstream tests update to serialize empty headers.
+        if (testCase.getId().contains("NullAndEmptyHeaders")) {
+            return true;
+        }
+
         if (testCase.getTags().contains("defaults")) {
             return true;
         }

diff --git a/private/aws-protocoltests-restjson/test/functional/restjson1.spec.ts b/private/aws-protocoltests-restjson/test/functional/restjson1.spec.ts
@@ -6719,7 +6719,7 @@ it("RestJsonNoInputAndOutputNoPayload:Response", async () => {
 /**
  * Do not send null values, empty strings, or empty lists over the wire in headers
  */
-it("RestJsonNullAndEmptyHeaders:Request", async () => {
+it.skip("RestJsonNullAndEmptyHeaders:Request", async () => {
   const client = new RestJsonProtocolClient({
     ...clientParams,
     requestHandler: new RequestSerializationTestHandler(),

diff --git a/private/aws-protocoltests-restxml/test/functional/restxml.spec.ts b/private/aws-protocoltests-restxml/test/functional/restxml.spec.ts
@@ -3735,7 +3735,7 @@ it("NoInputAndOutput:Response", async () => {
 /**
  * Do not send null values, empty strings, or empty lists over the wire in headers
  */
-it("NullAndEmptyHeaders:Request", async () => {
+it.skip("NullAndEmptyHeaders:Request", async () => {
   const client = new RestXmlProtocolClient({
     ...clientParams,
     requestHandler: new RequestSerializationTestHandler(),

diff --git a/private/aws-restjson-server/test/functional/restjson1.spec.ts b/private/aws-restjson-server/test/functional/restjson1.spec.ts
@@ -27857,7 +27857,7 @@ it("RestJsonNoInputAndOutputWithJson:ServerResponse", async () => {
 /**
  * Do not send null or empty headers
  */
-it("RestJsonNullAndEmptyHeaders:ServerResponse", async () => {
+it.skip("RestJsonNullAndEmptyHeaders:ServerResponse", async () => {
   class TestService implements Partial<RestJsonService<{}>> {
     NullAndEmptyHeadersServer(input: any, ctx: {}): Promise<NullAndEmptyHeadersServerServerOutput> {
       const response = {