Skip to content

Commit 7e34bde

Browse files
Update protocol test skip list
1 parent 92a3056 commit 7e34bde

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

codegen/core/src/main/java/software/amazon/smithy/python/codegen/integrations/RestJsonProtocolGenerator.java

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,9 @@ public class RestJsonProtocolGenerator implements ProtocolGenerator {
5252
"SDKAppliedContentEncoding_restJson1",
5353
"SDKAppendedGzipAfterProvidedEncoding_restJson1",
5454

55-
// TODO: update union deserialization to ignore `__type` for JSON protocols
56-
"RestJsonDeserializeIgnoreType",
55+
// TODO: These tests require a payload even when the httpPayload member is null. Should it?
56+
"RestJsonHttpWithHeadersButNoPayload",
57+
"RestJsonHttpWithEmptyStructurePayload",
5758

5859
// These tests do need to be fixed, but they're being disabled right now
5960
// since the way protocols work is changing.
@@ -62,10 +63,6 @@ public class RestJsonProtocolGenerator implements ProtocolGenerator {
6263
"RestJsonClientUsesExplicitlyProvidedMemberValuesOverDefaults",
6364
"RestJsonClientIgnoresNonTopLevelDefaultsOnMembersWithClientOptional",
6465
"RestJsonClientPopulatesDefaultsValuesWhenMissingInResponse",
65-
"RestJsonClientIgnoresDefaultValuesIfMemberValuesArePresentInResponse",
66-
"RestJsonClientPopulatesNestedDefaultsWhenMissingInResponseBody",
67-
"RestJsonHttpPrefixEmptyHeaders",
68-
"RestJsonNullAndEmptyHeaders",
6966
"HttpPrefixEmptyHeaders");
7067

7168
@Override

0 commit comments

Comments
 (0)