Unverified Commit 1fbafc83 authored by Russell Cohen's avatar Russell Cohen Committed by GitHub
Browse files

Fix default features test (#2881) 

## Motivation and Context
<!--- Why is this change required? What problem does it solve? -->
<!--- If it fixes an open issue, please link to the issue here -->

## Description
The default features test is actually broken and failed during the
release dry run

## Testing
- ~Added the default features test to CI~ got a linker error


----

_By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice._
parent 2e82790a

aws/sdk/integration-tests/dynamodb/Cargo.toml

+1 −1
Original line number Diff line number Diff line
@@ -16,7 +16,7 @@ aws-config = { path = "../../build/aws-sdk/sdk/aws-config" } 
aws-credential-types = { path = "../../build/aws-sdk/sdk/aws-credential-types", features = ["test-util"] }

aws-http = { path = "../../build/aws-sdk/sdk/aws-http" }

aws-sdk-dynamodb = { path = "../../build/aws-sdk/sdk/dynamodb" }

aws-smithy-async = { path = "../../build/aws-sdk/sdk/aws-smithy-async" }

aws-smithy-async = { path = "../../build/aws-sdk/sdk/aws-smithy-async", features = ["test-util"] }

aws-smithy-client = { path = "../../build/aws-sdk/sdk/aws-smithy-client", features = ["test-util", "rustls"] }

aws-smithy-http = { path = "../../build/aws-sdk/sdk/aws-smithy-http" }

aws-smithy-protocol-test = { path = "../../build/aws-sdk/sdk/aws-smithy-protocol-test" }

aws/sdk/integration-tests/no-default-features/tests/client-construction.rs

+3 −3
Original line number Diff line number Diff line
@@ -9,10 +9,10 @@ use aws_smithy_async::rt::sleep::AsyncSleep; 
use std::time::Duration;



// This will fail due to lack of a connector when constructing the SDK Config

// If this test doesn't panic, you may have accidentally unified features, resulting in

// the connector being enabled transitively

#[tokio::test]

#[should_panic(

    expected = "No HTTP connector was available. Enable the `rustls` crate feature or set a connector to fix this."

)]

#[should_panic(expected = "Enable the `rustls` crate feature or set a connector to fix this.")]

async fn test_clients_from_sdk_config() {

    aws_config::load_from_env().await;

}