Unverified Commit 26f1624d authored by Aaron Todd's avatar Aaron Todd Committed by GitHub
Browse files

ignore content length integration test (#3620)

## Motivation and Context

https://github.com/smithy-lang/smithy-rs/pull/3618 is blocked by semver
hazards check because the current version of this test in `aws-sdk-rust`
expects the test to
[panic](https://github.com/smithy-lang/smithy-rs/blob/release-2024-04-30/aws/sdk/integration-tests/s3/tests/streaming-response.rs#L23).
To land the content length enforcement PR we need to ignore the test and
release `smithy-rs` and a version of the SDK with the test ignored
instead of expecting it to panic.

----

_By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice._
parent 773d36e3
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -20,7 +20,7 @@ use tracing::debug;

// TODO(https://github.com/smithy-lang/smithy-rs/issues/3523): Unignore this test
#[tokio::test]
#[should_panic]
#[ignore]
async fn test_too_short_body_causes_an_error() {
    // this is almost impossible to reproduce with Hyper—you need to do stuff like run each request
    // in its own async runtime. But there's no reason a customer couldn't run their _own_ HttpClient