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.

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

Add protocol tests for default values #2002

Merged

Conversation

AndrewFossAWS
Copy link
Contributor

@AndrewFossAWS AndrewFossAWS commented Oct 5, 2023

Description of changes:

  • Add protocol tests for default values
  • Verified tests passed with the latest smithy-rs SDK

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

smithy-aws-protocol-tests/model/awsJson1_0/defaults.smithy Outdated Show resolved Hide resolved
smithy-aws-protocol-tests/model/awsJson1_0/defaults.smithy Outdated Show resolved Hide resolved
smithy-aws-protocol-tests/model/awsJson1_0/defaults.smithy Outdated Show resolved Hide resolved
smithy-aws-protocol-tests/model/awsJson1_0/defaults.smithy Outdated Show resolved Hide resolved
smithy-aws-protocol-tests/model/awsJson1_0/defaults.smithy Outdated Show resolved Hide resolved
smithy-aws-protocol-tests/model/awsJson1_0/required.smithy Outdated Show resolved Hide resolved
smithy-aws-protocol-tests/model/awsJson1_0/required.smithy Outdated Show resolved Hide resolved
smithy-aws-protocol-tests/model/awsJson1_0/required.smithy Outdated Show resolved Hide resolved
smithy-aws-protocol-tests/model/awsJson1_0/required.smithy Outdated Show resolved Hide resolved
@AndrewFossAWS AndrewFossAWS force-pushed the default-protocol-tests branch 5 times, most recently from 88abc9b to 881a441 Compare October 11, 2023 15:23
@AndrewFossAWS AndrewFossAWS force-pushed the default-protocol-tests branch 2 times, most recently from aea9acf to 3a778f5 Compare October 11, 2023 16:49
@AndrewFossAWS AndrewFossAWS marked this pull request as ready for review October 11, 2023 16:49
@AndrewFossAWS AndrewFossAWS requested a review from a team as a code owner October 11, 2023 16:49
Copy link
Contributor

@kstich kstich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also unresolved some comments that weren't completed.

@AndrewFossAWS AndrewFossAWS force-pushed the default-protocol-tests branch 4 times, most recently from 08e70be to 39f2b0b Compare October 25, 2023 01:13
@AndrewFossAWS AndrewFossAWS merged commit 6b2c80c into smithy-lang:main Oct 25, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants