Fix empty string/list headers protocol tests (#2403)

* serialize empty strings and lists to headers in restXml and restJson1 * undo build.gradle change
smithy-lang · Sep 23, 2024 · 480be03 · 480be03
1 parent 7813acb
commit 480be03
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/smithy-aws-protocol-tests/model/restJson1/http-headers.smithy b/smithy-aws-protocol-tests/model/restJson1/http-headers.smithy
@@ -424,11 +424,15 @@ operation NullAndEmptyHeadersClient {
 apply NullAndEmptyHeadersClient @httpRequestTests([
     {
         id: "RestJsonNullAndEmptyHeaders",
-        documentation: "Do not send null values, empty strings, or empty lists over the wire in headers",
+        documentation: "Do not send null values, but do send empty strings and empty lists over the wire in headers",
         protocol: restJson1,
         method: "GET",
         uri: "/NullAndEmptyHeadersClient",
-        forbidHeaders: ["X-A", "X-B", "X-C"],
+        forbidHeaders: ["X-A"],
+        headers: {
+            "X-B": ""
+            "X-C": ""
+        }
         body: "",
         params: {
             a: null,

diff --git a/smithy-aws-protocol-tests/model/restXml/http-headers.smithy b/smithy-aws-protocol-tests/model/restXml/http-headers.smithy
@@ -367,11 +367,15 @@ operation NullAndEmptyHeadersClient {
 apply NullAndEmptyHeadersClient @httpRequestTests([
     {
         id: "NullAndEmptyHeaders",
-        documentation: "Do not send null values, empty strings, or empty lists over the wire in headers",
+        documentation: "Do not send null values, but do send empty strings and empty lists over the wire in headers",
         protocol: restXml,
         method: "GET",
         uri: "/NullAndEmptyHeadersClient",
-        forbidHeaders: ["X-A", "X-B", "X-C"],
+        forbidHeaders: ["X-A"],
+        headers: {
+            "X-B": ""
+            "X-C": ""
+        }
         body: "",
         params: {
             a: null,