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 a generator for @httpMalformedRequestTests #393

Merged
merged 1 commit into from
Aug 5, 2021

Conversation

adamthom-amzn
Copy link
Contributor

@adamthom-amzn adamthom-amzn commented Jul 28, 2021

Description of changes:
@httpMalformedRequestTests allow server implementations to generate tests
that exercise how the implementation deals with requests that are rejected
before they reach the customer's business logic. See
smithy-lang/smithy#871 for more information.

This will not succeed build checks until smithy-lang/smithy#871 is committed and released.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@adamthom-amzn adamthom-amzn requested a review from a team as a code owner July 28, 2021 21:06
Copy link
Contributor

@gosar gosar left a comment

Choose a reason for hiding this comment

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

LG. Where would I see an output of these generators?

@httpMalformedRequestTests allow server implementations to generate tests
that exercise how the implementation deals with requests that are rejected
before they reach the customer's business logic. See
smithy-lang/smithy#871 for more information.
@trivikr trivikr merged commit 904bfb0 into smithy-lang:main Aug 5, 2021
srchase pushed a commit to srchase/smithy-typescript that referenced this pull request Mar 17, 2023
@httpMalformedRequestTests allow server implementations to generate tests
that exercise how the implementation deals with requests that are rejected
before they reach the customer's business logic. See
smithy-lang/smithy#871 for more information.
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.

4 participants