Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix unbound operations/shapes in protocol tests #560

Merged
merged 1 commit into from
Sep 10, 2020
Merged

Fix unbound operations/shapes in protocol tests #560

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion smithy-aws-protocol-tests/model/awsJson1_0/empty-input-output.smithy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,4 +123,3 @@ apply EmptyInputAndEmptyOutput @httpResponseTests([
])

structure EmptyInputAndEmptyOutputInput {}
structure EmptyInputAndEmptyOutputOutput {}
5 changes: 5 additions & 0 deletions smithy-aws-protocol-tests/model/awsQuery/main.smithy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,14 +27,19 @@ service AwsQuery {
QueryMaps,
QueryIdempotencyTokenAutoFill,

// Output tests
XmlEmptyBlobs,

// Output XML map tests
XmlMaps,
XmlMapsXmlName,
FlattenedXmlMap,
FlattenedXmlMapWithXmlName,
XmlEmptyMaps,

// Output XML list tests
XmlLists,
XmlEmptyLists,

// Output XML structure tests
SimpleScalarXmlProperties,
Expand Down
4 changes: 4 additions & 0 deletions smithy-aws-protocol-tests/model/ec2Query/main.smithy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,8 +51,12 @@ service AwsEc2 {
QueryLists,
QueryIdempotencyTokenAutoFill,

// Output tests
XmlEmptyBlobs,

// Output XML list tests
XmlLists,
XmlEmptyLists,

// Output XML structure tests
SimpleScalarXmlProperties,
Expand Down
4 changes: 2 additions & 2 deletions smithy-aws-protocol-tests/model/restXml/document-lists.smithy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -218,7 +218,7 @@ apply XmlLists @httpResponseTests([
])

@idempotent
@http(uri: "/XmlLists", method: "PUT")
@http(uri: "/XmlEmptyLists", method: "PUT")
@tags(["client-only"])
operation XmlEmptyLists {
input: XmlListsInputOutput,
Expand All @@ -231,7 +231,7 @@ apply XmlEmptyLists @httpRequestTests([
documentation: "Serializes Empty XML lists",
protocol: restXml,
method: "PUT",
uri: "/XmlLists",
uri: "/XmlEmptyLists",
body: """
<XmlListsInputOutput>
<stringList></stringList>
Expand Down
4 changes: 2 additions & 2 deletions smithy-aws-protocol-tests/model/restXml/document-maps.smithy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ apply XmlMaps @httpResponseTests([
}
])

@http(uri: "/XmlMaps", method: "POST")
@http(uri: "/XmlEmptyMaps", method: "POST")
@tags(["client-only"])
operation XmlEmptyMaps {
input: XmlMapsInputOutput,
Expand All @@ -113,7 +113,7 @@ apply XmlEmptyMaps @httpRequestTests([
documentation: "Serializes Empty XML maps",
protocol: restXml,
method: "POST",
uri: "/XmlMaps",
uri: "/XmlEmptyMaps",
body: """
<XmlMapsInputOutput>
<myMap></myMap>
Expand Down
2 changes: 1 addition & 1 deletion smithy-aws-protocol-tests/model/restXml/document-structs.smithy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -350,7 +350,7 @@ apply XmlBlobs @httpResponseTests([
])

/// Blobs are base64 encoded
@http(uri: "/XmlBlobs", method: "POST")
@http(uri: "/XmlEmptyBlobs", method: "POST")
@tags(["client-only"])
operation XmlEmptyBlobs {
input: XmlBlobsInputOutput,
Expand Down
6 changes: 6 additions & 0 deletions smithy-aws-protocol-tests/model/restXml/main.smithy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,9 @@ service RestXml {
NoInputAndNoOutput,
NoInputAndOutput,
EmptyInputAndEmptyOutput,
XmlEmptyLists,
XmlEmptyMaps,
XmlEmptyStrings,

// @httpHeader tests
InputAndOutputWithHeaders,
Expand Down Expand Up @@ -48,6 +51,9 @@ service RestXml {
HttpPayloadWithXmlNamespace,
HttpPayloadWithXmlNamespaceAndPrefix,

// Output tests
XmlEmptyBlobs,

// Errors
GreetingWithErrors,

Expand Down