Add query-compatible protocol tests

[peterrsongg]

Description

Motivation and Context

Testing

Dry run passes

Screenshots (if appropriate)

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• My code follows the code style of this project
• My change requires a change to the documentation
• I have updated the documentation accordingly
• I have read the README document
• I have added tests to cover my changes
• All new and existing tests passed

License

• I confirm that this pull request can be released under the Apache 2 license

[dscpinheiro]

More of a general comment, but this PR would've been easier to review if there were multiple commits (e.g. one for the protocol tests generator and another for the actual generated changes).

[dscpinheiro]

What happened to these tests? Did the Smithy team remove them? (I couldn't find the equivalent removals in the generator/TestServiceModels/ changes)

[peterrsongg]

So I added a new test-service called QueryCompatibleJsonRPC10. I didn't actually touch anything related to this QueryCompatible service that existed so I'm not sure how this got removed. This isn't something smithy related, I believe this is something we created manually https://github.com/aws/aws-sdk-net/tree/main/generator/TestServiceModels/aws-query-compatible

The one I added though is the shapes for the smithy protocol tests. I also think that since we are consuming the smity protocol tests for query-compatbile we probably don't need these ones. We also have custom unit tests from the SEP as well

[dscpinheiro]

Got it. Looking at the changes commit by commit maybe it was the generator/ProtocolTestsGenerator/smithy-dotnet-codegen/src/main/java/software/amazon/smithy/dotnet/codegen/HttpProtocolTestGenerator.java file?

But no need to change it now, we should revisit these hand-written test services later.

[peterrsongg]

More of a general comment, but this PR would've been easier to review if there were multiple commits (e.g. one for the protocol tests generator and another for the actual generated changes).

my bad i was in a hurry and didn't fix up the commits. I just fixed them up though to make it a bit easier.

[dscpinheiro]

Got it. Looking at the changes commit by commit maybe it was the generator/ProtocolTestsGenerator/smithy-dotnet-codegen/src/main/java/software/amazon/smithy/dotnet/codegen/HttpProtocolTestGenerator.java file?

But no need to change it now, we should revisit these hand-written test services later.