chore(codegen): enable RestJSON tests disabled during smithy 1.14.x b…

…ump (#3062) * chore(codegen): enable RestJSON tests disabled during smithy 1.14.x bump * chore(protocol_tests): enable RestJSON tests disabled during smithy 1.14.x bump
aws · Aug 31, 2023 · 010f9d6 · 010f9d6
1 parent 27fd7b7
commit 010f9d6
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 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
@@ -355,15 +355,6 @@ private static boolean filterProtocolTests(
             HttpMessageTestCase testCase,
             TypeScriptSettings settings
     ) {
-        // TODO: Remove when server protocol tests are fixed in
-        // https://github.com/aws/aws-sdk-js-v3/issues/3058
-        // TODO: Move to filter specific to server protocol tests if added in
-        // https://github.com/awslabs/smithy-typescript/issues/470
-        if (testCase.getId().equals("RestJsonTestPayloadStructure")
-            || testCase.getId().equals("RestJsonHttpWithHeadersButNoPayload")) {
-            return true;
-        }
-
         // TODO: Remove when requestCompression has been implemented.
         if (testCase.getId().startsWith("SDKAppliedContentEncoding_")
             || testCase.getId().startsWith("SDKAppendsGzipAndIgnoresHttpProvidedEncoding_")

diff --git a/private/aws-protocoltests-restjson/test/functional/restjson1.spec.ts b/private/aws-protocoltests-restjson/test/functional/restjson1.spec.ts
@@ -8915,7 +8915,7 @@ it("RestJsonHttpWithEmptyStructurePayload:Request", async () => {
 /**
  * Serializes a payload targeting a structure
  */
-it.skip("RestJsonTestPayloadStructure:Request", async () => {
+it("RestJsonTestPayloadStructure:Request", async () => {
   const client = new RestJsonProtocolClient({
     ...clientParams,
     requestHandler: new RequestSerializationTestHandler(),
@@ -8955,7 +8955,7 @@ it.skip("RestJsonTestPayloadStructure:Request", async () => {
 /**
  * Serializes an request with header members but no payload
  */
-it.skip("RestJsonHttpWithHeadersButNoPayload:Request", async () => {
+it("RestJsonHttpWithHeadersButNoPayload:Request", async () => {
   const client = new RestJsonProtocolClient({
     ...clientParams,
     requestHandler: new RequestSerializationTestHandler(),